<div dir="auto">Hi Paul,<div dir="auto">Thanks for the feedback.</div><div dir="auto"><br></div><div dir="auto">Can you please make the changes before merging ? or should I sent a new patch.</div><div dir="auto"><br></div><div dir="auto">Regards,</div><div dir="auto">Vedant Paranjape</div><div dir="auto"><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, 27 Sep, 2021, 12:07 , <<a href="mailto:paul.elder@ideasonboard.com">paul.elder@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>
The subject can be slightly more concise:<br>
<br>
gstreamer: Fix spelling of the word manager in an util function<br>
<br>
On Fri, Sep 24, 2021 at 03:29:37PM +0530, Vedant Paranjape wrote:<br>
> Fix all name in all instances of the function gst_libcamera_get_camera_mananger<br>
> to gst_libcamera_get_camera_manager. Spelling of manager was incorrect.<br>
<br>
This can be more concise too:<br>
<br>
Fix spelling of the function gst_libcamera_get_camera_mananger.<br>
<br>
<br>
Manipulating information density in english writing is yet another skill.<br>
<br>
> <br>
> This patch has no functional changes.<br>
> <br>
> Signed-off-by: Vedant Paranjape <<a href="mailto:vedantparanjape160201@gmail.com" target="_blank" rel="noreferrer">vedantparanjape160201@gmail.com</a>><br>
<br>
Otherwise, looks good.<br>
<br>
<br>
Reviewed-by: Paul Elder <<a href="mailto:paul.elder@ideasonboard.com" target="_blank" rel="noreferrer">paul.elder@ideasonboard.com</a>><br>
<br>
> ---<br>
>  src/gstreamer/gstlibcamera-utils.cpp   | 2 +-<br>
>  src/gstreamer/gstlibcamera-utils.h     | 2 +-<br>
>  src/gstreamer/gstlibcameraprovider.cpp | 2 +-<br>
>  src/gstreamer/gstlibcamerasrc.cpp      | 2 +-<br>
>  4 files changed, 4 insertions(+), 4 deletions(-)<br>
> <br>
> diff --git a/src/gstreamer/gstlibcamera-utils.cpp b/src/gstreamer/gstlibcamera-utils.cpp<br>
> index cef5b03fe204..3f2422863c03 100644<br>
> --- a/src/gstreamer/gstlibcamera-utils.cpp<br>
> +++ b/src/gstreamer/gstlibcamera-utils.cpp<br>
> @@ -239,7 +239,7 @@ G_LOCK_DEFINE_STATIC(cm_singleton_lock);<br>
>  static std::weak_ptr<CameraManager> cm_singleton_ptr;<br>
>  <br>
>  std::shared_ptr<CameraManager><br>
> -gst_libcamera_get_camera_mananger(int &ret)<br>
> +gst_libcamera_get_camera_manager(int &ret)<br>
>  {<br>
>       std::shared_ptr<CameraManager> cm;<br>
>  <br>
> diff --git a/src/gstreamer/gstlibcamera-utils.h b/src/gstreamer/gstlibcamera-utils.h<br>
> index 67a06db3d28a..7087fef0a94d 100644<br>
> --- a/src/gstreamer/gstlibcamera-utils.h<br>
> +++ b/src/gstreamer/gstlibcamera-utils.h<br>
> @@ -20,7 +20,7 @@ GstCaps *gst_libcamera_stream_configuration_to_caps(const libcamera::StreamConfi<br>
>  void gst_libcamera_configure_stream_from_caps(libcamera::StreamConfiguration &stream_cfg,<br>
>                                             GstCaps *caps);<br>
>  void gst_libcamera_resume_task(GstTask *task);<br>
> -std::shared_ptr<libcamera::CameraManager> gst_libcamera_get_camera_mananger(int &ret);<br>
> +std::shared_ptr<libcamera::CameraManager> gst_libcamera_get_camera_manager(int &ret);<br>
>  <br>
>  /**<br>
>   * \class GLibLocker<br>
> diff --git a/src/gstreamer/gstlibcameraprovider.cpp b/src/gstreamer/gstlibcameraprovider.cpp<br>
> index 948ba0d14037..aee6f9a2be2b 100644<br>
> --- a/src/gstreamer/gstlibcameraprovider.cpp<br>
> +++ b/src/gstreamer/gstlibcameraprovider.cpp<br>
> @@ -180,7 +180,7 @@ gst_libcamera_provider_probe(GstDeviceProvider *provider)<br>
>        * gains monitoring support. Meanwhile we need to cycle start()/stop()<br>
>        * to ensure every probe() calls return the latest list.<br>
>        */<br>
> -     cm = gst_libcamera_get_camera_mananger(ret);<br>
> +     cm = gst_libcamera_get_camera_manager(ret);<br>
>       if (ret) {<br>
>               GST_ERROR_OBJECT(self, "Failed to retrieve device list: %s",<br>
>                                g_strerror(-ret));<br>
> diff --git a/src/gstreamer/gstlibcamerasrc.cpp b/src/gstreamer/gstlibcamerasrc.cpp<br>
> index 15409106eac2..b545de4c4262 100644<br>
> --- a/src/gstreamer/gstlibcamerasrc.cpp<br>
> +++ b/src/gstreamer/gstlibcamerasrc.cpp<br>
> @@ -204,7 +204,7 @@ gst_libcamera_src_open(GstLibcameraSrc *self)<br>
>  <br>
>       GST_DEBUG_OBJECT(self, "Opening camera device ...");<br>
>  <br>
> -     cm = gst_libcamera_get_camera_mananger(ret);<br>
> +     cm = gst_libcamera_get_camera_manager(ret);<br>
>       if (ret) {<br>
>               GST_ELEMENT_ERROR(self, LIBRARY, INIT,<br>
>                                 ("Failed listing cameras."),<br>
> -- <br>
> 2.25.1<br>
> <br>
</blockquote></div>