<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Feb 16, 2015 at 3:54 PM, Martin Landa <span dir="ltr"><<a href="mailto:landa.martin@gmail.com" target="_blank">landa.martin@gmail.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">Hi, are you planing to backport it to relbr70?</blockquote><div><br></div><div>I guess I could do that right away in this case. Just out of habit, I'm not backporting things immediately after commit.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">BTW, I was always<br>
thinking about moving scripts which are duplicated in various branches<br>
to one common place. Martin<br></blockquote><div><br></div><div>This sounds good but I don't know where to put them, having them right there in the branch is quite handy and also not all of them are duplicated. More complicated tools such as g.html2man can differ.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<div class=""><div class="h5"><br>
2015-02-16 21:05 GMT+01:00  <<a href="mailto:svn_grass@osgeo.org">svn_grass@osgeo.org</a>>:<br>
> Author: wenzeslaus<br>
> Date: 2015-02-16 12:05:05 -0800 (Mon, 16 Feb 2015)<br>
> New Revision: 64658<br>
><br>
> Modified:<br>
>    grass/trunk/tools/grass_indent.sh<br>
> Log:<br>
> do not use tabs which are 8 spaces wide when indent level is 4 spaces<br>
><br>
> Indent script was creating source code with 4 spaces for indent level 1,<br>
> 1 tab for indent level 2, 1 tab and 4 spaces for indent level 3, 2 tabs<br>
> for indent level 4 etc. This is style which is hard to do manually<br>
> and cannot be easily guessed and supported by a text editor because it<br>
> mixes spaces and tabs<br>
> (<a href="http://lists.osgeo.org/pipermail/grass-dev/2014-September/070653.html" target="_blank">http://lists.osgeo.org/pipermail/grass-dev/2014-September/070653.html</a>).<br>
><br>
> The indent rules are still the same, each level must be in multiplies<br>
> of 4 spaces (columns) as specified in<br>
> <a href="http://lists.osgeo.org/pipermail/grass-dev/2014-August/070497.html" target="_blank">http://lists.osgeo.org/pipermail/grass-dev/2014-August/070497.html</a>.<br>
><br>
> No backwards compatibility is needed because just small portion of the<br>
> current code has no diff after applying the (original version of) indent<br>
> script. Now tested with v.external, i.segment and r.slope.aspect.<br>
><br>
> The -ut (--use-tabs) was in the list at<br>
> <a href="https://trac.osgeo.org/grass/wiki/Submitting/C" target="_blank">https://trac.osgeo.org/grass/wiki/Submitting/C</a>.<br>
> But it was not in the list at<br>
> <a href="http://grasswiki.osgeo.org/w/index.php?title=Development&oldid=21058#Explanation_of_C_indentation_rules" target="_blank">http://grasswiki.osgeo.org/w/index.php?title=Development&oldid=21058#Explanation_of_C_indentation_rules</a>.<br>
> And tabs are explicitly forbidden in instructions for Emacs users at<br>
> <a href="http://grasswiki.osgeo.org/w/index.php?title=GRASS_Programming_Howto&oldid=20573#Emacs" target="_blank">http://grasswiki.osgeo.org/w/index.php?title=GRASS_Programming_Howto&oldid=20573#Emacs</a>.<br>
><br>
> See also #1663 (Tabs in C source code) and grass-dev Tabs and spaces in C code<br>
> (<a href="http://lists.osgeo.org/pipermail/grass-dev/2014-August/070484.html" target="_blank">http://lists.osgeo.org/pipermail/grass-dev/2014-August/070484.html</a>).<br>
><br>
><br>
> Modified: grass/trunk/tools/grass_indent.sh<br>
> ===================================================================<br>
> --- grass/trunk/tools/grass_indent.sh   2015-02-16 19:48:39 UTC (rev 64657)<br>
> +++ grass/trunk/tools/grass_indent.sh   2015-02-16 20:05:05 UTC (rev 64658)<br>
> @@ -1,14 +1,28 @@<br>
>  #!/bin/sh<br>
><br>
> -# Indent source code according to GRASS SUBMITTING rules<br>
> +# Indent source code according to GRASS GIS submitting rules<br>
><br>
> +# Should be in sync with:<br>
> +#     <a href="https://trac.osgeo.org/grass/wiki/Submitting/C" target="_blank">https://trac.osgeo.org/grass/wiki/Submitting/C</a><br>
> +#     <a href="http://grasswiki.osgeo.org/wiki/Development#Explanation_of_C_indentation_rules" target="_blank">http://grasswiki.osgeo.org/wiki/Development#Explanation_of_C_indentation_rules</a><br>
> +<br>
> +# Dependencies:<br>
> +#     indent<br>
> +<br>
> +# Changes and their reasons:<br>
> +#    -ts8 -ut -> --no-tabs<br>
> +#        Do not use 8 space wide tabs when indent level is 4<br>
> +<br>
> +# TODO: replace short flags by long ones to improve readability<br>
> +<br>
> +<br>
>  if [ $# -lt 1 ] ; then<br>
>   echo "No files specified (give file name(s) as parameter)"<br>
>   exit 1<br>
>  else<br>
>   indent -npro -bad -bap -bbb -br -bli0 -bls -cli0 -ncs -fc1 -hnl -i4 \<br>
>        -nbbo -nbc -nbfda -nbfde -ncdb -ncdw -nce -nfca -npcs -nprs \<br>
> -      -npsl -nsc -nsob -saf -sai -saw -sbi0 -ss -ts8 -ut "$@"<br>
> +      -npsl -nsc -nsob -saf -sai -saw -sbi0 -ss --no-tabs "$@"<br>
><br>
>   # fix broken gettext macros:<br>
>   grep -l '\<_$' "$@" | \<br>
><br>
> _______________________________________________<br>
> grass-commit mailing list<br>
> <a href="mailto:grass-commit@lists.osgeo.org">grass-commit@lists.osgeo.org</a><br>
> <a href="http://lists.osgeo.org/mailman/listinfo/grass-commit" target="_blank">http://lists.osgeo.org/mailman/listinfo/grass-commit</a><br>
<br>
<br>
<br>
</div></div><span class=""><font color="#888888">--<br>
Martin Landa<br>
<a href="http://geo.fsv.cvut.cz/gwiki/Landa" target="_blank">http://geo.fsv.cvut.cz/gwiki/Landa</a><br>
<a href="http://gismentors.eu/mentors/landa" target="_blank">http://gismentors.eu/mentors/landa</a><br>
</font></span></blockquote></div><br></div></div>