[libcamera-devel] [PATCH 1/3] libcamera: media_object: Remove unneeded destructors

[Umang Jain]

Hi Laurent,

Thank you for the patch.

On 10/23/20 10:40 AM, Laurent Pinchart wrote:
> The MediaEntity, MediaLink and MediaPad classes don't need custom
> destructors. For MediaEntity and MediaPad, the destructors clear a
> vector embedded in the classes, which will be done by the default
> destructor. For MediaLink, the destructor is already empty. Remove them.
>
> Signed-off-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com>
Reviewed-by: Umang Jain <email at uajain.com>
> ---
>   include/libcamera/internal/media_object.h |  3 ---
>   src/libcamera/media_object.cpp            | 18 ------------------
>   2 files changed, 21 deletions(-)
>
> diff --git a/include/libcamera/internal/media_object.h b/include/libcamera/internal/media_object.h
> index be6fb8961349..54311c14895b 100644
> --- a/include/libcamera/internal/media_object.h
> +++ b/include/libcamera/internal/media_object.h
> @@ -51,7 +51,6 @@ private:
>   	MediaLink(const struct media_v2_link *link,
>   		  MediaPad *source, MediaPad *sink);
>   	MediaLink(const MediaLink &) = delete;
> -	~MediaLink() {}
>   
>   	MediaPad *source_;
>   	MediaPad *sink_;
> @@ -73,7 +72,6 @@ private:
>   
>   	MediaPad(const struct media_v2_pad *pad, MediaEntity *entity);
>   	MediaPad(const MediaPad &) = delete;
> -	~MediaPad();
>   
>   	unsigned int index_;
>   	MediaEntity *entity_;
> @@ -105,7 +103,6 @@ private:
>   	MediaEntity(MediaDevice *dev, const struct media_v2_entity *entity,
>   		    unsigned int major = 0, unsigned int minor = 0);
>   	MediaEntity(const MediaEntity &) = delete;
> -	~MediaEntity();
>   
>   	void addPad(MediaPad *pad);
>   
> diff --git a/src/libcamera/media_object.cpp b/src/libcamera/media_object.cpp
> index b7bf048c985d..056036635902 100644
> --- a/src/libcamera/media_object.cpp
> +++ b/src/libcamera/media_object.cpp
> @@ -191,15 +191,6 @@ MediaPad::MediaPad(const struct media_v2_pad *pad, MediaEntity *entity)
>   {
>   }
>   
> -MediaPad::~MediaPad()
> -{
> -	/*
> -	 * Don't delete the links as we only borrow the reference owned by
> -	 * MediaDevice.
> -	 */
> -	links_.clear();
> -}
> -
>   /**
>    * \fn MediaPad::index()
>    * \brief Retrieve the pad index
> @@ -371,15 +362,6 @@ MediaEntity::MediaEntity(MediaDevice *dev,
>   {
>   }
>   
> -MediaEntity::~MediaEntity()
> -{
> -	/*
> -	 * Don't delete the pads as we only borrow the reference owned by
> -	 * MediaDevice.
> -	 */
> -	pads_.clear();
> -}
> -
>   /**
>    * \brief Add \a pad to the entity's list of pads
>    * \param[in] pad The pad to add to the list