On Sun, Jul 20, 2014 at 9:56 AM, Sören Gebbert <soerengebb...@googlemail.com > wrote:
> 2014-07-20 6:06 GMT+02:00 Vaclav Petras <wenzesl...@gmail.com>: > > > > On Sat, Jul 19, 2014 at 8:02 PM, <svn_gr...@osgeo.org> wrote: > >> > >> Unfortunately changed the editor that i use the indention and converted > >> all tabs into spaces at save time. Hence, > >> the commit contains 95% tab to space conversion noise. > > > > > > I would actually agree with the change. I think that the GRASS > indentation > > style is a bug since mixed tabs and spaces are enforced as a rule (not > just > > allowed). However, it was decided that it won't be fixed and that current > > practice should be preserved. [1] > > Mixing tabs and spaces is really a mess. And i am part of the problem, > since i am using several different computer for development and > different editors, each with its own habits. > > > Fortunately, there is a script which can enforce the unusual indent style > > [2]. And also `svn diff --ignore-space-change` for cases like this > commit. > > Thank you for this hint, i will try to use it. > See also improved: http://trac.osgeo.org/grass/wiki/Submitting/C#Indentation And just as a reminder for others, for Python code, use pep8 directly for new code. For existing code use: http://trac.osgeo.org/grass/wiki/Submitting/Python#Style pep8 --config=tools/pep8config.txt directory_to_check
_______________________________________________ grass-dev mailing list grass-dev@lists.osgeo.org http://lists.osgeo.org/mailman/listinfo/grass-dev