Bug 169424 - Use const AbstractLocker& (instead of const LockHolder&) in more places.
Summary: Use const AbstractLocker& (instead of const LockHolder&) in more places.
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: JavaScriptCore (show other bugs)
Version: WebKit Local Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Mark Lam
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-03-09 12:28 PST by Mark Lam
Modified: 2017-03-09 13:07 PST (History)
8 users (show)

See Also:


Attachments
proposed patch. (17.23 KB, patch)
2017-03-09 12:29 PST, Mark Lam
fpizlo: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Mark Lam 2017-03-09 12:28:03 PST
This is the modern idiom, and it makes some of these functions more flexible to use.
Comment 1 Mark Lam 2017-03-09 12:29:34 PST
Created attachment 303950 [details]
proposed patch.
Comment 2 Filip Pizlo 2017-03-09 12:59:47 PST
Comment on attachment 303950 [details]
proposed patch.

Nice!
Comment 3 Mark Lam 2017-03-09 13:07:15 PST
Thanks for the review.  Landed in r213657: <http://trac.webkit.org/r213657>.