This is the mail archive of the gcc@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: Warnings about rcs_id strings: let's settle this


"Kean Johnston" <jkj@sco.com> writes:

| > For what it's worth, I agree that some variant of Kean's patch should
| > be included.  I think some of the concerns raised in the discussion of
| > the patch (enable at -Wextra? only for const variables?) should be
| > addressed, but hopefully we can do that in short order without
| > sidetracking again.
| 
| I too agree. I've read the whole thread, and had some other discussions
| about the patch, and here's what I'm about to change it to do. I'd like
| a nod from Joe, Gaby and you Zack, at the very least before I go
| charging
| off into vim-land again.
| 
| 1) -Wall will NOT turn on -Wunused-static-variable
| 2) -Wextra WILL turn on -Wunused-static-variable
| 3) -Wunused-static-variable disappears out of c-opts.c and stays only
|    in flags.h and toplev.c

These bits are OK with me.

-- Gaby


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]