gn format: Add 'outputs' to preferred-multiline lists

gn format: Add 'outputs' to preferred-multiline lists

For symmetry with 'inputs'.

R=brettw@chromium.org,tfarina@chromium.org
BUG=348474
Committed: https://crrev.com/fb0c4fd675c191bcc214088881d66170a2e828c0
Cr-Commit-Position: refs/heads/master@{#306668}

[brettw, 2014-12-03 19:21:00]

lgtm

[scottmg, 2014-12-03 20:41:56]

The CQ bit was checked by scottmg@chromium.org

[commit-bot: I haz the power, 2014-12-03 20:42:29]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/780543003/20001

[commit-bot: I haz the power, 2014-12-03 20:44:06]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2014-12-03 20:44:45]

Patchset 2 (id:??) landed as
https://crrev.com/fb0c4fd675c191bcc214088881d66170a2e828c0
Cr-Commit-Position: refs/heads/master@{#306668}

[tfarina, 2014-12-03 21:19:12]

thanks for considering it and changing to match "inputs".

lgtm

https://codereview.chromium.org/780543003/diff/20001/tools/gn/command_format.cc
File tools/gn/command_format.cc (right):

https://codereview.chromium.org/780543003/diff/20001/tools/gn/command_format....
tools/gn/command_format.cc:292: ident->value().value() == "deps" ||
ident->value().value() == "inputs" ||
would you mind moving the "inputs" part into its own line? I think that will
read better.

[scottmg, 2014-12-03 22:16:37]

On 2014/12/03 21:19:12, tfarina wrote:
> thanks for considering it and changing to match "inputs".
> 
> lgtm
> 
>
https://codereview.chromium.org/780543003/diff/20001/tools/gn/command_format.cc
> File tools/gn/command_format.cc (right):
> 
>
https://codereview.chromium.org/780543003/diff/20001/tools/gn/command_format....
> tools/gn/command_format.cc:292: ident->value().value() == "deps" ||
> ident->value().value() == "inputs" ||
> would you mind moving the "inputs" part into its own line? I think that will
> read better.

Yes, I agree. I wouldn't format it like that either, but clang-format did (sort
of unlucky because "deps" happens to be shorter) so I prefer to just live with
what it does. However, using a local temporary is probably better here anyway:
https://codereview.chromium.org/778833002