Em 07-07-2010 11:14, Karicheri, Muralidharan escreveu:
>
>
>> v4l2_device *v4l2_dev,
>> if (err && err != -ENOIOCTLCMD) {
>> v4l2_device_unregister_subdev(sd);
>> sd = NULL;
>> + } else {
>> + sd->initialized = 1;
>> }
>
> Wouldn't checkpatch.pl script complain about { } on the else part since
> there is only one statement?
IMO, it is because it analyzes the entire if clause. As the first part of the if has two
statements, CodingStyle accepts the usage of braces at the second part
(although this is not a common practice).
>> }
>>
>
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-media" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
