Bug 125075

Summary: [JSC] Get rid of some unused parameters in LLIntSlowPaths.cpp macros
Product: WebKit Reporter: Nick Diego Yamane (diegoyam) <nick.diego>
Component: JavaScriptCoreAssignee: Nick Diego Yamane (diegoyam) <nick.diego>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue, fpizlo
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Patch
none
Patch for landing none

Description Nick Diego Yamane (diegoyam) 2013-12-02 05:45:04 PST
[JSC] Get rid of some unused parameters in LLIntSlowPaths.cpp macros
Comment 1 Nick Diego Yamane (diegoyam) 2013-12-02 05:52:00 PST
Created attachment 218164 [details]
Patch
Comment 2 Michael Saboff 2013-12-02 08:53:01 PST
Comment on attachment 218164 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=218164&action=review

> Source/JavaScriptCore/llint/LLIntSlowPaths.cpp:145
> +#define LLINT_CALL_THROW(exec, exceptionToThrow) do {               \

Please fix the whitespace so the '\' lines up with the subsequent lines.

> Source/JavaScriptCore/llint/LLIntSlowPaths.cpp:157
> +#define LLINT_CALL_RETURN(exec, callTarget) do {                    \

Same whitespace issue here.
Comment 3 Nick Diego Yamane (diegoyam) 2013-12-02 09:25:11 PST
Created attachment 218177 [details]
Patch

Fixing '\' indentation as requested by Michael.
Comment 4 Nick Diego Yamane (diegoyam) 2013-12-02 09:33:51 PST
Created attachment 218180 [details]
Patch for landing
Comment 5 WebKit Commit Bot 2013-12-02 10:20:59 PST
Comment on attachment 218180 [details]
Patch for landing

Clearing flags on attachment: 218180

Committed r159940: <http://trac.webkit.org/changeset/159940>
Comment 6 WebKit Commit Bot 2013-12-02 10:21:01 PST
All reviewed patches have been landed.  Closing bug.