Add raw pointer overloads to ListHashSet via SmartPtr specialized functions
Created attachment 355545 [details] Patch
Attachment 355545 [details] did not pass style-queue: ERROR: Source/WTF/wtf/ListHashSet.h:639: This { should be at the end of the previous line [whitespace/braces] [4] ERROR: Source/WTF/wtf/ListHashSet.h:649: This { should be at the end of the previous line [whitespace/braces] [4] ERROR: Source/WTF/wtf/ListHashSet.h:659: This { should be at the end of the previous line [whitespace/braces] [4] ERROR: Source/WTF/wtf/ListHashSet.h:666: This { should be at the end of the previous line [whitespace/braces] [4] ERROR: Source/WTF/wtf/ListHashSet.h:673: This { should be at the end of the previous line [whitespace/braces] [4] ERROR: Source/WTF/wtf/ListHashSet.h:680: This { should be at the end of the previous line [whitespace/braces] [4] Total errors found: 6 in 5 files If any of these errors are false positives, please file a bug against check-webkit-style.
Created attachment 355551 [details] Patch
Attachment 355551 [details] did not pass style-queue: ERROR: Source/WTF/wtf/ListHashSet.h:639: This { should be at the end of the previous line [whitespace/braces] [4] ERROR: Source/WTF/wtf/ListHashSet.h:649: This { should be at the end of the previous line [whitespace/braces] [4] ERROR: Source/WTF/wtf/ListHashSet.h:659: This { should be at the end of the previous line [whitespace/braces] [4] ERROR: Source/WTF/wtf/ListHashSet.h:666: This { should be at the end of the previous line [whitespace/braces] [4] ERROR: Source/WTF/wtf/ListHashSet.h:673: This { should be at the end of the previous line [whitespace/braces] [4] ERROR: Source/WTF/wtf/ListHashSet.h:680: This { should be at the end of the previous line [whitespace/braces] [4] Total errors found: 6 in 5 files If any of these errors are false positives, please file a bug against check-webkit-style.
Comment on attachment 355551 [details] Patch omg, thank you. I've been wanting to do this for months.
(In reply to zalan from comment #5) > Comment on attachment 355551 [details] > Patch > > omg, thank you. I've been wanting to do this for months. Yeah, I saw the FIXME and decided to do it. I have a follow up which fixes the InlineItem list.
Comment on attachment 355551 [details] Patch Clearing flags on attachment: 355551 Committed r238467: <https://trac.webkit.org/changeset/238467>
All reviewed patches have been landed. Closing bug.
<rdar://problem/46226142>
Nice!
One of the API tests added with this change is failing on iOS: TestWTF.WTF_ListHashSet.UniquePtrKey_InsertBeforeUsingRawPointer /Volumes/Data/slave/ios-simulator-12-release/build/Tools/TestWebKitAPI/Tests/WTF/ListHashSet.cpp:453 Expected equality of these values: ptrWith2 Which is: 0x7faee4f03c90 firstWith4.get() Which is: 0x7faee4f03ca0 https://build.webkit.org/builders/Apple%20iOS%2012%20Simulator%20Release%20WK2%20%28Tests%29/builds/1130/steps/run-api-tests/logs/stdio
(In reply to Ryan Haddad from comment #11) > One of the API tests added with this change is failing on iOS: > > TestWTF.WTF_ListHashSet.UniquePtrKey_InsertBeforeUsingRawPointer > > > /Volumes/Data/slave/ios-simulator-12-release/build/Tools/TestWebKitAPI/Tests/ > WTF/ListHashSet.cpp:453 > Expected equality of these values: > ptrWith2 > Which is: 0x7faee4f03c90 > firstWith4.get() > Which is: 0x7faee4f03ca0 > > https://build.webkit.org/builders/ > Apple%20iOS%2012%20Simulator%20Release%20WK2%20%28Tests%29/builds/1130/steps/ > run-api-tests/logs/stdio Oh, I think this was fixed in https://trac.webkit.org/changeset/238486/webkit
(In reply to Ryan Haddad from comment #12) > (In reply to Ryan Haddad from comment #11) > > One of the API tests added with this change is failing on iOS: > > > > TestWTF.WTF_ListHashSet.UniquePtrKey_InsertBeforeUsingRawPointer > > > > > > /Volumes/Data/slave/ios-simulator-12-release/build/Tools/TestWebKitAPI/Tests/ > > WTF/ListHashSet.cpp:453 > > Expected equality of these values: > > ptrWith2 > > Which is: 0x7faee4f03c90 > > firstWith4.get() > > Which is: 0x7faee4f03ca0 > > > > https://build.webkit.org/builders/ > > Apple%20iOS%2012%20Simulator%20Release%20WK2%20%28Tests%29/builds/1130/steps/ > > run-api-tests/logs/stdio > Oh, I think this was fixed in https://trac.webkit.org/changeset/238486/webkit Yeah, I fixed it already.