Bug 158172
| Summary: | Vet every C++ lambda that captures only [this] | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Brady Eidson <beidson> |
| Component: | WebCore Misc. | Assignee: | Nobody <webkit-unassigned> |
| Status: | NEW | ||
| Severity: | Normal | CC: | cdumez |
| Priority: | P2 | ||
| Version: | WebKit Nightly Build | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
| See Also: |
https://bugs.webkit.org/show_bug.cgi?id=158173 https://bugs.webkit.org/show_bug.cgi?id=158166 |
||
Brady Eidson
Vet every C++ lambda that captures only [this]
At least a few of them are definitely unsafe, in that they are deferred but don't guarantee the lifetime of `this`
I found these by grepping for (pretty much) every lambda usage in WebCore and then narrowing down to [this] captures:
grep -rn '\[.*\]\s*\((.*)\)\?\s*{\s*$' . | grep "\[this\]"
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |