[libcamera-devel] [PATCH 1/5] test: message: Fix message handling in MessageReceiver

Niklas Söderlund niklas.soderlund at ragnatech.se
Wed Nov 27 16:01:11 CET 2019


Hi Laurent,

Thanks for your work.

On 2019-11-27 10:49:05 +0200, Laurent Pinchart wrote:
> Forward messages that we don't handle to the base Object class, to avoid
> both blocking the ThreadMove message and mistaking it as the test
> message.
> 
> Signed-off-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com>

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

> ---
>  test/message.cpp | 5 +++++
>  1 file changed, 5 insertions(+)
> 
> diff --git a/test/message.cpp b/test/message.cpp
> index 3775c30a20b3..cf21d5ca50d1 100644
> --- a/test/message.cpp
> +++ b/test/message.cpp
> @@ -37,6 +37,11 @@ public:
>  protected:
>  	void message(Message *msg)
>  	{
> +		if (msg->type() != Message::None) {
> +			Object::message(msg);
> +			return;
> +		}
> +
>  		if (thread() != Thread::current())
>  			status_ = InvalidThread;
>  		else
> -- 
> Regards,
> 
> Laurent Pinchart
> 
> _______________________________________________
> libcamera-devel mailing list
> libcamera-devel at lists.libcamera.org
> https://lists.libcamera.org/listinfo/libcamera-devel

-- 
Regards,
Niklas Söderlund


More information about the libcamera-devel mailing list