[webkit-reviews] review granted: [Bug 232265] Refactoring and PutByVal cleanup : [Attachment 443410] Patch

[bugzilla-daemon at webkit.org]

Saam Barati <sbarati at apple.com> has granted Mikhail R. Gadelha
<mikhail at igalia.com>'s request for review:
Bug 232265: Refactoring and PutByVal cleanup
https://bugs.webkit.org/show_bug.cgi?id=232265

Attachment 443410: Patch

https://bugs.webkit.org/attachment.cgi?id=443410&action=review




--- Comment #9 from Saam Barati <sbarati at apple.com> ---
Comment on attachment 443410
  --> https://bugs.webkit.org/attachment.cgi?id=443410
Patch

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

Nice. r=me

> Source/JavaScriptCore/ChangeLog:22
> +	   6. Removed a lot of whitespace.

We don't typically do this. Although the style says not to leave trailing
whitespace, we usually don't go back and remove trailing whitespace. I think
the main motivation is to make it easier to `blame` code.