Bug 180395 - Math: don't redundantly check for exceptions, just release scope
Summary: Math: don't redundantly check for exceptions, just release scope
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: JavaScriptCore (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: JF Bastien
URL:
Keywords: InRadar
Depends on: 180297
Blocks:
  Show dependency treegraph
 
Reported: 2017-12-04 23:18 PST by JF Bastien
Modified: 2017-12-04 23:52 PST (History)
10 users (show)

See Also:


Attachments
patch (2.39 KB, patch)
2017-12-04 23:20 PST, JF Bastien
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description JF Bastien 2017-12-04 23:18:00 PST
In #180297 two of the exeptions checks could just have been exception scope releases before the return, which is ever-so-slightly more efficient. Make it so.
Comment 1 JF Bastien 2017-12-04 23:20:33 PST
Created attachment 328442 [details]
patch

Mark rubber-stamped this in-person earlier today.
Comment 2 WebKit Commit Bot 2017-12-04 23:51:36 PST
Comment on attachment 328442 [details]
patch

Clearing flags on attachment: 328442

Committed r225519: <https://trac.webkit.org/changeset/225519>
Comment 3 WebKit Commit Bot 2017-12-04 23:51:38 PST
All reviewed patches have been landed.  Closing bug.
Comment 4 Radar WebKit Bug Importer 2017-12-04 23:52:19 PST
<rdar://problem/35849413>