<div dir="ltr"><div>Hello,</div><div><br></div><div>I'm fine with your suggestion to reuse the cover later as message.<br></div></div><br><div class="gmail_quote gmail_quote_container"><div dir="ltr" class="gmail_attr">On Sun, Feb 16, 2025 at 7:45 PM Laurent Pinchart <<a href="mailto:laurent.pinchart@ideasonboard.com">laurent.pinchart@ideasonboard.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi Nerijus,<br>
<br>
Thank you for the patch, and welcome to libcamera.<br>
<br>
On Sun, Feb 16, 2025 at 09:21:31AM +0200, Nerijus Bendžiūnas wrote:<br>
<br>
A commit message would be nice. The explanation you provided in the<br>
cover letter would be a good fit:<br>
<br>
Fix a copy/paste/replace typo. Without this fix, the last element (4th)<br>
is always zero.<br>
<br>
> Signed-off-by: Nerijus Bendžiūnas <<a href="mailto:nerijus.bendziunas@gmail.com" target="_blank">nerijus.bendziunas@gmail.com</a>><br>
<br>
Reviewed-by: Laurent Pinchart <<a href="mailto:laurent.pinchart@ideasonboard.com" target="_blank">laurent.pinchart@ideasonboard.com</a>><br>
<br>
If you're fine with the updated commit message there's no need to<br>
resend, I can update the patch when applying.<br>
<br>
> ---<br>
>  src/gstreamer/<a href="http://gstlibcamera-controls.cpp.in" rel="noreferrer" target="_blank">gstlibcamera-controls.cpp.in</a> | 2 +-<br>
>  1 file changed, 1 insertion(+), 1 deletion(-)<br>
> <br>
> diff --git a/src/gstreamer/<a href="http://gstlibcamera-controls.cpp.in" rel="noreferrer" target="_blank">gstlibcamera-controls.cpp.in</a> b/src/gstreamer/<a href="http://gstlibcamera-controls.cpp.in" rel="noreferrer" target="_blank">gstlibcamera-controls.cpp.in</a><br>
> index ace36b71..d937b19e 100644<br>
> --- a/src/gstreamer/<a href="http://gstlibcamera-controls.cpp.in" rel="noreferrer" target="_blank">gstlibcamera-controls.cpp.in</a><br>
> +++ b/src/gstreamer/<a href="http://gstlibcamera-controls.cpp.in" rel="noreferrer" target="_blank">gstlibcamera-controls.cpp.in</a><br>
> @@ -39,7 +39,7 @@ static void value_set_rectangle(GValue *value, const Rectangle &rect)<br>
> <br>
>       GValue height = G_VALUE_INIT;<br>
>       g_value_init(&height, G_TYPE_INT);<br>
> -     g_value_set_int(&x, size.height);<br>
> +     g_value_set_int(&height, size.height);<br>
>       gst_value_array_append_and_take_value(value, &height);<br>
>  }<br>
> <br>
<br>
-- <br>
Regards,<br>
<br>
Laurent Pinchart<br>
</blockquote></div><div><br clear="all"></div><br><span class="gmail_signature_prefix">-- </span><br><div dir="ltr" class="gmail_signature">Nerijus</div>