From 8d846ed8a7d9d7c9783472a618a7f369f0843e67 Mon Sep 17 00:00:00 2001 From: Laurent Pinchart Date: Tue, 21 May 2019 16:18:01 +0300 Subject: cam: Rename conf variable referring to command line option to opt Naming a variable that refers to command line options 'conf' is confusing as we using 'config' and 'cfg' to refer to camera and stream configurations. Rename it to 'opt'. Signed-off-by: Laurent Pinchart Reviewed-by: Jacopo Mondi --- src/cam/main.cpp | 26 +++++++++++++------------- 1 file changed, 13 insertions(+), 13 deletions(-) (limited to 'src/cam/main.cpp') diff --git a/src/cam/main.cpp b/src/cam/main.cpp index 5ecd7e0e..41558896 100644 --- a/src/cam/main.cpp +++ b/src/cam/main.cpp @@ -98,19 +98,19 @@ static std::unique_ptr prepareCameraConfig() /* Use roles and get a default configuration. */ for (auto const &value : streamOptions) { - KeyValueParser::Options conf = value.toKeyValues(); + KeyValueParser::Options opt = value.toKeyValues(); - if (!conf.isSet("role")) { + if (!opt.isSet("role")) { roles.push_back(StreamRole::VideoRecording); - } else if (conf["role"].toString() == "viewfinder") { + } else if (opt["role"].toString() == "viewfinder") { roles.push_back(StreamRole::Viewfinder); - } else if (conf["role"].toString() == "video") { + } else if (opt["role"].toString() == "video") { roles.push_back(StreamRole::VideoRecording); - } else if (conf["role"].toString() == "still") { + } else if (opt["role"].toString() == "still") { roles.push_back(StreamRole::StillCapture); } else { std::cerr << "Unknown stream role " - << conf["role"].toString() << std::endl; + << opt["role"].toString() << std::endl; return nullptr; } } @@ -125,18 +125,18 @@ static std::unique_ptr prepareCameraConfig() /* Apply configuration explicitly requested. */ unsigned int i = 0; for (auto const &value : streamOptions) { - KeyValueParser::Options conf = value.toKeyValues(); + KeyValueParser::Options opt = value.toKeyValues(); StreamConfiguration &cfg = config->at(i++); - if (conf.isSet("width")) - cfg.size.width = conf["width"]; + if (opt.isSet("width")) + cfg.size.width = opt["width"]; - if (conf.isSet("height")) - cfg.size.height = conf["height"]; + if (opt.isSet("height")) + cfg.size.height = opt["height"]; /* TODO: Translate 4CC string to ID. */ - if (conf.isSet("pixelformat")) - cfg.pixelFormat = conf["pixelformat"]; + if (opt.isSet("pixelformat")) + cfg.pixelFormat = opt["pixelformat"]; } return config; -- cgit v1.2.1