[Webkit-unassigned] [Bug 124594] [WTF] Clean up the constructors in MediaTime

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Tue Nov 19 12:46:19 PST 2013


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





--- Comment #1 from Darin Adler <darin at apple.com>  2013-11-19 12:44:54 PST ---
Since we think the units of the arguments to the MediaTime constructor are unclear, I suggest we make the actual constructors private and have the public construction be through named functions that make the argument types clearer. It seems like we already mostly use that pattern; I suggest going the rest of the way. Step 1: Make constructors private and find out what fails. Step 2: Add named public static member functions that are easy to read at each of those call sites.

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