diff options
| author | Kieran Bingham <kieran.bingham@ideasonboard.com> | 2018-05-23 10:43:28 +0100 |
|---|---|---|
| committer | Laurent Pinchart <laurent.pinchart@ideasonboard.com> | 2018-05-23 14:40:24 +0300 |
| commit | e62d871295d22653caa52a1cbe9161e1256a860c (patch) | |
| tree | be322f9c43602aa74d9cc111228dc1b1204de982 /lib/video-source.c | |
| parent | aa82df887ab995473cd83c89777cdf4bc4685dd0 (diff) | |
v4l2: Document expected fallthrough
Compilers may now enable warnings for implicit fallthrough of switch
case statements, to help catch errors. One such case is highlighted in
our v4l2 module:
v4l2.c: In function ‘v4l2_enum_frame_sizes’:
v4l2.c:179:23: warning: this statement may fall through [-Wimplicit-fallthrough=]
frame->step_height = frmenum.stepwise.step_height;
~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
v4l2.c:180:3: note: here
case V4L2_FRMSIZE_TYPE_CONTINUOUS:
^~~~
This fall through is expected, and the warning can be removed simply by
adding a comment to confirm that.
Signed-off-by: Kieran Bingham <kieran.bingham@ideasonboard.com>
Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Diffstat (limited to 'lib/video-source.c')
0 files changed, 0 insertions, 0 deletions