General Code Cleanup
Closed, ResolvedPublic
Actions

Assigned To

None

Authored By

	• goch
	Nov 6 2012, 1:06 PM

Description

As discussed in todays MITK meeting, there are several things which could be done to clean up MITK code in general:

Remove all trailing whitespaces from all code files
Replace all leading tabs by two spaces
Remove all personal and temporary files which were commited by accident and add them to the .gitignore

This should be done in preparation to adding a correponding git hook to the server to make sure no new problems will be comitted.

Mentioned In: T27033: Clean up stale remote branches
T13564: Relax DicomSeriesReader, verify its results more (support basic NM,CR,DX,SC files)

New remote branch pushed: bug-13553-code-cleanup

New remote branch pushed: bug-13553-add-files-to-gitignore

[88de65]: Merge branch 'bug-13553-add-files-to-gitignore'

2012-11-06 17:36:03 Caspar Goch [cb3bff]
Added more files to ignore

2012-11-06 16:53:05 Caspar Goch [7c5495]
Removed some temporary files, also some Mac and Windows housekeeping files and added them to gitignore

Used for whitespace cleanup:

replace tabs by two spaces
cat AllSourceFiles.txt | xargs sed -i 's/<TAB>/ /g'
where <TAB> = CTRL+V tab

remove trailing whitespaces:
cat AllSourceFiles.txt | xargs sed -i 's/:space:*$//'

New remote branch pushed: bug-13553-further-additions-to-gitignore

[126cad]: Merge branch 'bug-13553-further-additions-to-gitignore'

2012-11-08 11:49:55 Caspar Goch [d4da59]
Added patch files to gitignore

New remote branch pushed: bug-13553-whitespace-cleanup

[37739b]: Merge branch 'bug-13553-whitespace-cleanup'

2012-11-09 17:43:02 Caspar Goch [4a7524]
Removed trailing whitespaces

2012-11-09 17:42:03 Caspar Goch [4ad396]
Replaced tabs by 2 spaces

New remote branch pushed: bug-13553-remove-ws-introduced-before-hook

[2a4ca7]: Merge branch 'bug-13553-remove-ws-introduced-before-hook'

2012-11-15 12:52:44 Caspar Goch [071180]
Removed trailing whitespaces introduced before hook took effect