Opened 3 years ago

Last modified 3 years ago

#532 new defect

Trailing white spaces

Reported by: pcampalani Owned by: dmisev
Priority: minor Milestone: Future
Component: undecided Version: development
Keywords: trailing white spaces Cc:
Complexity: Medium

Description

Trailing white spaces should be cleared out from the code sources, and it would be better done in some dedicated patch.

http://codeimpossible.com/2012/04/02/Trailing-whitespace-is-evil-Don-t-commit-evil-into-your-repo-/

It is good practice to make sure no new trailing white-spaces are re-introduced when formatting a patch.

E.g.

  • shell: sed -i 's/[ \t]*$//'
  • vi: :%s/\s\+$//g

Change History (2)

comment:1 Changed 3 years ago by dmisev

Yes we should do it in a dedicated patch, otherwise it's hard to review patches because so many changes are just whitespace.

comment:2 Changed 3 years ago by pcampalani

  • Milestone set to Future
Note: See TracTickets for help on using tickets.