Opened 11 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 , 11 years ago
comment:2 by , 11 years ago
Milestone: | → Future |
---|
comment:3 by , 6 years ago
Resolution: | → fixed |
---|---|
Status: | new → closed |
Note:
See TracTickets
for help on using tickets.
Yes we should do it in a dedicated patch, otherwise it's hard to review patches because so many changes are just whitespace.