diff options
| author | Jacopo Mondi <jacopo@jmondi.org> | 2021-04-18 16:42:17 +0200 |
|---|---|---|
| committer | Jacopo Mondi <jacopo@jmondi.org> | 2021-04-21 09:52:34 +0200 |
| commit | 1fca6228fece89c9270ff1abb293fdf48f1b577b (patch) | |
| tree | cd1aeaa7cf980d0703bbd16ce969a48b2586f751 /src/android | |
| parent | d40430116b1b65d8deeeeea106368e20d8534a03 (diff) | |
libcamera: Drop argument from LIBCAMERA_DECLARE_PRIVATE
The LIBCAMERA_DECLARE_PRIVATE() macro, used by the library classes
that inherit from libcamera::Extensible in order to implement the
PIMPL pattern, expands to:
public: \
class Private; \
friend class Private;
The 'klass' argument is not used and it might confuse developers as
it might hint that the class that defines the pattern's implementation
can be freely named, while it is actually hardcoded to 'Private'.
Drop the argument from the macro definition.
Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Reviewed-by: Niklas Söderlund <niklas.soderlund@ragnatech.se>
Reviewed-by: Hanlin Chen <hanlinchen@google.com>
Signed-off-by: Jacopo Mondi <jacopo@jmondi.org>
Diffstat (limited to 'src/android')
| -rw-r--r-- | src/android/camera_buffer.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/android/camera_buffer.h b/src/android/camera_buffer.h index 7e8970b4..c88124b2 100644 --- a/src/android/camera_buffer.h +++ b/src/android/camera_buffer.h @@ -14,7 +14,7 @@ class CameraBuffer final : public libcamera::Extensible { - LIBCAMERA_DECLARE_PRIVATE(CameraBuffer) + LIBCAMERA_DECLARE_PRIVATE() public: CameraBuffer(buffer_handle_t camera3Buffer, int flags); |