diff options
author | Barnabás Pőcze <barnabas.pocze@ideasonboard.com> | 2025-03-21 10:28:41 +0100 |
---|---|---|
committer | Barnabás Pőcze <barnabas.pocze@ideasonboard.com> | 2025-04-22 14:49:10 +0200 |
commit | f31da7272eb0d8dfcd551057b24b5ca4f6985713 (patch) | |
tree | dff0ec5685f68435befccbfc2570d4c9e574c3f0 /src | |
parent | 86c45c8fdf36b6983ad0d585184218f541bb4fd5 (diff) |
libcamera: ipa_module: Avoid unnecessary copy when getting signature
The `signature()` getter can just return a reference to the private vector
member variable, and let the caller make a copy if needed. Since the
return type is const qualified, this was likely the original intention.
Signed-off-by: Barnabás Pőcze <barnabas.pocze@ideasonboard.com>
Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Reviewed-by: Paul Elder <paul.elder@ideasonboard.com>
Diffstat (limited to 'src')
-rw-r--r-- | src/libcamera/ipa_module.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/libcamera/ipa_module.cpp b/src/libcamera/ipa_module.cpp index 9ca74be6..e6ea61e4 100644 --- a/src/libcamera/ipa_module.cpp +++ b/src/libcamera/ipa_module.cpp @@ -373,7 +373,7 @@ const struct IPAModuleInfo &IPAModule::info() const * * \return The IPA module signature */ -const std::vector<uint8_t> IPAModule::signature() const +const std::vector<uint8_t> &IPAModule::signature() const { return signature_; } |