Re: adding feature to indent

[david]

Jean-Christophe,
Please send the update again. The bug-indent filter seems to have
removed your attachment.
I would prefer it if you send the whole updated files rather than
patches as then I can see your updates in the context of the version of
the file you actually updated - which may be older than the current SVN
head. I hope you also send updates to the documentation, as without them
your update is worthless.

If I were reviewing your code I'd say that you should get rid of the
labels and goto's altogether :-)

Jean-Christophe Dubois wrote:
> Hi,
>
> I'd like to propose a patch to ease the "linux coding style" with indent.
>
> in the Linux source tree there is a script that shows the recommended Linux 
> coding style using indent in scripts/Lindent. The content is as follow:
>
> #!/bin/sh
> indent -npro -kr -i8 -ts8 -sob -l80 -ss -ncs "$@"
>
> So indent is almost there. However, anytime I post an "indented" file with 
> labels on a linux mailing list, I am told that any (goto) label should start 
> at column 1 always. I don't think there is any option in the actual indent to 
> force (goto) label to start at column 1.
>
> So this is what my attached patch is doing. It adds an option to specify the 
> label offset and also add a "linux" profile that sums up the above line with 
> the new "label offset" option.
>
> Note that if the label offset option is not specified on the command line 
> then 
> there should be no change in behavior compared to the "old" indent.
>
> I have not updated the documentation (man, ...) with the new option.
>
> Thanks.
>
> JC
>
>
>   
> ------------------------------------------------------------------------
>
> _______________________________________________
> bug-indent mailing list
> address@hidden
> http://lists.gnu.org/mailman/listinfo/bug-indent
>