[PATCH] qcam: Drop Qt version checks

Neal Gompa neal at gompa.dev
Wed Sep 11 15:47:16 CEST 2024


On Wed, Sep 11, 2024 at 1:44 AM Laurent Pinchart
<laurent.pinchart at ideasonboard.com> wrote:
>
> The Qt version checks to support different minor Qt5 versions are not
> needed anymore, now that we switched to Qt6. Drop them.
>
> Signed-off-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com>
> ---
>  src/apps/qcam/main_window.cpp   | 10 ----------
>  src/apps/qcam/viewfinder_qt.cpp |  4 ----
>  2 files changed, 14 deletions(-)
>
> diff --git a/src/apps/qcam/main_window.cpp b/src/apps/qcam/main_window.cpp
> index d515beeda275..dd2aa19618a1 100644
> --- a/src/apps/qcam/main_window.cpp
> +++ b/src/apps/qcam/main_window.cpp
> @@ -37,16 +37,6 @@
>
>  using namespace libcamera;
>
> -#if QT_VERSION < QT_VERSION_CHECK(5, 14, 0)
> -/*
> - * Qt::fixed was introduced in v5.14, and ::fixed deprecated in v5.15. Allow
> - * usage of Qt::fixed unconditionally.
> - */
> -namespace Qt {
> -constexpr auto fixed = ::fixed;
> -} /* namespace Qt */
> -#endif
> -
>  /**
>   * \brief Custom QEvent to signal capture completion
>   */
> diff --git a/src/apps/qcam/viewfinder_qt.cpp b/src/apps/qcam/viewfinder_qt.cpp
> index b0d63f3fa5ea..e7bd8af941f3 100644
> --- a/src/apps/qcam/viewfinder_qt.cpp
> +++ b/src/apps/qcam/viewfinder_qt.cpp
> @@ -27,15 +27,11 @@
>
>  static const QMap<libcamera::PixelFormat, QImage::Format> nativeFormats
>  {
> -#if QT_VERSION >= QT_VERSION_CHECK(5, 2, 0)
>         { libcamera::formats::ABGR8888, QImage::Format_RGBX8888 },
>         { libcamera::formats::XBGR8888, QImage::Format_RGBX8888 },
> -#endif
>         { libcamera::formats::ARGB8888, QImage::Format_RGB32 },
>         { libcamera::formats::XRGB8888, QImage::Format_RGB32 },
> -#if QT_VERSION >= QT_VERSION_CHECK(5, 14, 0)
>         { libcamera::formats::RGB888, QImage::Format_BGR888 },
> -#endif
>         { libcamera::formats::BGR888, QImage::Format_RGB888 },
>         { libcamera::formats::RGB565, QImage::Format_RGB16 },
>  };
>
> base-commit: f75b8dd26feaca86701704390dea18c71e2f0350

Reviewed-by: Neal Gompa <neal at gompa.dev>


-- 
真実はいつも一つ!/ Always, there's only one truth!


More information about the libcamera-devel mailing list