[libcamera-devel] [PATCH 4/5] libcamera: object: Document danger of deleting object from other thread

[Niklas Söderlund]

Hi Laurent,

Thanks for your work.

On 2019-11-27 10:49:08 +0200, Laurent Pinchart wrote:
> Object instances receive messages dispatched from the event loop of the
> thread they belong to. Deleting an object from a different thread is
> thus dangerous, unless the caller ensures that no message delivery is in
> progress. Document this in the Object class documentation.
> 
> Signed-off-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com>

Reviewed-by: Niklas Söderlund <niklas.soderlund at ragnatech.se>

> ---
>  src/libcamera/object.cpp | 4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/src/libcamera/object.cpp b/src/libcamera/object.cpp
> index 1f787271f782..e76faf48b8ed 100644
> --- a/src/libcamera/object.cpp
> +++ b/src/libcamera/object.cpp
> @@ -40,6 +40,10 @@ LOG_DEFINE_CATEGORY(Object)
>   * implementing easy message passing between threads by inheriting from the
>   * Object class.
>   *
> + * Deleting an object from a thread other than the one the object is bound to is
> + * unsafe, unless the caller ensures that the object isn't processing any
> + * message concurrently.
> + *
>   * Object slots connected to signals will also run in the context of the
>   * object's thread, regardless of whether the signal is emitted in the same or
>   * in another thread.
> -- 
> Regards,
> 
> Laurent Pinchart
> 
> _______________________________________________
> libcamera-devel mailing list
> libcamera-devel at lists.libcamera.org
> https://lists.libcamera.org/listinfo/libcamera-devel

-- 
Regards,
Niklas Söderlund