Updates:
Status: Invalid
Cc: [email protected]
Comment #1 on issue 2302 by [email protected]: include/v8.h unused
parameter warnings in g++ with show all warnings enabled
http://code.google.com/p/v8/issues/detail?id=2302
There are 2 reasons why we won't change this:
* A very technical one: Leaving out parameter names violates our style
guide, so it won't even go through our pre-submit scripts.
* A less technical one: I don't think that the -Wunused-parameter flag
makes sense in most circumstances because there are tons of valid C++
code/patters where parameters are not used. One could use an 'unused'
attribute in GCC, but I think this is too compiler-dependent, heavyweight
and total overkill. (Some Haskell compilers use a much more lightweight
mechanism for such circumstances by not warning about unused arguments
starting with a '_').
How did you compile v8? In the old scons build we even explicitly set
-Wno-unused-parameter, perhaps this is missing in the new GYP build. Jakob?
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
