HashMap::ensure() should return an AddResult like all the other add-like functitions
Created attachment 272211 [details] Patch
Attachment 272211 [details] did not pass style-queue: ERROR: Tools/TestWebKitAPI/Tests/WTF/HashMap.cpp:515: More than one command on the same line [whitespace/newline] [4] ERROR: Tools/TestWebKitAPI/Tests/WTF/HashMap.cpp:519: More than one command on the same line [whitespace/newline] [4] ERROR: Tools/TestWebKitAPI/Tests/WTF/HashMap.cpp:530: More than one command on the same line [whitespace/newline] [4] ERROR: Tools/TestWebKitAPI/Tests/WTF/HashMap.cpp:534: More than one command on the same line [whitespace/newline] [4] ERROR: Tools/TestWebKitAPI/Tests/WTF/HashMap.cpp:545: More than one command on the same line [whitespace/newline] [4] ERROR: Tools/TestWebKitAPI/Tests/WTF/HashMap.cpp:550: More than one command on the same line [whitespace/newline] [4] Total errors found: 6 in 4 files If any of these errors are false positives, please file a bug against check-webkit-style.
Committed r197123: <http://trac.webkit.org/changeset/197123>
Comment on attachment 272211 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=272211&action=review > Tools/TestWebKitAPI/Tests/WTF/HashMap.cpp:547 > + EXPECT_EQ(1u, *addResult.iterator->value.get()); No need for get(). > Tools/TestWebKitAPI/Tests/WTF/HashMap.cpp:552 > + EXPECT_EQ(1u, *addResult2.iterator->value.get()); No need for get().