Skip to content

Fix CMake whitespace#2608

Merged
JoelMarcey merged 1 commit into
facebook:masterfrom
tstarling:fix-cmake-indent
May 14, 2014
Merged

Fix CMake whitespace#2608
JoelMarcey merged 1 commit into
facebook:masterfrom
tstarling:fix-cmake-indent

Conversation

@tstarling

Copy link
Copy Markdown
Contributor

Applied a two-space indent. It was a mixture of 2, 3, 4 and 8 space indents and tabs; I don't know how you could look at it without going crazy. I mostly used Vim's "=" command, even in cases where its output is potentially controversial, because at least it's consistent.

Also modified vertical whitespace in a couple of places. A whitespace-insensitive diff will show you where.

@tstarling

Copy link
Copy Markdown
Contributor Author

Rebased, and fixed DOS line endings in FindBISON.cmake, which were erroneously reported as trailing whitespace introduced in this commit.

@tstarling

Copy link
Copy Markdown
Contributor Author

Fixed another "whitespace error".

Applied a two-space indent. It was a mixture of 2, 3, 4 and 8 space
indents and tabs; I don't know how you could look at it without going
crazy. I mostly used Vim's "=" command, even in cases where its output
is potentially controversial, because at least it's consistent.

Also modified vertical whitespace in a couple of places. A
whitespace-insensitive diff will show you where.

Fixed DOS line endings in FindBISON.cmake, which were reported by git as
whitespace errors.
@JoelMarcey JoelMarcey added this to the Lockdown milestone May 12, 2014
@JoelMarcey JoelMarcey merged commit d931d10 into facebook:master May 14, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants