[Webkit-unassigned] [Bug 60638] check-webkit-style should warn about types we normally pass as reference

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Fri May 13 09:02:30 PDT 2011


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





--- Comment #8 from David Levin <levin at chromium.org>  2011-05-13 09:02:30 PST ---
It looks like people are being guided to using "const IntPoint&"

https://bugs.webkit.org/show_bug.cgi?id=60679#c5

Perhaps the guideline is if it begins with a capital letter (aka a custom defined type), then it is a pass by ref (const is optional) or pointer but not by value.

Then if there are exceptions to this rule we can special case those types. To say that they should be pass by value, but it sounds like IntPoint is not an exception.

-- 
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