Opened 10 years ago

Closed 6 years ago

#532 closed defect (fixed)

Trailing white spaces

Reported by: Piero Campalani Owned by: Dimitar Misev
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 (3)

comment:1 by Dimitar Misev, 10 years ago

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 by Piero Campalani, 10 years ago

Milestone: Future

comment:3 by Dimitar Misev, 6 years ago

Resolution: fixed
Status: newclosed
Note: See TracTickets for help on using tickets.