lavd/v4l2: simplify first_field()
authorGiorgio Vazzana <mywing81@gmail.com>
Fri, 5 Sep 2014 15:24:43 +0000 (17:24 +0200)
committerMichael Niedermayer <michaelni@gmx.at>
Fri, 5 Sep 2014 16:36:04 +0000 (18:36 +0200)
There is no need to pass fd as a second parameter. Additionally remove
unneeded curly braces.

Signed-off-by: Michael Niedermayer <michaelni@gmx.at>
libavdevice/v4l2.c

index a765e64..2fd4966 100644 (file)
@@ -225,18 +225,16 @@ static int device_init(AVFormatContext *ctx, int *width, int *height,
     return res;
 }
 
-static int first_field(const struct video_data *s, int fd)
+static int first_field(const struct video_data *s)
 {
     int res;
     v4l2_std_id std;
 
-    res = v4l2_ioctl(fd, VIDIOC_G_STD, &std);
-    if (res < 0) {
+    res = v4l2_ioctl(s->fd, VIDIOC_G_STD, &std);
+    if (res < 0)
         return 0;
-    }
-    if (std & V4L2_STD_NTSC) {
+    if (std & V4L2_STD_NTSC)
         return 0;
-    }
 
     return 1;
 }
@@ -935,7 +933,7 @@ static int v4l2_read_header(AVFormatContext *ctx)
         (res = mmap_start(ctx)) < 0)
             goto fail;
 
-    s->top_field_first = first_field(s, s->fd);
+    s->top_field_first = first_field(s);
 
     st->codec->codec_type = AVMEDIA_TYPE_VIDEO;
     st->codec->codec_id = codec_id;