800611 - Remove trailing whitespace from signaling code

Status: RESOLVED FIXED

(Core :: WebRTC: Signaling, defect)

Description

[Ethan Hugg [:ehugg]]

Trailing whitespace gets called by splinter reviews but the signaling code already is has quite a bit.  Remove all the trailing whitespace so reviews will be easier and we can all set our editors to delete trailing whitespace on save.

Comment 1

[Ethan Hugg [:ehugg]]

Attachment: trailing whitespace removed from signaling code

Comment 2

[Ethan Hugg [:ehugg]]

Comment on attachment 670581 [details] [diff] [review]
trailing whitespace removed from signaling code


I used this command: 

sed -i -e 's/[\t ]*$//g' `find . -name '*.h' -or -name '*.c' -or -name '*.cpp' -or -name '*.cc'`

Verified that it's whitespace-only by doing:
cp -r signaling ~/tmp/signaling.org
hg qpush
diff -ur signaling ~/tmp/signaling.org
> lots of output
diff -wur signaling ~/tmp/signaling.org
> no output

Comment 3

[Randell Jesup [:jesup] (needinfo me)]

WHen I tried to apply this to inbound, I got a lot of errors.  Can you rebase the patch please?

Comment 4

[Ethan Hugg [:ehugg]]

Attachment: Trailing whitepace removed from signaling code

Comment 5

[Ethan Hugg [:ehugg]]

Comment on attachment 670805 [details] [diff] [review]
Trailing whitepace removed from signaling code


Re-based on today's M-I rather than older M-C.

Comment 6

[Randell Jesup [:jesup] (needinfo me)]

https://hg.mozilla.org/integration/mozilla-inbound/rev/c7e467ada390

Comment 7

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/c7e467ada390