[FFmpeg-devel] [PATCH] Set frame rate on v4l2 devices
jose.goncalves at inov.pt
jose.goncalves
Mon Sep 13 12:53:33 CEST 2010
Quoting M?ns Rullg?rd <mans at mansr.com>:
> No, coding style is things like placement of braces, which I did not
> comment on.
>
>> I agree with most of them (not all). But... if you take a look to
>> the v4l2_set_parameters() function you will see that I followed the
>> coding style previously used, what seemed the correct thing to do.
>
> Old mistakes are no excuse for making new ones.
New patch attempt attached. Some comments though.
For me, initializing a variable in declaration or not, or placing a
comment or not, is coding style... but let's not argue about that.
I don't agree with two of your comments.
First, initializing a struct with {0} instead of using a memset(). I
always use memset() because gcc gives a warning of "missing
initializer" in the other way if you activate the '-Wextra' option.
Second, testing ioctl error against -1 instead of negative. As rule of
thumb I test for a function error as the negation of returning OK. Most
ioctls return 0 for OK, but some return a positive value, hence testing
for negative.
Regards,
Jos? Gon?alves
-------------- next part --------------
[Attachment stripped: Original attachment type: "text/x-patch", name: "ffmpeg-r25112-set_frame_rate.patch"]
More information about the ffmpeg-devel
mailing list