Hi again,
it seems opinions are dived about how and what should we do.
I agree that turning on errors for unused parameter for Objective-C
files is undesirable, because we can't omit the parameter name.
Additionally it isn't easy (or maybe it is impossible) to build only
Objective-C sources
Hi all,
I noticed many patches to fix newly introduced unused
parameter warnings on GTK and EFL ports nowadays.
(ToT EFL WebKit is warning free right now.)
CMake ports now handle unused parameters as warnings, not build
errors everywhere. (-Werror and -Wno-error=unused-parameter )
Mac port
On Jan 26, 2015, at 6:40 AM, Osztrogonác Csaba o...@inf.u-szeged.hu wrote:
Hi all,
I noticed many patches to fix newly introduced unused
parameter warnings on GTK and EFL ports nowadays.
(ToT EFL WebKit is warning free right now.)
CMake ports now handle unused parameters as warnings,
I have found serious bugs in WebKit with unused parameter warnings; I found
some when first turning on the warning. I don’t have a specific story to tell,
but I remember there was more than one case.
So far it’s been impractical to turn on this warning for our Objective-C code,
since we can’t
On Jan 26, 2015, at 9:28 AM, Darin Adler da...@apple.com wrote:
I have found serious bugs in WebKit with unused parameter warnings; I found
some when first turning on the warning. I don’t have a specific story to
tell, but I remember there was more than one case.
So far it’s been
On Jan 26, 2015, at 10:11 AM, Filip Pizlo fpi...@apple.com wrote:
So I actually do the opposite of pushing the configuration conditionals to
the edges - I put them into the innards.
Yes, this explains why the unused argument warnings are so frustrating for you!
I don’t know what it says for
Are there reasons any or all of these shouldn't be written as macros?
Yes: Macros are terrible.
Darin mentioned to me in person a way to resolve the problems I listed. I’ll
follow up with a patch soon.
Geoff___
webkit-dev mailing list
On Mon, Jan 26, 2015 at 2:31 PM, Geoffrey Garen gga...@apple.com wrote:
And I do agree that it can be inconvenient to deal with these warnings in
heavily #ifdef’d code. I think there are some good strategies for avoiding
that, and I’d like to talk about specific irritating cases so I can
8 matches
Mail list logo