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 majord...@vger.kernel.org
> 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 majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
