summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorHirokazu Honda <hiroh@chromium.org>2021-10-05 16:31:11 +0900
committerLaurent Pinchart <laurent.pinchart@ideasonboard.com>2021-10-15 05:05:39 +0300
commitf277590d5cb13ad7caff189d21ce87df54f709f1 (patch)
tree09b6cfac62ad0831cb5b2ac124587e2445a38c56
parent3b93746907dfd1367b107af943f1df5a37a2eb29 (diff)
qcam: Remove using namespace in header files
"using namespace" in a header file propagates the namespace to the files including the header file. So it should be avoided. This removes "using namespace" in header files in qcam. Signed-off-by: Hirokazu Honda <hiroh@chromium.org> Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
-rw-r--r--src/qcam/dng_writer.h10
-rw-r--r--src/qcam/main.cpp4
-rw-r--r--src/qcam/main_window.h40
3 files changed, 27 insertions, 27 deletions
diff --git a/src/qcam/dng_writer.h b/src/qcam/dng_writer.h
index 20905f37..e4486288 100644
--- a/src/qcam/dng_writer.h
+++ b/src/qcam/dng_writer.h
@@ -15,15 +15,13 @@
#include <libcamera/framebuffer.h>
#include <libcamera/stream.h>
-using namespace libcamera;
-
class DNGWriter
{
public:
- static int write(const char *filename, const Camera *camera,
- const StreamConfiguration &config,
- const ControlList &metadata,
- const FrameBuffer *buffer, const void *data);
+ static int write(const char *filename, const libcamera::Camera *camera,
+ const libcamera::StreamConfiguration &config,
+ const libcamera::ControlList &metadata,
+ const libcamera::FrameBuffer *buffer, const void *data);
};
#endif /* HAVE_TIFF */
diff --git a/src/qcam/main.cpp b/src/qcam/main.cpp
index 5eff90a3..d3f01a85 100644
--- a/src/qcam/main.cpp
+++ b/src/qcam/main.cpp
@@ -18,6 +18,8 @@
#include "main_window.h"
#include "message_handler.h"
+using namespace libcamera;
+
void signalHandler([[maybe_unused]] int signal)
{
qInfo() << "Exiting";
@@ -66,7 +68,7 @@ int main(int argc, char **argv)
sa.sa_handler = &signalHandler;
sigaction(SIGINT, &sa, nullptr);
- CameraManager *cm = new CameraManager();
+ CameraManager *cm = new libcamera::CameraManager();
ret = cm->start();
if (ret) {
diff --git a/src/qcam/main_window.h b/src/qcam/main_window.h
index a16bea09..4d8e806b 100644
--- a/src/qcam/main_window.h
+++ b/src/qcam/main_window.h
@@ -29,8 +29,6 @@
#include "../cam/stream_options.h"
#include "viewfinder.h"
-using namespace libcamera;
-
class QAction;
class QComboBox;
@@ -50,7 +48,8 @@ class MainWindow : public QMainWindow
Q_OBJECT
public:
- MainWindow(CameraManager *cm, const OptionsParser::Options &options);
+ MainWindow(libcamera::CameraManager *cm,
+ const OptionsParser::Options &options);
~MainWindow();
bool event(QEvent *e) override;
@@ -64,9 +63,10 @@ private Q_SLOTS:
void saveImageAs();
void captureRaw();
- void processRaw(FrameBuffer *buffer, const ControlList &metadata);
+ void processRaw(libcamera::FrameBuffer *buffer,
+ const libcamera::ControlList &metadata);
- void queueRequest(FrameBuffer *buffer);
+ void queueRequest(libcamera::FrameBuffer *buffer);
private:
int createToolbars();
@@ -77,13 +77,13 @@ private:
int startCapture();
void stopCapture();
- void addCamera(std::shared_ptr<Camera> camera);
- void removeCamera(std::shared_ptr<Camera> camera);
+ void addCamera(std::shared_ptr<libcamera::Camera> camera);
+ void removeCamera(std::shared_ptr<libcamera::Camera> camera);
- void requestComplete(Request *request);
+ void requestComplete(libcamera::Request *request);
void processCapture();
void processHotplug(HotplugEvent *e);
- void processViewfinder(FrameBuffer *buffer);
+ void processViewfinder(libcamera::FrameBuffer *buffer);
/* UI elements */
QToolBar *toolbar_;
@@ -102,21 +102,21 @@ private:
const OptionsParser::Options &options_;
/* Camera manager, camera, configuration and buffers */
- CameraManager *cm_;
- std::shared_ptr<Camera> camera_;
- FrameBufferAllocator *allocator_;
+ libcamera::CameraManager *cm_;
+ std::shared_ptr<libcamera::Camera> camera_;
+ libcamera::FrameBufferAllocator *allocator_;
- std::unique_ptr<CameraConfiguration> config_;
- std::map<FrameBuffer *, std::unique_ptr<Image>> mappedBuffers_;
+ std::unique_ptr<libcamera::CameraConfiguration> config_;
+ std::map<libcamera::FrameBuffer *, std::unique_ptr<Image>> mappedBuffers_;
/* Capture state, buffers queue and statistics */
bool isCapturing_;
bool captureRaw_;
- Stream *vfStream_;
- Stream *rawStream_;
- std::map<const Stream *, QQueue<FrameBuffer *>> freeBuffers_;
- QQueue<Request *> doneQueue_;
- QQueue<Request *> freeQueue_;
+ libcamera::Stream *vfStream_;
+ libcamera::Stream *rawStream_;
+ std::map<const libcamera::Stream *, QQueue<libcamera::FrameBuffer *>> freeBuffers_;
+ QQueue<libcamera::Request *> doneQueue_;
+ QQueue<libcamera::Request *> freeQueue_;
QMutex mutex_; /* Protects freeBuffers_, doneQueue_, and freeQueue_ */
uint64_t lastBufferTime_;
@@ -124,7 +124,7 @@ private:
uint32_t previousFrames_;
uint32_t framesCaptured_;
- std::vector<std::unique_ptr<Request>> requests_;
+ std::vector<std::unique_ptr<libcamera::Request>> requests_;
};
#endif /* __QCAM_MAIN_WINDOW__ */