diff options
| author | Barnabás Pőcze <pobrn@protonmail.com> | 2025-01-21 13:12:54 +0100 |
|---|---|---|
| committer | Barnabás Pőcze <pobrn@protonmail.com> | 2025-02-27 11:30:23 +0100 |
| commit | d40250e03b172ecc5f81299499ea0901f3d5b16d (patch) | |
| tree | bf6cc13bc4f87c48d01e53068388543b1469c561 /include | |
| parent | f72c76eb6e06a41d2aaff8c8c4002dea21a7774d (diff) | |
libcamera: base: log: Remove move constructor
C++17 guarantees move and copy elision in certain cases,
such as when returning a prvalue of the same type as the
return type of the function.
This is what the `_log()` functions do, thus there is no need
for the move constructor, so remove it. Furthermore, do not
just remove the implementation, but instead delete it as well.
Signed-off-by: Barnabás Pőcze <pobrn@protonmail.com>
Reviewed-by: Jacopo Mondi <jacopo.mondi@ideasonboard.com>
Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Diffstat (limited to 'include')
| -rw-r--r-- | include/libcamera/base/log.h | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/include/libcamera/base/log.h b/include/libcamera/base/log.h index 62093012..b3050eed 100644 --- a/include/libcamera/base/log.h +++ b/include/libcamera/base/log.h @@ -61,8 +61,6 @@ public: LogMessage(const char *fileName, unsigned int line, const LogCategory &category, LogSeverity severity, const std::string &prefix = std::string()); - - LogMessage(LogMessage &&); ~LogMessage(); std::ostream &stream() { return msgStream_; } @@ -75,7 +73,7 @@ public: const std::string msg() const { return msgStream_.str(); } private: - LIBCAMERA_DISABLE_COPY(LogMessage) + LIBCAMERA_DISABLE_COPY_AND_MOVE(LogMessage) void init(const char *fileName, unsigned int line); |