#1137: [PATCH] fix codingstd/c_indent.t wrt switch/case labels
------------------------------+---------------------------------------------
Reporter: plobsing | Owner: jkeenan
Type: patch | Status: assigned
Priority: minor | Milestone:
Component: coding_standards | Version: 1.6.0
Severity: low | Keywords:
Lang: | Patch:
Platform: |
------------------------------+---------------------------------------------
Comment(by coke):
Replying to [comment:3 plobsing]:
> Replying to [comment:2 coke]:
> > Looks like making
> >
> >
> {{{
> if ($line =~ /:$/) {
> }}}
> >
> > slightly smarter would be sufficient; say, optional leading whitespace
> > followed by valid label characters ending with a colon - that should
> > be enough to stop it from catching 'case FOO:'.
>
> This is by design. The spec says all labels and explicitly includes case
labels:
> > Labels ('''including case labels''') must be outdented two columns
relative to the code they label.
Ah, so it does. Ok. Looking through all the reported failures, here's a
case that appears to be valid but fails your check.
case FLOATTYPE_16: /* native */
If you can allow for the trailing comment, we apply your changes and
reformat the labels/switch statements.
Thanks!
--
Ticket URL: <https://trac.parrot.org/parrot/ticket/1137#comment:5>
Parrot <https://trac.parrot.org/parrot/>
Parrot Development
_______________________________________________
parrot-tickets mailing list
[email protected]
http://lists.parrot.org/mailman/listinfo/parrot-tickets
