[wasm] [codestyle] Fix pointer alignment in wasm-debug.cc

[wasm] [codestyle] Fix pointer alignment in wasm-debug.cc

Our .clang-format is derived on the Google style, which sets
PointerAlignment to left (e.g. "Type* name"), but sets
DerivePointerAlignment to true. Once we started with the wrong style,
this made all new code in wasm-debug.cc use PointerAlignment=right,
resulting in lots of code using the wrong style.

For this CL, I ran
clang-format -style="{DerivePointerAlignment: false, BasedOnStyle: \
  Google}" -i src/wasm/wasm-debug.cc

R=titzer@chromium.org

Review-Url: https://codereview.chromium.org/2635003002
Cr-Commit-Position: refs/heads/master@{#42380}
Committed: https://chromium.googlesource.com/v8/v8/+/2702d0fbe31f58bebb26d1946295632648a85f7e

[Clemens Hammacher, 2017-01-16 15:19:25]

The CQ bit was checked by clemensh@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-01-16 15:19:34]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Clemens Hammacher, 2017-01-16 15:20:59]

Maybe we should explicitly set DerivePointerAlignment to false in our
.clang-format. WDYT?
+jochen

[titzer, 2017-01-16 15:25:04]

lgtm

[Clemens Hammacher, 2017-01-16 15:30:25]

On 2017/01/16 at 15:20:59, Clemens Hammacher wrote:
> Maybe we should explicitly set DerivePointerAlignment to false in our
.clang-format. WDYT?
> +jochen

Landing this one for now, and set DerivePointerAlignment to false in another CL:
http://crrev.com/2631183002

Will CC v8-team about that other CL once jochen gives +1 (seems to be most
responsible for .clang-format).

[Clemens Hammacher, 2017-01-16 15:30:58]

The CQ bit was checked by clemensh@chromium.org

[commit-bot: I haz the power, 2017-01-16 15:31:04]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-01-16 15:44:06]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1484580658267310, "parent_rev":
"4f556e9e4b0679a862822f48e05511200194c632", "commit_rev":
"2702d0fbe31f58bebb26d1946295632648a85f7e"}

[commit-bot: I haz the power, 2017-01-16 15:44:11]

Description was changed from

==========
[wasm] [codestyle] Fix pointer alignment in wasm-debug.cc

Our .clang-format is derived on the Google style, which sets
PointerAlignment to left (e.g. "Type* name"), but sets
DerivePointerAlignment to true. Once we started with the wrong style,
this made all new code in wasm-debug.cc use PointerAlignment=right,
resulting in lots of code using the wrong style.

For this CL, I ran
clang-format -style="{DerivePointerAlignment: false, BasedOnStyle: \
  Google}" -i src/wasm/wasm-debug.cc

R=titzer@chromium.org
==========

to

==========
[wasm] [codestyle] Fix pointer alignment in wasm-debug.cc

Our .clang-format is derived on the Google style, which sets
PointerAlignment to left (e.g. "Type* name"), but sets
DerivePointerAlignment to true. Once we started with the wrong style,
this made all new code in wasm-debug.cc use PointerAlignment=right,
resulting in lots of code using the wrong style.

For this CL, I ran
clang-format -style="{DerivePointerAlignment: false, BasedOnStyle: \
  Google}" -i src/wasm/wasm-debug.cc

R=titzer@chromium.org

Review-Url: https://codereview.chromium.org/2635003002
Cr-Commit-Position: refs/heads/master@{#42380}
Committed:
https://chromium.googlesource.com/v8/v8/+/2702d0fbe31f58bebb26d1946295632648a...
==========

[commit-bot: I haz the power, 2017-01-16 15:44:11]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/v8/v8/+/2702d0fbe31f58bebb26d1946295632648a...