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 /include | |
| 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 'include')
| -rw-r--r-- | include/libcamera/internal/ipa_module.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/include/libcamera/internal/ipa_module.h b/include/libcamera/internal/ipa_module.h index 7c49d3f3..15f19492 100644 --- a/include/libcamera/internal/ipa_module.h +++ b/include/libcamera/internal/ipa_module.h @@ -29,7 +29,7 @@ public: bool isValid() const; const struct IPAModuleInfo &info() const; - const std::vector<uint8_t> signature() const; + const std::vector<uint8_t> &signature() const; const std::string &path() const; bool load(); |