[Webkit-unassigned] [Bug 26988] Haiku-specific files for WebCore
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Mon Jul 20 13:20:00 PDT 2009
https://bugs.webkit.org/show_bug.cgi?id=26988
--- Comment #35 from Ryan Leavengood <leavengood at gmail.com> 2009-07-20 13:19:57 PDT ---
(In reply to comment #34)
>
> This is a new style that WebKit is moving to that likely post dates the other
> implementations, so that's why they use an enum.
>
> For example see https://bugs.webkit.org/show_bug.cgi?id=26695, where this was
> said when someone was adding a new bool parameter:
> We're trying to avoid adding boolean parameters, because they are not very
> clear (when reading code like "hitTestResultAtPoint(point, true, false,
> true)",
> what do the booleans mean?). One strategy to replace a boolean parameter is
> to
> use a two-value enum, like this:
>
> enum HitTestScrollbars { ShouldHitTestScrollbars, DontHitTestScrollbars };
For what it is worth, I agree with David on this. It definitely makes sense and
since we are a new port, let's move in a better direction, especially if the
rest of WebKit is going that way.
Let's also maybe keep it in mind for the browser code too.
Regards,
Ryan
--
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