Summary: | Add absoluteValue method for LayoutUnits to allow overloading abs() | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Levi Weintraub <leviw> | ||||
Component: | Layout and Rendering | Assignee: | Levi Weintraub <leviw> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | darin, eae, eric, jchaffraix | ||||
Priority: | P2 | ||||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Bug Depends on: | |||||||
Bug Blocks: | 60318 | ||||||
Attachments: |
|
Description
Levi Weintraub
2012-04-30 11:16:03 PDT
Created attachment 139485 [details]
Patch
Comment on attachment 139485 [details]
Patch
Darin Adler is a better reviewer for this than I am. But this is OK for me. I'd like you to wait a few hours and see if you can get him to comment as well though.
The reason why I suggest Darin, is that he might have a clever suggestion for your std::abs() problem other than this one. (In reply to comment #3) > The reason why I suggest Darin, is that he might have a clever suggestion for your std::abs() problem other than this one. Deal! I'll wait till near EOD PST. (In reply to comment #4) > (In reply to comment #3) > > The reason why I suggest Darin, is that he might have a clever suggestion for your std::abs() problem other than this one. > > Deal! I'll wait till near EOD PST. Unless there's argument, I'll move forward on landing this. Committed r115691: <http://trac.webkit.org/changeset/115691> Comment on attachment 139485 [details]
Patch
Clearing flags
Comment on attachment 139485 [details]
Patch
Only thing I might have done differently would have been to overload it for int and FractionalLayoutUnit from the start rather than defining it as a function that takes LayoutUnit.
|