Script to sanitize source files

The script does the following:
* Adds a newline to the end of files if it does not have one.
* Sets svn:eol-style to LF.
* Replaces CRLF with LF.
* Replaces Tabs with 4 whitespaces.
* Removes trailing whitespaces.

Created to fix http://code.google.com/p/skia/issues/detail?id=779 : New buildbot step that fixes and submits whitespace and newline source file fixes.
Committed: https://code.google.com/p/skia/source/detail?r=5234

[bsalomon, 2012-08-21 15:02:00]

https://codereview.appspot.com/6465078/diff/2001/tools/sanitize_source_files.py
File tools/sanitize_source_files.py (right):

https://codereview.appspot.com/6465078/diff/2001/tools/sanitize_source_files....
tools/sanitize_source_files.py:129: if output == 'CRLF':
I think this should be if output != 'LF'. The property can be unspecified not
just CRLF or LF. I think we want LF on all source files.

[rmistry, 2012-08-21 16:47:19]

https://codereview.appspot.com/6465078/diff/2001/tools/sanitize_source_files.py
File tools/sanitize_source_files.py (right):

https://codereview.appspot.com/6465078/diff/2001/tools/sanitize_source_files....
tools/sanitize_source_files.py:129: if output == 'CRLF':
On 2012/08/21 15:02:01, bsalomon wrote:
> I think this should be if output != 'LF'. The property can be unspecified not
> just CRLF or LF. I think we want LF on all source files.

Done.

[epoger, 2012-08-22 17:55:55]

LGTM at patchset 4

https://codereview.appspot.com/6465078/diff/8001/tools/sanitize_source_files.py
File tools/sanitize_source_files.py (right):

https://codereview.appspot.com/6465078/diff/8001/tools/sanitize_source_files....
tools/sanitize_source_files.py:25: file_modifiers: list - Modifiers which are
applied to the complete file
maybe "Modifiers which are applied" -> "line-modification methods which should
be applied"

https://codereview.appspot.com/6465078/diff/8001/tools/sanitize_source_files....
tools/sanitize_source_files.py:27: line_modifiers: list - Modifiers which are
applied to lines in a file (Eg:
maybe "Modifiers which are applied" -> "file-modification methods which should
be applied"

https://codereview.appspot.com/6465078/diff/8001/tools/sanitize_source_files....
tools/sanitize_source_files.py:37: if (len(full_item_path.split('.')) > 1 and
I don't know if it matters, but this would fail to recognize   bert.ernie.cpp 
(for example)

[rmistry, 2012-08-22 18:07:08]

https://codereview.appspot.com/6465078/diff/8001/tools/sanitize_source_files.py
File tools/sanitize_source_files.py (right):

https://codereview.appspot.com/6465078/diff/8001/tools/sanitize_source_files....
tools/sanitize_source_files.py:25: file_modifiers: list - Modifiers which are
applied to the complete file
On 2012/08/22 17:55:55, epoger wrote:
> maybe "Modifiers which are applied" -> "line-modification methods which should
> be applied"

Done.

https://codereview.appspot.com/6465078/diff/8001/tools/sanitize_source_files....
tools/sanitize_source_files.py:27: line_modifiers: list - Modifiers which are
applied to lines in a file (Eg:
On 2012/08/22 17:55:55, epoger wrote:
> maybe "Modifiers which are applied" -> "file-modification methods which should
> be applied"

Done.

https://codereview.appspot.com/6465078/diff/8001/tools/sanitize_source_files....
tools/sanitize_source_files.py:37: if (len(full_item_path.split('.')) > 1 and
On 2012/08/22 17:55:55, epoger wrote:
> I don't know if it matters, but this would fail to recognize   bert.ernie.cpp 
> (for example)

Fixed this with full_item_path_split('.')[-1]

[rmistry, 2012-08-22 18:10:23]

Submitting..

[status.rpni, 2019-08-07 19:36:04]

Selesai

https://codereview.appspot.com/6465078/diff/2001/tools/sanitize_source_files.py
File tools/sanitize_source_files.py (right):

https://codereview.appspot.com/6465078/diff/2001/tools/sanitize_source_files....
tools/sanitize_source_files.py:129: if output == 'CRLF':
On 2012/08/21 16:47:20, rmistry wrote:
> <font style="vertical-align: inherit;"><font style="vertical-align:
inherit;">Pada 2012/08/21 15:02:01, bsalomon menulis:> Saya pikir ini seharusnya
jika output! = 'LF'. </font><font style="vertical-align: inherit;">Properti
dapat tidak ditentukan bukan hanya CRLF atau LF. </font><font
style="vertical-align: inherit;">Saya pikir kami ingin LF pada semua file
sumber.
> 
> </font></font>
> <font style="vertical-align: inherit;"><font style="vertical-align:
inherit;">Selesai
> </font></font>

Acknowledged.