<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Sun, Jul 20, 2014 at 9:56 AM, Sören Gebbert <span dir="ltr"><<a href="mailto:soerengebbert@googlemail.com" target="_blank">soerengebbert@googlemail.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div id=":16e" class="" style="overflow:hidden">2014-07-20 6:06 GMT+02:00 Vaclav Petras <<a href="mailto:wenzeslaus@gmail.com">wenzeslaus@gmail.com</a>>:<br>


<div class="">><br>
> On Sat, Jul 19, 2014 at 8:02 PM, <<a href="mailto:svn_grass@osgeo.org">svn_grass@osgeo.org</a>> wrote:<br>
>><br>
>> Unfortunately changed the editor that i use the indention and converted<br>
>> all tabs into spaces at save time. Hence,<br>
>> the commit contains 95% tab to space conversion noise.<br>
><br>
><br>
> I would actually agree with the change. I think that the GRASS indentation<br>
> style is a bug since mixed tabs and spaces are enforced as a rule (not just<br>
> allowed). However, it was decided that it won't be fixed and that current<br>
> practice should be preserved. [1]<br>
<br>
</div>Mixing tabs and spaces is really a mess. And i am part of the problem,<br>
since i am using several different computer for development and<br>
different editors, each with its own habits.<br>
<div class=""><br>
> Fortunately, there is a script which can enforce the unusual indent style<br>
> [2]. And also `svn diff --ignore-space-change` for cases like this commit.<br>
<br>
</div>Thank you for this hint, i will try to use it.</div></blockquote></div><br>See also improved:<br><br><a href="http://trac.osgeo.org/grass/wiki/Submitting/C#Indentation">http://trac.osgeo.org/grass/wiki/Submitting/C#Indentation</a><br>

<br></div><div class="gmail_extra">And just as a reminder for others, for Python code, use pep8 directly for new code. For existing code use:<br><br><a href="http://trac.osgeo.org/grass/wiki/Submitting/Python#Style">http://trac.osgeo.org/grass/wiki/Submitting/Python#Style</a><br>

<br>  pep8 --config=tools/pep8config.txt directory_to_check<br></div></div>