[webkit-reviews] review granted: [Bug 121310] HashMap should work with move-only keys : [Attachment 211911] Patch
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Tue Sep 17 09:03:52 PDT 2013
Darin Adler <darin at apple.com> has granted Anders Carlsson
<andersca at apple.com>'s request for review:
Bug 121310: HashMap should work with move-only keys
https://bugs.webkit.org/show_bug.cgi?id=121310
Attachment 211911: Patch
https://bugs.webkit.org/attachment.cgi?id=211911&action=review
------- Additional Comments from Darin Adler <darin at apple.com>
View in context: https://bugs.webkit.org/attachment.cgi?id=211911&action=review
>> Source/WTF/wtf/HashMap.h:140
>> + template<typename K, typename V>
>> + AddResult inlineSet(K&&, V&&);
>
> Missing spaces around && [whitespace/operators] [3]
Might read better on one line instead of two.
>> Source/WTF/wtf/HashMap.h:143
>> + template<typename K, typename V>
>> + AddResult inlineAdd(K&&, V&&);
>
> Missing spaces around && [whitespace/operators] [3]
Might read better on one line instead of two.
> Source/WebCore/html/InputType.cpp:90
> -static PassOwnPtr<InputTypeFactoryMap> createInputTypeFactoryMap()
> +static OwnPtr<InputTypeFactoryMap> createInputTypeFactoryMap()
The pattern here should change so we aren’t calling HashMap::add repeatedly,
since those calls will be inlined. Need to do this in a loop instead. I guess I
can make that change after you land. I would suggest we make a static const
table that has two function pointers, one to the function that returns the type
name and the other being the create function.
More information about the webkit-reviews
mailing list