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
On 1/8/2015 7:44 AM, Nick Østergaard wrote:
> Hi
>
> 2015-01-08 0:40 GMT+01:00 John Beard :
>> Hi,
>>
>> Is the policy for braced case indentation as follows?
>
> Policy is found in
> http://bazaar.launchpad.net/~kicad-product-committers/kicad/product/view/head:/Documentation/coding_style_policy.
Hi
2015-01-08 0:40 GMT+01:00 John Beard :
> Hi,
>
> Is the policy for braced case indentation as follows?
Policy is found in
http://bazaar.launchpad.net/~kicad-product-committers/kicad/product/view/head:/Documentation/coding_style_policy.pdf
> case SOMETHING:
> {
> int i = 0;
> break;
>
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
4 matches
Mail list logo