[Webkit-unassigned] [Bug 81733] New: Make Length Calculation functions regular
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Tue Mar 20 20:50:00 PDT 2012
https://bugs.webkit.org/show_bug.cgi?id=81733
Summary: Make Length Calculation functions regular
Product: WebKit
Version: 528+ (Nightly build)
Platform: All
OS/Version: All
Status: UNCONFIRMED
Severity: Normal
Priority: P2
Component: CSS
AssignedTo: webkit-unassigned at lists.webkit.org
ReportedBy: joethomas at motorola.com
Currently length calculation functions in LengthFunctions.h are inline.
A decent rule of thumb is to not inline a function if it is more than 10 lines long. Also it's typically not cost effective to inline functions with loops or switch statements (Reference: http://google-styleguide.googlecode.com/svn/trunk/cppguide.xml#Inline_Functions). Each of these functions are expected to grow by 10 more line as part of bug 27160.
Also for bug 27160, in order to calculate viewport size using RenderView class, the inclusion of RenderView.h in LengthFunctions.h cause circular dependency between headers and it fails to compile, so it's better to separate the implementation of these functions.
--
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