<div dir="auto">Thanks for your reply. I'll update the commit message<div dir="auto"><br></div><div dir="auto">Regards,</div><div dir="auto">Vedant</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, 28 Jul, 2021, 12:33 Laurent Pinchart, <<a href="mailto:laurent.pinchart@ideasonboard.com">laurent.pinchart@ideasonboard.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Vedant,<br>
<br>
Thank you for the patch.<br>
<br>
On Wed, Jul 28, 2021 at 01:17:24AM +0530, Vedant Paranjape wrote:<br>
> Change format specifier %s to %u in request pad template<br>
<br>
This is pretty obvious from the patch itself. What isn't obvious,<br>
however, is why, and that's what a commit message body should focus on.<br>
Every commit message needs to answer the "why" question, may explain<br>
"how" when not immediately visible, and could also explain "what" if<br>
it's not obvious.<br>
<br>
> Signed-off-by: Vedant Paranjape <<a href="mailto:vedantparanjape160201@gmail.com" target="_blank" rel="noreferrer">vedantparanjape160201@gmail.com</a>><br>
> ---<br>
>  src/gstreamer/gstlibcamerasrc.cpp | 2 +-<br>
>  1 file changed, 1 insertion(+), 1 deletion(-)<br>
> <br>
> diff --git a/src/gstreamer/gstlibcamerasrc.cpp b/src/gstreamer/gstlibcamerasrc.cpp<br>
> index bb8ea07a..553fad02 100644<br>
> --- a/src/gstreamer/gstlibcamerasrc.cpp<br>
> +++ b/src/gstreamer/gstlibcamerasrc.cpp<br>
> @@ -149,7 +149,7 @@ GstStaticPadTemplate src_template = {<br>
>  <br>
>  /* More pads can be requested in state < PAUSED */<br>
>  GstStaticPadTemplate request_src_template = {<br>
> -     "src_%s", GST_PAD_SRC, GST_PAD_REQUEST, TEMPLATE_CAPS<br>
> +     "src_%u", GST_PAD_SRC, GST_PAD_REQUEST, TEMPLATE_CAPS<br>
>  };<br>
>  <br>
>  void<br>
<br>
-- <br>
Regards,<br>
<br>
Laurent Pinchart<br>
</blockquote></div>