Summary: | Update hashThreadState() to exclude __opaque_flags. | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Mark Lam <mark.lam> | ||||
Component: | JavaScriptCore | Assignee: | Mark Lam <mark.lam> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | benjamin, cdumez, cmarcelo, commit-queue, ews-watchlist, keith_miller, msaboff, saam, tzagallo, webkit-bug-importer | ||||
Priority: | P2 | Keywords: | InRadar | ||||
Version: | WebKit Nightly Build | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Bug Depends on: | 235156 | ||||||
Bug Blocks: | |||||||
Attachments: |
|
Description
Mark Lam
2022-01-11 12:53:52 PST
Created attachment 448875 [details]
proposed patch.
Comment on attachment 448875 [details] proposed patch. View in context: https://bugs.webkit.org/attachment.cgi?id=448875&action=review > Source/WTF/wtf/threads/Signals.cpp:175 > + // Exclude the __opaque_flags field which is reserved for OS use. Nit: Maybe say that __opaque_flags is at the end of the payload. (In reply to Keith Miller from comment #2) > Nit: Maybe say that __opaque_flags is at the end of the payload. Thanks for the review. I've added the comment. Landed in r287908: <http://trac.webkit.org/r287908>. Re-opened since this is blocked by bug 235156 Re-landed in r287961: <http://trac.webkit.org/r287961>. |