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@lists.launchpad.net
> Unsubscribe : https://launchpad.net/~kicad-developers
> More help : https://help.launchpad.net/ListHelp
>
_______________________________________________
Mailing list: https://launchpad.net/~kicad-developers
Post to : kicad-developers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~kicad-developers
More help : https://help.launchpad.net/ListHelp
