Skip to content

Strip all trailing whitespace in files #250

@mithro

Description

@mithro

A large number of the files has trailing whitespace on the end of the lines. It would be nice if these didn't exist :-P -- You can see which files have trailing white space by using ack --type=cpp -R ' +$' -l | sort > trailing.txt.

This can be fixed in an automated fashion with the following command line;

sed -i -e's/ \+$//' $(find -name *.cpp) $(find -name *.h) $(find -name *.hpp)

Which will produce a patch like this -> mithro@71b305e -- it is probably better to have one of the primary contributors do this change however.

If you do this, you should create a .git-blame-ignore-revs file with the hash of the whitespace fix CL. You can see an example here -> https://github.com/Samsung/ChromiumGStreamerBackend/blob/master/.git-blame-ignore-revs

If you want to do more aggressive reformatting, I'm happy to show you how with clang-format.

Metadata

Metadata

Assignees

No one assigned

    Labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions