From patchwork Fri May 24 08:23:33 2019 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: =?utf-8?q?Niklas_S=C3=B6derlund?= X-Patchwork-Id: 1284 Return-Path: Received: from vsp-unauthed02.binero.net (vsp-unauthed02.binero.net [195.74.38.227]) by lancelot.ideasonboard.com (Postfix) with ESMTPS id 5269160BFA for ; Fri, 24 May 2019 10:24:39 +0200 (CEST) X-Halon-ID: 5df5c701-7dfd-11e9-8ab4-005056917a89 Authorized-sender: niklas@soderlund.pp.se Received: from bismarck.berto.se (unknown [89.233.230.99]) by bin-vsp-out-01.atm.binero.net (Halon) with ESMTPA id 5df5c701-7dfd-11e9-8ab4-005056917a89; Fri, 24 May 2019 10:24:37 +0200 (CEST) From: =?utf-8?q?Niklas_S=C3=B6derlund?= To: libcamera-devel@lists.libcamera.org Date: Fri, 24 May 2019 10:23:33 +0200 Message-Id: <20190524082333.8974-3-niklas.soderlund@ragnatech.se> X-Mailer: git-send-email 2.21.0 In-Reply-To: <20190524082333.8974-1-niklas.soderlund@ragnatech.se> References: <20190524082333.8974-1-niklas.soderlund@ragnatech.se> MIME-Version: 1.0 Subject: [libcamera-devel] [PATCH v3 2/2] cam: Add CamApp class X-BeenThere: libcamera-devel@lists.libcamera.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 24 May 2019 08:24:39 -0000 Add more structure to main.cpp by breaking up the logic into a CamApp class. This makes the code easier to read and removes all of the organically grown global variables. Signed-off-by: Niklas Söderlund Reviewed-by: Laurent Pinchart --- src/cam/main.cpp | 198 ++++++++++++++++++++++++++++++++--------------- 1 file changed, 136 insertions(+), 62 deletions(-) diff --git a/src/cam/main.cpp b/src/cam/main.cpp index d071a75dd5347cf7..dbf04917bcc5aa38 100644 --- a/src/cam/main.cpp +++ b/src/cam/main.cpp @@ -18,17 +18,115 @@ using namespace libcamera; -OptionsParser::Options options; -std::shared_ptr camera; -EventLoop *loop; +class CamApp +{ +public: + CamApp(); + + static CamApp *instance(); + + int init(int argc, char **argv); + void cleanup(); + + int exec(); + void quit(); + +private: + int parseOptions(int argc, char *argv[]); + int run(); + + static CamApp *app_; + OptionsParser::Options options_; + CameraManager *cm_; + std::shared_ptr camera_; + EventLoop *loop_; +}; + +CamApp *CamApp::app_ = nullptr; + +CamApp::CamApp() + : cm_(nullptr), camera_(nullptr), loop_(nullptr) +{ + CamApp::app_ = this; +} + +CamApp *CamApp::instance() +{ + return CamApp::app_; +} + +int CamApp::init(int argc, char **argv) +{ + int ret; + + ret = parseOptions(argc, argv); + if (ret < 0) + return ret == -EINTR ? 0 : ret; + + cm_ = CameraManager::instance(); + + ret = cm_->start(); + if (ret) { + std::cout << "Failed to start camera manager: " + << strerror(-ret) << std::endl; + return ret; + } + + if (options_.isSet(OptCamera)) { + camera_ = cm_->get(options_[OptCamera]); + if (!camera_) { + std::cout << "Camera " + << std::string(options_[OptCamera]) + << " not found" << std::endl; + cm_->stop(); + return -ENODEV; + } + + if (camera_->acquire()) { + std::cout << "Failed to acquire camera" << std::endl; + camera_.reset(); + cm_->stop(); + return -EINVAL; + } + + std::cout << "Using camera " << camera_->name() << std::endl; + } + + loop_ = new EventLoop(cm_->eventDispatcher()); + + return 0; +} + +void CamApp::cleanup() +{ + delete loop_; + loop_ = nullptr; + + if (camera_) { + camera_->release(); + camera_.reset(); + } + + cm_->stop(); +} + +int CamApp::exec() +{ + int ret; + + ret = run(); + cleanup(); + + return ret; +} -void signalHandler(int signal) +void CamApp::quit() { - std::cout << "Exiting" << std::endl; - loop->exit(); + if (loop_) + loop_->exit(); } -static int parseOptions(int argc, char *argv[]) +int CamApp::parseOptions(int argc, char *argv[]) { KeyValueParser streamKeyValue; streamKeyValue.addOption("role", OptionString, @@ -58,77 +156,53 @@ static int parseOptions(int argc, char *argv[]) "help"); parser.addOption(OptList, OptionNone, "List all cameras", "list"); - options = parser.parse(argc, argv); - if (!options.valid()) + options_ = parser.parse(argc, argv); + if (!options_.valid()) return -EINVAL; - if (options.empty() || options.isSet(OptHelp)) { + if (options_.empty() || options_.isSet(OptHelp)) { parser.usage(); - return options.empty() ? -EINVAL : -EINTR; + return options_.empty() ? -EINVAL : -EINTR; } return 0; } +int CamApp::run() +{ + if (options_.isSet(OptList)) { + std::cout << "Available cameras:" << std::endl; + for (const std::shared_ptr &cam : cm_->cameras()) + std::cout << "- " << cam->name() << std::endl; + } + + if (options_.isSet(OptCapture)) { + Capture capture(camera_.get()); + return capture.run(loop_, options_); + } + + return 0; +} + +void signalHandler(int signal) +{ + std::cout << "Exiting" << std::endl; + CamApp::instance()->quit(); +} + int main(int argc, char **argv) { - int ret; + CamApp app; - ret = parseOptions(argc, argv); - if (ret < 0) - return ret == -EINTR ? 0 : EXIT_FAILURE; - - CameraManager *cm = CameraManager::instance(); - - ret = cm->start(); - if (ret) { - std::cout << "Failed to start camera manager: " - << strerror(-ret) << std::endl; + if (app.init(argc, argv)) return EXIT_FAILURE; - } - - loop = new EventLoop(cm->eventDispatcher()); struct sigaction sa = {}; sa.sa_handler = &signalHandler; sigaction(SIGINT, &sa, nullptr); - if (options.isSet(OptList)) { - std::cout << "Available cameras:" << std::endl; - for (const std::shared_ptr &cam : cm->cameras()) - std::cout << "- " << cam->name() << std::endl; - } + if (app.exec()) + return EXIT_FAILURE; - if (options.isSet(OptCamera)) { - camera = cm->get(options[OptCamera]); - if (!camera) { - std::cout << "Camera " - << std::string(options[OptCamera]) - << " not found" << std::endl; - goto out; - } - - if (camera->acquire()) { - std::cout << "Failed to acquire camera" << std::endl; - goto out; - } - - std::cout << "Using camera " << camera->name() << std::endl; - } - - if (options.isSet(OptCapture)) { - Capture capture(camera.get()); - ret = capture.run(loop, options); - } - - if (camera) { - camera->release(); - camera.reset(); - } -out: - delete loop; - - cm->stop(); - - return ret; + return 0; }