Bug 125075 – [JSC] Get rid of some unused parameters in LLIntSlowPaths.cpp macros

• Component: JavaScriptCore
• Status: RESOLVED FIXED

[Nick Diego Yamane (diegoyam)]

[JSC] Get rid of some unused parameters in LLIntSlowPaths.cpp macros

[Nick Diego Yamane (diegoyam)]

Created attachment 218164 [details]
Patch

[Michael Saboff]

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.

[Nick Diego Yamane (diegoyam)]

Created attachment 218177 [details]
Patch

Fixing '\' indentation as requested by Michael.

[Nick Diego Yamane (diegoyam)]

Created attachment 218180 [details]
Patch for landing

[WebKit Commit Bot]

Comment on attachment 218180 [details]
Patch for landing

Clearing flags on attachment: 218180

Committed r159940: <http://trac.webkit.org/changeset/159940>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.