Bug 87079

Summary:

[V8] Replace 'throwError(); return Undefined();' with 'return throwError()'

Product:

WebKit

Reporter:

Kentaro Hara <haraken>

Component:

WebCore JavaScript

Assignee:

Kentaro Hara <haraken>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

abarth, japhet, webkit.review.bot

Priority:

Version:

528+ (Nightly build)

Hardware:

Unspecified

OS:

Unspecified

Bug Depends on:

Bug Blocks:

Attachments:

Description	Flags
Patch	none

Description Kentaro Hara 2012-05-21 22:33:07 PDT

We can replace

    throwError();
    return v8::Undefined();

with

    return throwError();

Comment 1 Kentaro Hara 2012-05-21 22:36:09 PDT

Created attachment 143189 [details]
Patch

Comment 2 Adam Barth 2012-05-21 23:50:04 PDT

Comment on attachment 143189 [details]
Patch

So much prettier.

Comment 3 WebKit Review Bot 2012-05-22 00:24:40 PDT

Comment on attachment 143189 [details]
Patch

Clearing flags on attachment: 143189

Committed r117923: <http://trac.webkit.org/changeset/117923>

Comment 4 WebKit Review Bot 2012-05-22 00:24:53 PDT

All reviewed patches have been landed.  Closing bug.