RESOLVED FIXED 156746
Clean up the ParkingLot uparking API a bit
https://bugs.webkit.org/show_bug.cgi?id=156746
Summary Clean up the ParkingLot uparking API a bit
Filip Pizlo
Reported 2016-04-19 08:56:18 PDT
Patch forthcoming.
Attachments
the patch (15.10 KB, patch)
2016-04-19 08:59 PDT, Filip Pizlo
ggaren: review+
the patch (30.73 KB, patch)
2016-04-19 17:40 PDT, Filip Pizlo
saam: review+
patch for landing (11.99 KB, patch)
2016-04-19 19:05 PDT, Filip Pizlo
no flags
Filip Pizlo
Comment 1 2016-04-19 08:59:27 PDT
Created attachment 276726 [details] the patch
WebKit Commit Bot
Comment 2 2016-04-19 09:01:27 PDT
Attachment 276726 [details] did not pass style-queue: ERROR: Source/WTF/wtf/Condition.h:183: Place brace on its own line for function definitions. [whitespace/braces] [4] ERROR: Source/WTF/wtf/Lock.cpp:97: Place brace on its own line for function definitions. [whitespace/braces] [4] ERROR: Source/WTF/benchmarks/LockSpeedTest.cpp:82: Extra space before ( in function call [whitespace/parens] [4] Total errors found: 3 in 7 files If any of these errors are false positives, please file a bug against check-webkit-style.
Geoffrey Garen
Comment 3 2016-04-19 11:13:17 PDT
Comment on attachment 276726 [details] the patch r=me
Filip Pizlo
Comment 4 2016-04-19 17:40:02 PDT
Created attachment 276784 [details] the patch
Filip Pizlo
Comment 5 2016-04-19 17:40:21 PDT
I put this up for re-review because I added a lot of stuff.
WebKit Commit Bot
Comment 6 2016-04-19 17:41:03 PDT
Attachment 276784 [details] did not pass style-queue: ERROR: Source/WTF/wtf/Condition.h:183: Place brace on its own line for function definitions. [whitespace/braces] [4] ERROR: Source/WTF/wtf/Lock.cpp:97: Place brace on its own line for function definitions. [whitespace/braces] [4] ERROR: Source/WTF/benchmarks/LockSpeedTest.cpp:93: Extra space before ( in function call [whitespace/parens] [4] ERROR: Source/WTF/benchmarks/LockSpeedTest.cpp:172: Place brace on its own line for function definitions. [whitespace/braces] [4] ERROR: Source/WTF/benchmarks/LockSpeedTest.cpp:445: Place brace on its own line for function definitions. [whitespace/braces] [4] Total errors found: 5 in 7 files If any of these errors are false positives, please file a bug against check-webkit-style.
Saam Barati
Comment 7 2016-04-19 19:03:03 PDT
Comment on attachment 276784 [details] the patch r=me for the API change. I think that the LockSpeedTest changes should go in a separate patch.
Filip Pizlo
Comment 8 2016-04-19 19:05:55 PDT
Created attachment 276789 [details] patch for landing
WebKit Commit Bot
Comment 9 2016-04-19 19:08:32 PDT
Attachment 276789 [details] did not pass style-queue: ERROR: Source/WTF/wtf/Condition.h:183: Place brace on its own line for function definitions. [whitespace/braces] [4] ERROR: Source/WTF/wtf/Lock.cpp:97: Place brace on its own line for function definitions. [whitespace/braces] [4] Total errors found: 2 in 6 files If any of these errors are false positives, please file a bug against check-webkit-style.
Filip Pizlo
Comment 10 2016-04-19 21:25:33 PDT
Note You need to log in before you can comment on or make changes to this bug.