[Bug c/17178] [3.5 Regression] static consts and -Wunused-variable

[pinskia at gcc dot gnu dot org]

------- Additional Comments From pinskia at gcc dot gnu dot org  2004-08-24 23:53 -------
Mark, could you settle  this one as Zack said he did it deliberately but did not change the 
documentation because:
[16:32] < pinskia> zwol: http://gcc.gnu.org/ml/gcc-patches/2003-05/msg00468.html
[16:44] * stevenb (~steven@70-90.ipact.nl) has left the channel
[16:45] < zwol> pinskia: you might deduce from my response that i think that change was wrong.
[16:45] < pinskia> well it is still a regression which needs to be fixed.
[16:46] < pinskia> I should note that Mac OS X has a huge number of these static const variables in the 
headers is we are going to warn about them
[16:47] < zwol> static const char*, sure, i can see the rcsid[] argument.  but static const int, that's 
different, that's someone expecting C++ const semantics, and it's very likely their program is not doing 
what they thought it was.
[16:47] < zwol> Mac OS X headers should be fixed then.
[16:48] < zwol> no, i have no sympathy.
[16:48] < zwol> why do you ask?
[16:49] < pinskia> I don't care it is a regression and that is all I am going to say
[16:50] < pinskia> Should I quote the manual  which you did not change when you committed the 
change?
[16:50] < pinskia> Warn whenever a local variable or non-constant static variable is unused aside from 
its declaration
[16:50] < pinskia> "non-constant static"
[16:50] * gadek (~gadek@209.29.internet.uqam.ca) has joined
[16:51] < zwol> I do not consider the manual to be Free and therefore (while I will fix obvious 
problems that are pointed out to me) I will not look through it for problems when I make changes.
[16:51] < zwol> and, my response to "it is a regression" is "no, it is a deliberate change, justify why it 
should be changed back."
[16:51] < zwol> and if you don't care to justify it, then i don't care.
[16:52] < pinskia> but you did not justify why it was changed in the first place which means it was not 
a deliberate change 

-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |mmitchel at gcc dot gnu dot
                   |                            |org


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=17178