On Tue, Mar 29, 2005 at 01:52:32PM -0500, Nicholas Wourms wrote: >Corinna Vinschen wrote: >Since I assume you and Chris are quite familiar with gcc, I will not >waste your time by going into detail regarding what all pedantic >covers. The bottom line is that giving the developer the ability to >compile at that level can provide one more opportunity to catch a bug >before it manifests itself.
You have correctly surmised that both Corinna and I understand what pedantic mode is. You have to take that thought a step further, however, and realize that the fact that there is no -pedantic in CFLAGS is because we both want to use the full expressive power of gcc without worrying if we are complying with ISO C or ISO C++. This is a conscious choice, not an oversight. >>and it's an annoying mess to have to care for this all the time instead >>of fixing the real problems. > >Give me a break, the work is already done (by me). This is no more >difficult to maintain then any other attribute flags. You set it and >forget it. Remove it when you remove the statement, it doesn't get >easier then that. I'm not asking you to do the work of identifying >issues which do not allow compiling at pedantic mode, I've already done >that. I thought Corinna's intent was clear in that sentence. Apparently you didn't. What she was saying is, for this to be done right, both of us would have to be rigorous in the future about making sure that we add decorate every extension we use, or, worse, avoid using gcc extensions which we've come to rely on. That would be annoying. However, if I am correctly interpreting your intent, it sounds like you are saying that no one but you would have to worry about sprinkling __extension__'s throughout the code and that we could just write code as we always do. If that is the case, then I don't see how it matters if we check in your code or not. You'll constantly be updating things to match the latest checkins one way or the other. However, if your patches are not going to be checked in, then you don't have to worry about packaging up your changes for cygwin-patches, which is less work for you. If you are expecting that both of us will sign onto the need to use __extension__ and use it for all new code then your "give me a break" statement does not make much sense. Btw, the use of a ?: c is a conscious decision. cgf