[Webkit-unassigned] [Bug 80760] WinLauncher should show loading errors

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu Mar 15 09:56:12 PDT 2012


https://bugs.webkit.org/show_bug.cgi?id=80760





--- Comment #29 from Brent Fulgham <bfulgham at webkit.org>  2012-03-15 09:56:12 PST ---
(In reply to comment #28)
> Created an attachment (id=132055)
 --> (https://bugs.webkit.org/attachment.cgi?id=132055&action=review) [details]
> Patch
> 
> Why parameters unused in the implementation should be removed from the declaration is inexplicable to me.

The pragmatic reason is that it silences a compiler warning, making it easier to notice warnings that have real consequences.

The higher-level reason is that it signals to people reading the sources that the parameter is not used.

If you like, you can comment out the name of the parameter (e.g., "... double /*blah*/, int used, ...").

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.



More information about the webkit-unassigned mailing list