https://bugzilla.wikimedia.org/show_bug.cgi?id=19157


Brion Vibber <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |REOPENED
         Resolution|FIXED                       |




--- Comment #6 from Brion Vibber <[email protected]>  2009-08-24 00:11:35 UTC 
---
Copying my notes from Code Review on r52494:

Eww... this is a really ugly calling convention.

If the function is named 'isSomething', it needs to return a boolean otherwise
you're just asking for trouble. An optional outparam for returning a message
key would be much less annoying in this context.


-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
You are on the CC list for the bug.

_______________________________________________
Wikibugs-l mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l

Reply via email to