bug-indent
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: Allow customising goto label indentation


From: Matthew Wilcox
Subject: Re: Allow customising goto label indentation
Date: Mon, 1 Oct 2007 10:12:03 -0600
User-agent: Mutt/1.5.13 (2006-08-11)

On Mon, Oct 01, 2007 at 06:02:51PM +0200, david wrote:
> To submit a update that supports a new feature two more things beside
> the code itself are needed:
> 
> 1) an update to indent.texinfo that describes the new feature at all
> relevant locations in the document.

The patch includes an update to indent.texinfo.  Did I miss some parts
of indent.texinfo that need updating to match?  I've snipped all but the
documentation parts from the patch below.

> 2) updates to the regression tests that demonstrate correct working of
> the feature:
> 
>         regression/TEST         - runs indent with the required options
> on your <name>.c sample code
>         regression/<name>.c   - new code sample pre application of intent
>         regression/standard/<name>.c - code sample that shows correctly
> applied indentation with your feature active.

OK, I'll look at the regression testsuite.  Thanks.

> --- indent-2.2.9.orig/doc/indent.texinfo
> +++ indent-2.2.9/doc/indent.texinfo
> @@ -1188,6 +1188,16 @@
>  columns wide, but (as of version 1.2) may be changed by the @option{-ts}
>  option.  Tabs are treated as the equivalent number of spaces.
>  
> address@hidden address@hidden
> address@hidden address@hidden
> +The indentation of goto labels is controlled by the @option{-li} parameter.  
> If
> +it is set to zero or a positive number, this indicates how far from the left
> +margin to indent a label.  If it is set to a negative number, this indicates
> +how far back from the current indent level to place the label.  The default
> +setting is -2 which matches the behaviour of earlier versions of indent.
> +Note that this parameter does not affect the placing of case labels; see the
> address@hidden parameter for that.
> +
>  @kindex address@hidden
>  @kindex address@hidden
>  @kindex -nip
> @@ -1656,6 +1666,11 @@
>  Set maximum line length for comment formatting to @address@hidden
>  @xref{Comments}.
>  
> address@hidden address@hidden
> address@hidden address@hidden
> +Set indentation for labels to @address@hidden
> address@hidden
> +
>  @item -lp
>  @itemx --continue-at-parentheses
>  Line up continued lines at address@hidden

-- 
Intel are signing my paycheques ... these opinions are still mine
"Bill, look, we understand that you're interested in selling us this
operating system, but compare it to ours.  We can't possibly take such
a retrograde step."




reply via email to

[Prev in Thread] Current Thread [Next in Thread]