[Webkit-unassigned] [Bug 25494] add PositionConstructors.h in preparation for adding more position constructor functions

[bugzilla-daemon at webkit.org]

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





------- Comment #11 from eric at webkit.org  2009-06-01 16:00 PDT -------
(In reply to comment #7)
> (From update of attachment 29924 [review])
> I don't think calling this header PositionConstructors.h is right.
> "Constructor" means a specific thing in C++, and these aren't it, so it's kind
> of confusing.
> 
> I actually think it's fine for these functions to be in Position.h, since they
> are effectively part of the interface to Position, but if you can think of a
> good alternate header name, that's ok too.

I think a separate header is better for keeping more classes out of Position.h


-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.