Re: Allow customising goto label indentation

[david]

Great!
I added all issues so that others can see what is needed as many assume
that just a
code snippet is sufficient. And then you get postings like Todd's today
that rightly point out that some switches are not described, or that
feature xyz has ceased to work.


Matthew Wilcox wrote:
> 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
>>     
>
>