← Back to team overview

kicad-developers team mailing list archive

Re: Uncrustify case indentation rules

 

John,

Good catch.  I guess that explains why I've seen a few places of this in
the code that I've had to fix.  I committed your patch in the product
branch r5360.  Keep up the good work.

Thanks,

Wayne

On 1/7/2015 6:40 PM, John Beard wrote:
> Hi,
> 
> Is the policy for braced case indentation as follows?
> 
> case SOMETHING:
> {
>     int i = 0;
>     break;
> }
> 
> This seems to be the case for tidying-up commits like r5288, but the
> uncrustify.cfg rules call for:
> 
> case SOMETHING:
>     {
>         int i = 0;
>         break;
>     }
> 
> If this is in fact incorrect, please find attached a patch for
> uncrustify.cfg.
> 
> Thanks,
> 
> John
> 
> 
> 
> _______________________________________________
> Mailing list: https://launchpad.net/~kicad-developers
> Post to     : kicad-developers@xxxxxxxxxxxxxxxxxxx
> Unsubscribe : https://launchpad.net/~kicad-developers
> More help   : https://help.launchpad.net/ListHelp
> 



References