Summary: | check-webkit-style: warn about WTF::BlockPtr use in JavaScriptCore until ARC is enabled | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | David Kilzer (:ddkilzer) <ddkilzer> | ||||||
Component: | Tools / Tests | Assignee: | David Kilzer (:ddkilzer) <ddkilzer> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | darin, ews-watchlist, glenn, hi, jbedard, joepeck, keith_miller, mark.lam, msaboff, saam, simon.fraser, tzagallo, webkit-bug-importer | ||||||
Priority: | P2 | Keywords: | InRadar | ||||||
Version: | WebKit Nightly Build | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Unspecified | ||||||||
See Also: |
https://bugs.webkit.org/show_bug.cgi?id=220956 https://bugs.webkit.org/show_bug.cgi?id=221107 |
||||||||
Bug Depends on: | 152799, 186961 | ||||||||
Bug Blocks: | 221117 | ||||||||
Attachments: |
|
Description
David Kilzer (:ddkilzer)
2021-01-28 15:11:09 PST
This issue was introduced when WTF was converted to ARC since WTF::BlockPtr existed at that time: Bug 186961: [Cocoa] Convert the small bit of Objective-C++ code in WTF to ARC <https://bugs.webkit.org/show_bug.cgi?id=186961> Bug 152799: Add a smart block pointer <https://bugs.webkit.org/show_bug.cgi?id=152799> (In reply to David Kilzer (:ddkilzer) from comment #2) > This issue was introduced when WTF was converted to ARC since WTF::BlockPtr > existed at that time: Or rather, it became possible for this issue to exist once this happened, but we only know of one bug (Bug 220956) that was actually caused by this. Created attachment 418702 [details]
Patch v1
(In reply to David Kilzer (:ddkilzer) from comment #4) > Created attachment 418702 [details] > Patch v1 Test failures are not related to this patch (missing results, or these *.xml files should be skipped since they're not tests. Created attachment 418729 [details]
Patch v2
(In reply to David Kilzer (:ddkilzer) from comment #6) > Created attachment 418729 [details] > Patch v2 Made the regex check for "BlockPtr<" a little more robust. Comment on attachment 418729 [details]
Patch v2
Nice! r=me
The webkitpy failure is unrelated to this change (I think this Python test is flaky): [41/1961] webkitcorepy.tests.task_pool_unittest.TaskPoolUnittest.test_invalid_shutdown failed: Traceback (most recent call last): File "/Volumes/Data/worker/WebKitPy-Tests-EWS/build/Tools/Scripts/libraries/webkitcorepy/webkitcorepy/tests/task_pool_unittest.py", line 136, in test_invalid_shutdown pool.do(wait, 2) AssertionError: Exception not raised (In reply to David Kilzer (:ddkilzer) from comment #9) > The webkitpy failure is unrelated to this change (I think this Python test > is flaky): > > [41/1961] > webkitcorepy.tests.task_pool_unittest.TaskPoolUnittest.test_invalid_shutdown > failed: > Traceback (most recent call last): > File > "/Volumes/Data/worker/WebKitPy-Tests-EWS/build/Tools/Scripts/libraries/ > webkitcorepy/webkitcorepy/tests/task_pool_unittest.py", line 136, in > test_invalid_shutdown > pool.do(wait, 2) > AssertionError: Exception not raised <https://ews-build.webkit.org/#/builders/5/builds/42925> Committed r272064: <https://trac.webkit.org/changeset/272064> All reviewed patches have been landed. Closing bug and clearing flags on attachment 418729 [details]. |