<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Sat, Jul 19, 2014 at 8:02 PM,  <span dir="ltr"><<a href="mailto:svn_grass@osgeo.org" target="_blank">svn_grass@osgeo.org</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=":1f6" class="" style="overflow:hidden">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.</div></blockquote></div><br></div><div class="gmail_extra">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]<br>

<br></div><div class="gmail_extra">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.<br></div><div class="gmail_extra"><br>

</div><div class="gmail_extra">Anyway, I'm glad you discovered and fixed the bug. It is now painful for me to see that tests did not showed the bug. That's maybe why in an ideal world, tests are written by different person than the code (and without access/looking to the source code).<br>

</div><div class="gmail_extra"><br></div><div class="gmail_extra">Vaclav<br></div><div class="gmail_extra"><br>[1] <a href="http://trac.osgeo.org/grass/ticket/1663">http://trac.osgeo.org/grass/ticket/1663</a><br>[2] <a href="http://trac.osgeo.org/grass/browser/grass/trunk/tools/grass_indent.sh">http://trac.osgeo.org/grass/browser/grass/trunk/tools/grass_indent.sh</a><br>

<br><br></div></div>