diff options
| author | Stefan Klug <stefan.klug@ideasonboard.com> | 2024-08-08 18:30:41 +0100 |
|---|---|---|
| committer | Kieran Bingham <kieran.bingham@ideasonboard.com> | 2024-09-12 11:12:28 +0200 |
| commit | 33a49ce4a265c88ade6cc7e94d5124f5d6ebaea0 (patch) | |
| tree | 6f483e9011ec37d4373d3cf9231a0b25220fdfee /Documentation/coding-style.rst | |
| parent | 75fc9447382a22260e98a7950859e0adb379d595 (diff) | |
libcamera: v4l2_videodevice: Improve readability
The handling for the sequence number validation within
V4L2VideoDevice::dequeueBuffer makes use of a std::optional, which can
be used as a boolean in conditional statements. This has the impact in
this use case that it can be mis-read to be interpretting the value for
firstFrame_ which is assigned as the buf.sequence.
Remove this potential for confusion by making it clear that the first
frame handling is only performed when firstFrame_ does not have a value
assigned.
Signed-off-by: Stefan Klug <stefan.klug@ideasonboard.com>
Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
[Kieran: Rework commit message]
Signed-off-by: Kieran Bingham <kieran.bingham@ideasonboard.com>
Diffstat (limited to 'Documentation/coding-style.rst')
0 files changed, 0 insertions, 0 deletions