[Webkit-unassigned] [Bug 21839] SVGAnimatedTemplate needs HashMap.h included explicitly
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Thu Oct 23 14:09:29 PDT 2008
https://bugs.webkit.org/show_bug.cgi?id=21839
darin at apple.com changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #24616|review? |review+
Flag| |
------- Comment #2 from darin at apple.com 2008-10-23 14:09 PDT -------
(From update of attachment 24616)
Change Log needs to have your name, not just dglazkov, in the header.
Also, you should remove the "WARNING: NO TEST CASES ADDED OR CHANGED" if you
don't need any tests and add a link to the bug.
It saves the committer time if you do those things right.
The reason this isn't needed for JSC platforms is that PlatformString.h
includes <kjs/identifier.h> which ends up including HashMap.h. To avoid making
ourselves crazy, we might just want to add a HashMap.h include to
PlatformString.h to eliminate this difference.
r=me, as is
--
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.
More information about the webkit-unassigned
mailing list