diff options
author | Jacopo Mondi <jacopo@jmondi.org> | 2022-08-03 12:55:39 +0200 |
---|---|---|
committer | Jacopo Mondi <jacopo@jmondi.org> | 2022-08-03 15:07:20 +0200 |
commit | 2e77ccbb93a14a0c976bb3ca9d5dbbdc32a12847 (patch) | |
tree | 8c79730c86021cfd96157d7f61f47346e7ac1b51 /Documentation/contributing.rst | |
parent | ac54f2ac6dede6a4d8ed761660c3f74b42a497a3 (diff) |
libcamera: yaml_parser: Return nullopt on error
The YamlParser::getList<>() function returns an std::optional<> to allow
callers to identify cases where parsing the .yaml file failed from cases
where the parsed list is just empty.
The current implementation returns a default constructed std::optional
in case of errors with
return {};
The returned value is thus equal to std::nullopt, but the code can be
easily misinterpreted as returning an empty vector by a reader.
Signed-off-by: Jacopo Mondi <jacopo@jmondi.org>
Reviewed-by: Paul Elder <paul.elder@ideasonboard.com>
Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Diffstat (limited to 'Documentation/contributing.rst')
0 files changed, 0 insertions, 0 deletions