Clang format changes

Format changes  achieved by running
clang-format -i -style=Chromium

BUG=

Committed: https://crrev.com/ff63ed2888fcc6747353a777d885b2001369bab6
Cr-Commit-Position: refs/heads/master@{#11427}

[peah-webrtc, 2016-01-27 13:43:23]

Description was changed from

==========
Format changes  achieved by running
clang-format -i -style=Chromium

BUG=
==========

to

==========
Format changes  achieved by running
clang-format -i -style=Chromium

BUG=
==========

[peah-webrtc, 2016-01-27 13:46:09]

peah@webrtc.org changed reviewers:
+ kwiberg@webrtc.org

[kwiberg-webrtc, 2016-01-27 14:08:21]

lgtm (but see comment)

None of these files had completely impossible indentation. I thought you said at
least one file had 4-space indentation, for example?

https://codereview.webrtc.org/1639283002/diff/1/webrtc/modules/audio_processi...
File webrtc/modules/audio_processing/aec/aec_core_mips.c (right):

https://codereview.webrtc.org/1639283002/diff/1/webrtc/modules/audio_processi...
webrtc/modules/audio_processing/aec/aec_core_mips.c:342: : "memory");
Ouch. May I ask for a follow-up CL that makes sure each asm line is only one
line again here? I think if you just reduce the amount of space here and there
you can make it fit in a way clang-format approves of...

[peah-webrtc, 2016-01-28 10:04:42]

On 2016/01/27 14:08:21, kwiberg-webrtc wrote:
> lgtm (but see comment)
> 
> None of these files had completely impossible indentation. I thought you said
at
> least one file had 4-space indentation, for example?
> 
>
https://codereview.webrtc.org/1639283002/diff/1/webrtc/modules/audio_processi...
> File webrtc/modules/audio_processing/aec/aec_core_mips.c (right):
> 
>
https://codereview.webrtc.org/1639283002/diff/1/webrtc/modules/audio_processi...
> webrtc/modules/audio_processing/aec/aec_core_mips.c:342: : "memory");
> Ouch. May I ask for a follow-up CL that makes sure each asm line is only one
> line again here? I think if you just reduce the amount of space here and there
> you can make it fit in a way clang-format approves of...

Hi, 
Totally agree. Ok if I immediately revert the referred changes in
aec_core_mips.c?

I had forgotten about the 4 space indentation comment, and checking now it
actually was in aecm_core.c. I thought it also was in the helper files for
aec_core.c but I was wrong about that.

[kwiberg-webrtc, 2016-01-28 10:43:02]

On 2016/01/28 10:04:42, peah-webrtc wrote:

> Totally agree. Ok if I immediately revert the referred changes in
> aec_core_mips.c?

Well, it's probably better to reduce the amount of whitespace in the strings so
that clang-format will not break them up this way. And that's a manual edit, so
I don't want that in this CL. I won't object if you want to do what you
suggested instead, though.

> I had forgotten about the 4 space indentation comment, and checking now it
> actually was in aecm_core.c. I thought it also was in the helper files for
> aec_core.c but I was wrong about that.

OK.

[peah-webrtc, 2016-01-28 11:43:31]

The CQ bit was checked by peah@webrtc.org

[peah-webrtc, 2016-01-28 11:43:31]

The patchset sent to the CQ was uploaded after l-g-t-m from kwiberg@webrtc.org
Link to the patchset: https://codereview.webrtc.org/1639283002/#ps20001 (title:
"Reverted the reformatting of the MIPS assembly code")

[commit-bot: I haz the power, 2016-01-28 11:43:36]

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

[commit-bot: I haz the power, 2016-01-28 11:45:42]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-01-28 11:45:43]

Try jobs failed on following builders:
  ios_arm64_dbg on tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/ios_arm64_dbg/builds/7137)
  ios_dbg on tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/ios_dbg/builds/12309)
  ios_rel on tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/ios_rel/builds/10967)

[peah-webrtc, 2016-01-29 14:41:06]

The CQ bit was checked by peah@webrtc.org

[peah-webrtc, 2016-01-29 14:41:06]

The patchset sent to the CQ was uploaded after l-g-t-m from kwiberg@webrtc.org
Link to the patchset: https://codereview.webrtc.org/1639283002/#ps40001 (title:
"Merged with latest code from master")

[commit-bot: I haz the power, 2016-01-29 14:41:09]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1639283002/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1639283002/40001

[commit-bot: I haz the power, 2016-01-29 15:46:15]

Description was changed from

==========
Format changes  achieved by running
clang-format -i -style=Chromium

BUG=
==========

to

==========
Format changes  achieved by running
clang-format -i -style=Chromium

BUG=
==========

[commit-bot: I haz the power, 2016-01-29 15:46:16]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-01-29 15:46:23]

Description was changed from

==========
Format changes  achieved by running
clang-format -i -style=Chromium

BUG=
==========

to

==========
Format changes  achieved by running
clang-format -i -style=Chromium

BUG=

Committed: https://crrev.com/ff63ed2888fcc6747353a777d885b2001369bab6
Cr-Commit-Position: refs/heads/master@{#11427}
==========

[commit-bot: I haz the power, 2016-01-29 15:46:24]

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