[libcamera-devel] [PATCH v2 1/2] android: jpeg: Rectify variable naming style.
Jacopo Mondi
jacopo at jmondi.org
Mon Sep 13 15:55:35 CEST 2021
Hi Umang,
On Mon, Sep 13, 2021 at 09:31:09AM +0530, Umang Jain wrote:
> Rectify variable renaming style for YPlaneSize, UVPlaneSize.
> libcamera uses camelCase where first letter should be in lower case.
>
> Fixes: e355ca0087cd9("android: jpeg: Split and pass the thumbnail planes to encoder")
> Signed-off-by: Umang Jain <umang.jain at ideasonboard.com>
> Reviewed-by: Hirokazu Honda <hiroh at chromium.org>
> Reviewed-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com>
Reviewed-by: Jacopo Mondi <jacopo at jmondi.org>
Thanks
j
> ---
> src/android/jpeg/post_processor_jpeg.cpp | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/src/android/jpeg/post_processor_jpeg.cpp b/src/android/jpeg/post_processor_jpeg.cpp
> index ef2d98cc..f6d47f63 100644
> --- a/src/android/jpeg/post_processor_jpeg.cpp
> +++ b/src/android/jpeg/post_processor_jpeg.cpp
> @@ -82,10 +82,10 @@ void PostProcessorJpeg::generateThumbnail(const FrameBuffer &source,
> */
> std::vector<Span<uint8_t>> thumbnailPlanes;
> const PixelFormatInfo &formatNV12 = PixelFormatInfo::info(formats::NV12);
> - size_t YPlaneSize = formatNV12.planeSize(targetSize, 0);
> - size_t UVPlaneSize = formatNV12.planeSize(targetSize, 1);
> - thumbnailPlanes.push_back({ rawThumbnail.data(), YPlaneSize });
> - thumbnailPlanes.push_back({ rawThumbnail.data() + YPlaneSize, UVPlaneSize });
> + size_t yPlaneSize = formatNV12.planeSize(targetSize, 0);
> + size_t uvPlaneSize = formatNV12.planeSize(targetSize, 1);
> + thumbnailPlanes.push_back({ rawThumbnail.data(), yPlaneSize });
> + thumbnailPlanes.push_back({ rawThumbnail.data() + yPlaneSize, uvPlaneSize });
>
> int jpeg_size = thumbnailEncoder_.encode(thumbnailPlanes,
> *thumbnail, {}, quality);
> --
> 2.31.1
>
More information about the libcamera-devel
mailing list