[libcamera-devel] [PATCH 3/4] libcamera: thread: Drop doc of asynchronous signals delivery to applications

[Niklas Söderlund]

Hi Laurent,

Thanks for your work.

On 2021-05-23 05:31:54 +0300, Laurent Pinchart wrote:
> The asynchronous signal delivery mechanism can't be used by application,
> as it requires libcamera to be aware of the application event loop,
> which has been dropped a long time ago. Drop the corresponding
> documentation paragraph.
> 
> Signed-off-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com>

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

> ---
>  src/libcamera/thread.cpp | 8 --------
>  1 file changed, 8 deletions(-)
> 
> diff --git a/src/libcamera/thread.cpp b/src/libcamera/thread.cpp
> index bc1dcedea0d1..d59e43966d26 100644
> --- a/src/libcamera/thread.cpp
> +++ b/src/libcamera/thread.cpp
> @@ -64,14 +64,6 @@
>   * be overridden by selecting a different connection type when calling
>   * Signal::connect().
>   *
> - * Asynchronous signal delivery is used internally in libcamera, but is also
> - * available to applications if desired. To use this feature, applications
> - * shall create receiver classes that inherit from the Object class, and
> - * provide an event loop to the CameraManager as explained above. Note that
> - * Object instances created by the application are limited to living in the
> - * application's main thread. Creating Object instances from another thread of
> - * an application causes undefined behaviour.
> - *
>   * \section thread-reentrancy Reentrancy and Thread-Safety
>   *
>   * Through the documentation, several terms are used to define how classes and
> -- 
> Regards,
> 
> Laurent Pinchart
> 

-- 
Regards,
Niklas Söderlund