From patchwork Thu May 23 15:13:06 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: 1275 Return-Path: Received: from bin-mail-out-05.binero.net (bin-mail-out-05.binero.net [195.74.38.228]) by lancelot.ideasonboard.com (Postfix) with ESMTPS id 23B0361865 for ; Thu, 23 May 2019 17:13:54 +0200 (CEST) X-Halon-ID: 5e52aba0-7d6d-11e9-8d05-005056917f90 Authorized-sender: niklas@soderlund.pp.se Received: from bismarck.berto.se (unknown [89.233.230.99]) by bin-vsp-out-02.atm.binero.net (Halon) with ESMTPA id 5e52aba0-7d6d-11e9-8d05-005056917f90; Thu, 23 May 2019 17:13:50 +0200 (CEST) From: =?utf-8?q?Niklas_S=C3=B6derlund?= To: libcamera-devel@lists.libcamera.org Date: Thu, 23 May 2019 17:13:06 +0200 Message-Id: <20190523151306.22007-3-niklas.soderlund@ragnatech.se> X-Mailer: git-send-email 2.21.0 In-Reply-To: <20190523151306.22007-1-niklas.soderlund@ragnatech.se> References: <20190523151306.22007-1-niklas.soderlund@ragnatech.se> MIME-Version: 1.0 Subject: [libcamera-devel] [PATCH v2 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: Thu, 23 May 2019 15:13:54 -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 | 184 ++++++++++++++++++++++++++++++++--------------- 1 file changed, 125 insertions(+), 59 deletions(-) diff --git a/src/cam/main.cpp b/src/cam/main.cpp index fe7d4f90dbf14ffd..f7fe83aafd81b633 100644 --- a/src/cam/main.cpp +++ b/src/cam/main.cpp @@ -18,17 +18,117 @@ 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[]); + + OptionsParser::Options options_; + CameraManager *cm_; + std::shared_ptr camera_; + EventLoop *loop_; +}; + +CamApp::CamApp() + : cm_(nullptr), camera_(nullptr), loop_(nullptr) +{ +} + +CamApp *CamApp::instance() +{ + static CamApp app; + return &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() +{ + 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; + return capture.run(camera_.get(), loop_, options_); + } + + return 0; +} -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 +158,43 @@ 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; } +void signalHandler(int signal) +{ + std::cout << "Exiting" << std::endl; + CamApp::instance()->quit(); +} + int main(int argc, char **argv) { + CamApp *app = CamApp::instance(); int ret; - 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; + ret = app->init(argc, argv); + if (ret) 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; - } + ret = app->exec(); - if (options.isSet(OptCamera)) { - camera = cm->get(options[OptCamera]); - if (!camera) { - std::cout << "Camera " - << std::string(options[OptCamera]) - << " not found" << std::endl; - goto out; - } + app->cleanup(); - if (camera->acquire()) { - std::cout << "Failed to acquire camera" << std::endl; - goto out; - } + if (ret) + return EXIT_FAILURE; - std::cout << "Using camera " << camera->name() << std::endl; - } - - if (options.isSet(OptCapture)) { - Capture capture; - ret = capture.run(camera.get(), loop, options); - } - - if (camera) { - camera->release(); - camera.reset(); - } -out: - delete loop; - - cm->stop(); - - return ret; + return 0; }