Bug 180395

Summary:

Math: don't redundantly check for exceptions, just release scope

Product:

WebKit

Reporter:

JF Bastien <jfbastien>

Component:

JavaScriptCore

Assignee:

JF Bastien <jfbastien>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

commit-queue, darin, fpizlo, jfbastien, keith_miller, mark.lam, msaboff, rmorisset, saam, webkit-bug-importer

Priority:

Keywords:

InRadar

Version:

WebKit Nightly Build

Hardware:

Unspecified

OS:

Unspecified

Bug Depends on:

180297

Bug Blocks:

Attachments:

Description	Flags
patch	none

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>