From 6b67094cd231548df3a42d72351ca9e8841b7033 Mon Sep 17 00:00:00 2001 From: Stefan Klug Date: Fri, 20 Sep 2024 12:13:41 +0200 Subject: libcamera: yaml-parser: Differentiate between empty and empty string When accessing a nonexistent key on a dict the YamlObject returns an empty element. This element can happily be cast to a string which is unexpected. For example the following statement: yamlDict["nonexistent"].get("default") is expected to return "default" but actually returns "". Fix this by introducing an empty type to distinguish between an empty YamlObject and a YamlObject of type value containing an empty string. For completeness add an isEmpty() function and an explicit cast to bool to be able to test for that type. Extend the tests accordingly. Signed-off-by: Stefan Klug Reviewed-by: Kieran Bingham Reviewed-by: Paul Elder --- test/meson.build | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'test/meson.build') diff --git a/test/meson.build b/test/meson.build index dcd169a8..5ed052ed 100644 --- a/test/meson.build +++ b/test/meson.build @@ -73,7 +73,7 @@ internal_tests = [ {'name': 'timer-thread', 'sources': ['timer-thread.cpp']}, {'name': 'unique-fd', 'sources': ['unique-fd.cpp']}, {'name': 'utils', 'sources': ['utils.cpp']}, - {'name': 'yaml-parser', 'sources': ['yaml-parser.cpp'], 'should_fail': true}, + {'name': 'yaml-parser', 'sources': ['yaml-parser.cpp']}, ] internal_non_parallel_tests = [ -- cgit v1.2.1