Bug 173837 - [TestWebKitAPI] Fix false-positive bad release warnings found by clang static analyzer
Summary: [TestWebKitAPI] Fix false-positive bad release warnings found by clang static...
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: David Kilzer (:ddkilzer)
Depends on:
Reported: 2017-06-26 09:43 PDT by David Kilzer (:ddkilzer)
Modified: 2017-06-26 11:02 PDT (History)
7 users (show)

See Also:

Patch v1 (4.33 KB, patch)
2017-06-26 09:45 PDT, David Kilzer (:ddkilzer)
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description David Kilzer (:ddkilzer) 2017-06-26 09:43:33 PDT
The following tests use a retain/release pattern that the clang static analyzer can't grok, which leads to a false-positive "bad release" warning:


It's better to use a common idiom for storing the delegate object anyway.
Comment 1 David Kilzer (:ddkilzer) 2017-06-26 09:45:58 PDT
Created attachment 313846 [details]
Patch v1
Comment 2 Geoffrey Garen 2017-06-26 10:59:33 PDT
Comment on attachment 313846 [details]
Patch v1

Comment 3 WebKit Commit Bot 2017-06-26 11:02:42 PDT
Comment on attachment 313846 [details]
Patch v1

Clearing flags on attachment: 313846

Committed r218815: <http://trac.webkit.org/changeset/218815>
Comment 4 WebKit Commit Bot 2017-06-26 11:02:44 PDT
All reviewed patches have been landed.  Closing bug.