diff options
author | Umang Jain <email@uajain.com> | 2020-12-04 12:24:49 +0530 |
---|---|---|
committer | Kieran Bingham <kieran.bingham@ideasonboard.com> | 2020-12-04 10:14:02 +0000 |
commit | e6663c0f8c86305acfc9562f75c11b75356a67ec (patch) | |
tree | 7bd29f7f99d1d14d28fca61257b85a337d46920c /simple-cam.cpp | |
parent | c6626f9caf632efe05b521730cbf52da823dbd43 (diff) |
simple-cam: Make return codes consistent for main()
Use EXIT_FAILURE or EXIT_SUCCESS to indicate program execution
status. These are the return codes that should be used for main()
as per the C++ standard.
Signed-off-by: Umang Jain <email@uajain.com>
Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Signed-off-by: Kieran Bingham <kieran.bingham@ideasonboard.com>
Diffstat (limited to 'simple-cam.cpp')
-rw-r--r-- | simple-cam.cpp | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/simple-cam.cpp b/simple-cam.cpp index 588018f..fefc980 100644 --- a/simple-cam.cpp +++ b/simple-cam.cpp @@ -184,7 +184,7 @@ int main() int ret = camera->configure(config.get()); if (ret) { std::cout << "CONFIGURATION FAILED!" << std::endl; - return -1; + return EXIT_FAILURE; } #endif @@ -217,7 +217,7 @@ int main() int ret = allocator->allocate(cfg.stream()); if (ret < 0) { std::cerr << "Can't allocate buffers" << std::endl; - return -ENOMEM; + return EXIT_FAILURE; } unsigned int allocated = allocator->buffers(cfg.stream()).size(); @@ -250,7 +250,7 @@ int main() if (!request) { std::cerr << "Can't create request" << std::endl; - return -ENOMEM; + return EXIT_FAILURE; } const std::unique_ptr<FrameBuffer> &buffer = buffers[i]; @@ -259,7 +259,7 @@ int main() { std::cerr << "Can't set buffer for request" << std::endl; - return ret; + return EXIT_FAILURE; } /* @@ -341,5 +341,5 @@ int main() camera.reset(); cm->stop(); - return 0; + return EXIT_SUCCESS; } |