summaryrefslogtreecommitdiff
path: root/src/android/jpeg/encoder.h
diff options
context:
space:
mode:
authorHirokazu Honda <hiroh@chromium.org>2020-10-21 10:39:53 +0900
committerKieran Bingham <kieran.bingham@ideasonboard.com>2020-10-21 11:18:12 +0100
commit90c193f2a70093917730b86359c6e459c28d2c24 (patch)
treef32f6dffbb7542df962adf1fa46f7b4f026d1e14 /src/android/jpeg/encoder.h
parentb8dd5ce944eca4d17df585bb059595729905f7ec (diff)
android: Modify Encoder interface
In Encoder::encode(), the |source| argument doesn't have to be a pointer. This replaces its type, const pointer, with const reference as the latter is preferred to the former. libcamera::Span is cheap to construct/copy/move. We should deal with the type as pass-by-value parameter. Therefore this also drops the const reference in the |destination| argument. Signed-off-by: Hirokazu Honda <hiroh@chromium.org> Reviewed-by: Umang Jain <email@uajain.com> Reviewed-by: Kieran Bingham <kieran.bingham@ideasonboard.com> Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> Signed-off-by: Kieran Bingham <kieran.bingham@ideasonboard.com>
Diffstat (limited to 'src/android/jpeg/encoder.h')
-rw-r--r--src/android/jpeg/encoder.h4
1 files changed, 2 insertions, 2 deletions
diff --git a/src/android/jpeg/encoder.h b/src/android/jpeg/encoder.h
index 4e859961..270ea609 100644
--- a/src/android/jpeg/encoder.h
+++ b/src/android/jpeg/encoder.h
@@ -17,8 +17,8 @@ public:
virtual ~Encoder() {}
virtual int configure(const libcamera::StreamConfiguration &cfg) = 0;
- virtual int encode(const libcamera::FrameBuffer *source,
- const libcamera::Span<uint8_t> &destination,
+ virtual int encode(const libcamera::FrameBuffer &source,
+ libcamera::Span<uint8_t> destination,
const libcamera::Span<const uint8_t> &exifData) = 0;
};