[libcamera-devel,v2,1/6] android: camera_metadata: Mark isValid() as const

Message ID 20200725144058.129388-2-jacopo@jmondi.org
State Accepted
Headers show
Series
  • android: camera_device: Generate correct preview template
Related show

Commit Message

Jacopo Mondi July 25, 2020, 2:40 p.m. UTC
The CameraMetadata::isValid() method does not modify the object state
and can be called on const instances of the CameraMetadata class.

Mark the method as const to allow that.

Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.comR
Reviewed-by: Kieran Bingham <kieran.bingham@ideasonboard.com>
Signed-off-by: Jacopo Mondi <jacopo@jmondi.org>
---
 src/android/camera_metadata.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Laurent Pinchart July 25, 2020, 4:15 p.m. UTC | #1
On Sat, Jul 25, 2020 at 04:40:53PM +0200, Jacopo Mondi wrote:
> The CameraMetadata::isValid() method does not modify the object state
> and can be called on const instances of the CameraMetadata class.
> 
> Mark the method as const to allow that.
> 
> Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.comR

s/comR/com>/

> Reviewed-by: Kieran Bingham <kieran.bingham@ideasonboard.com>
> Signed-off-by: Jacopo Mondi <jacopo@jmondi.org>
> ---
>  src/android/camera_metadata.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/src/android/camera_metadata.h b/src/android/camera_metadata.h
> index 75a9d7066f31..348a018165ef 100644
> --- a/src/android/camera_metadata.h
> +++ b/src/android/camera_metadata.h
> @@ -17,7 +17,7 @@ public:
>  	CameraMetadata(size_t entryCapacity, size_t dataCapacity);
>  	~CameraMetadata();
>  
> -	bool isValid() { return valid_; }
> +	bool isValid() const { return valid_; }
>  	bool addEntry(uint32_t tag, const void *data, size_t data_count);
>  
>  	camera_metadata_t *get();

Patch

diff --git a/src/android/camera_metadata.h b/src/android/camera_metadata.h
index 75a9d7066f31..348a018165ef 100644
--- a/src/android/camera_metadata.h
+++ b/src/android/camera_metadata.h
@@ -17,7 +17,7 @@  public:
 	CameraMetadata(size_t entryCapacity, size_t dataCapacity);
 	~CameraMetadata();
 
-	bool isValid() { return valid_; }
+	bool isValid() const { return valid_; }
 	bool addEntry(uint32_t tag, const void *data, size_t data_count);
 
 	camera_metadata_t *get();