| Message ID | 20251025142300.64480-2-uajain@igalia.com |
|---|---|
| State | Superseded |
| Headers | show |
| Series |
|
| Related | show |
Hi, Le samedi 25 octobre 2025 à 15:22 +0100, Umang Jain a écrit : > Shorten the following utility function name: > gst_libcamera_stream_configuration_to_caps() > to > gst_libcamera_stream_cfg_to_caps() > > Comestic fix only for slight readability enhancement. > > Signed-off-by: Umang Jain <uajain@igalia.com> > Reviewed-by: Barnabás Pőcze <barnabas.pocze@ideasonboard.com> > Reviewed-by: Nicolas Dufresne <nicolas.dufresne@collabora.com> Didn't Laurent voted to remove this one ? Please agree with him first, my preference was also aligned to Laurent on this one. Nicolas > --- > src/gstreamer/gstlibcamera-utils.cpp | 4 ++-- > src/gstreamer/gstlibcamera-utils.h | 4 ++-- > src/gstreamer/gstlibcamerasrc.cpp | 4 ++-- > 3 files changed, 6 insertions(+), 6 deletions(-) > > diff --git a/src/gstreamer/gstlibcamera-utils.cpp > b/src/gstreamer/gstlibcamera-utils.cpp > index bfb094c9..09f274b5 100644 > --- a/src/gstreamer/gstlibcamera-utils.cpp > +++ b/src/gstreamer/gstlibcamera-utils.cpp > @@ -408,8 +408,8 @@ gst_libcamera_stream_formats_to_caps(const StreamFormats > &formats) > } > > GstCaps * > -gst_libcamera_stream_configuration_to_caps(const StreamConfiguration > &stream_cfg, > - GstVideoTransferFunction transfer) > +gst_libcamera_stream_cfg_to_caps(const StreamConfiguration &stream_cfg, > + GstVideoTransferFunction transfer) > { > GstCaps *caps = gst_caps_new_empty(); > GstStructure *s = bare_structure_from_format(stream_cfg.pixelFormat); > diff --git a/src/gstreamer/gstlibcamera-utils.h b/src/gstreamer/gstlibcamera- > utils.h > index 35df56fb..06ea9289 100644 > --- a/src/gstreamer/gstlibcamera-utils.h > +++ b/src/gstreamer/gstlibcamera-utils.h > @@ -16,8 +16,8 @@ > #include <gst/video/video.h> > > GstCaps *gst_libcamera_stream_formats_to_caps(const libcamera::StreamFormats > &formats); > -GstCaps *gst_libcamera_stream_configuration_to_caps(const > libcamera::StreamConfiguration &stream_cfg, > - GstVideoTransferFunction > transfer); > +GstCaps *gst_libcamera_stream_cfg_to_caps(const > libcamera::StreamConfiguration &stream_cfg, > + GstVideoTransferFunction transfer); > void gst_libcamera_configure_stream_from_caps(libcamera::StreamConfiguration > &stream_cfg, > GstCaps *caps, > GstVideoTransferFunction *transfer); > void gst_libcamera_get_framerate_from_caps(GstCaps *caps, GstStructure > *element_caps); > diff --git a/src/gstreamer/gstlibcamerasrc.cpp > b/src/gstreamer/gstlibcamerasrc.cpp > index 011a12fc..a1556cc5 100644 > --- a/src/gstreamer/gstlibcamerasrc.cpp > +++ b/src/gstreamer/gstlibcamerasrc.cpp > @@ -639,7 +639,7 @@ gst_libcamera_src_negotiate(GstLibcameraSrc *self) > GstPad *srcpad = state->srcpads_[i]; > const StreamConfiguration &stream_cfg = state->config_- > >at(i); > > - g_autoptr(GstCaps) caps = > gst_libcamera_stream_configuration_to_caps(stream_cfg, transfer[i]); > + g_autoptr(GstCaps) caps = > gst_libcamera_stream_cfg_to_caps(stream_cfg, transfer[i]); > gst_libcamera_framerate_to_caps(caps, element_caps); > > if (status == CameraConfiguration::Adjusted && > @@ -667,7 +667,7 @@ gst_libcamera_src_negotiate(GstLibcameraSrc *self) > GstBufferPool *video_pool = nullptr; > GstVideoInfo info; > > - g_autoptr(GstCaps) caps = > gst_libcamera_stream_configuration_to_caps(stream_cfg, transfer[i]); > + g_autoptr(GstCaps) caps = > gst_libcamera_stream_cfg_to_caps(stream_cfg, transfer[i]); > > gst_video_info_from_caps(&info, caps); > gst_libcamera_pad_set_video_info(srcpad, &info);
diff --git a/src/gstreamer/gstlibcamera-utils.cpp b/src/gstreamer/gstlibcamera-utils.cpp index bfb094c9..09f274b5 100644 --- a/src/gstreamer/gstlibcamera-utils.cpp +++ b/src/gstreamer/gstlibcamera-utils.cpp @@ -408,8 +408,8 @@ gst_libcamera_stream_formats_to_caps(const StreamFormats &formats) } GstCaps * -gst_libcamera_stream_configuration_to_caps(const StreamConfiguration &stream_cfg, - GstVideoTransferFunction transfer) +gst_libcamera_stream_cfg_to_caps(const StreamConfiguration &stream_cfg, + GstVideoTransferFunction transfer) { GstCaps *caps = gst_caps_new_empty(); GstStructure *s = bare_structure_from_format(stream_cfg.pixelFormat); diff --git a/src/gstreamer/gstlibcamera-utils.h b/src/gstreamer/gstlibcamera-utils.h index 35df56fb..06ea9289 100644 --- a/src/gstreamer/gstlibcamera-utils.h +++ b/src/gstreamer/gstlibcamera-utils.h @@ -16,8 +16,8 @@ #include <gst/video/video.h> GstCaps *gst_libcamera_stream_formats_to_caps(const libcamera::StreamFormats &formats); -GstCaps *gst_libcamera_stream_configuration_to_caps(const libcamera::StreamConfiguration &stream_cfg, - GstVideoTransferFunction transfer); +GstCaps *gst_libcamera_stream_cfg_to_caps(const libcamera::StreamConfiguration &stream_cfg, + GstVideoTransferFunction transfer); void gst_libcamera_configure_stream_from_caps(libcamera::StreamConfiguration &stream_cfg, GstCaps *caps, GstVideoTransferFunction *transfer); void gst_libcamera_get_framerate_from_caps(GstCaps *caps, GstStructure *element_caps); diff --git a/src/gstreamer/gstlibcamerasrc.cpp b/src/gstreamer/gstlibcamerasrc.cpp index 011a12fc..a1556cc5 100644 --- a/src/gstreamer/gstlibcamerasrc.cpp +++ b/src/gstreamer/gstlibcamerasrc.cpp @@ -639,7 +639,7 @@ gst_libcamera_src_negotiate(GstLibcameraSrc *self) GstPad *srcpad = state->srcpads_[i]; const StreamConfiguration &stream_cfg = state->config_->at(i); - g_autoptr(GstCaps) caps = gst_libcamera_stream_configuration_to_caps(stream_cfg, transfer[i]); + g_autoptr(GstCaps) caps = gst_libcamera_stream_cfg_to_caps(stream_cfg, transfer[i]); gst_libcamera_framerate_to_caps(caps, element_caps); if (status == CameraConfiguration::Adjusted && @@ -667,7 +667,7 @@ gst_libcamera_src_negotiate(GstLibcameraSrc *self) GstBufferPool *video_pool = nullptr; GstVideoInfo info; - g_autoptr(GstCaps) caps = gst_libcamera_stream_configuration_to_caps(stream_cfg, transfer[i]); + g_autoptr(GstCaps) caps = gst_libcamera_stream_cfg_to_caps(stream_cfg, transfer[i]); gst_video_info_from_caps(&info, caps); gst_libcamera_pad_set_video_info(srcpad, &info);