Posted by nod_ on December 11, 2011 at 4:22pm
4 followers
Jump to:
| Project: | OpenLayers |
| Version: | 7.x-2.x-dev |
| Component: | Code |
| Category: | task |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | closed (fixed) |
Issue Summary
This module has history and it shows. Here is the issue for all whitespace, coding standard problems and punctuation in docs.
No change in API, naming or anything. Only annoying EOL whitespace and EOF new line and maybe a full stop missing in comments and the like. The best patches are the one where a git diff --ignore-space-at-eol wouldn't show anything.
Comments
#1
Just committed whitespace trim in 3.x. Diff is empty with -d option :) now we don't have to worry about editors trimming all whitespace on save.
here is the script i used :
#!/bin/shFILES=`find openlayers | grep -v .git | grep -v .idea`
for file in $FILES
do
sed -i 's/ *$//' $file
done
I leave as to be ported for now if zzolo wants this in 2.x
#2
*yay* Thanks & I would like to say goodbye to
--ignore-space-at-eolin 2.x too :)#3
Yeah, lets put this in 2.x.
I do have to admit, my editor, Coda is quick to leave whitespace. :(
#4
committed to 7.x-2.x too :)
#5
Awesome & thanks.
Makes patching & reviewing really more efficient :)
#6
Automatically closed -- issue fixed for 2 weeks with no activity.