Add support for RGB565

.github/workflows/Build.yml

@@ -30,6 +30,8 @@ jobs:
         make -j$(nproc)
     - name: Run Unittests
       run: $GITHUB_WORKSPACE/build/YUViewUnitTest/YUViewUnitTest
+      env:
+        QT_QPA_PLATFORM: 'offscreen'
   build-mac-native:
     runs-on: ${{ matrix.os }}
     strategy:
@@ -51,6 +53,8 @@ jobs:
         make -j $(sysctl -n hw.logicalcpu)
     - name: Run Unittests
       run: $GITHUB_WORKSPACE/build/YUViewUnitTest/YUViewUnitTest
+      env:
+        QT_QPA_PLATFORM: 'offscreen'
   build-linux-mac:
     runs-on: ${{ matrix.os }}
     strategy:
@@ -105,6 +109,8 @@ jobs:
         make -j 4
     - name: Run Unittests
       run: $GITHUB_WORKSPACE/build/YUViewUnitTest/YUViewUnitTest
+      env:
+        QT_QPA_PLATFORM: 'offscreen'
     - name: Build App (Mac)
       if: runner.os	== 'macOS'
       run: |
@@ -153,6 +159,8 @@ jobs:
         curl -L https://github.com/ChristianFeldmann/YUViewQt/releases/download/QtBase-6.9.0/qtBase-6-9-0-windows-2022.zip -o Qt.zip
         7z x  Qt.zip
         echo "${{ github.workspace }}\..\..\YUViewQt\YUViewQt\Qt\bin" | Out-File -FilePath $env:GITHUB_PATH -Encoding utf8 -Append
+    - name: Test qmake
+      run : qmake --version
     - name: Install libde265
       run: |
         curl -L https://github.com/ChristianFeldmann/libde265/releases/download/v1.1/libde265.dll -o libde265.dll
@@ -176,20 +184,20 @@ jobs:
         echo "Creating Build dir and entering it"
         mkdir build
         cd build
-        echo "Qmake Version:"
-        d:\a\YUViewQt\YUViewQt\Qt\bin\qmake --version
         echo "Executing qmake..."
-        d:\a\YUViewQt\YUViewQt\Qt\bin\qmake CONFIG+=UNITTESTS ..
+        qmake CONFIG+=UNITTESTS ..
         echo "Executing jom:"
         jom
     - name: Run Unittests
-      run: D:\a\YUView\YUView\build\YUViewUnitTest\YUViewUnitTest
+      run: ${{ github.workspace }}\build\YUViewUnitTest\YUViewUnitTest
+      env:
+        QT_QPA_PLATFORM: 'offscreen'
     - name: WindeployQT
       run: |
         mkdir deploy
         cd deploy
         cp ../build/YUViewApp/YUView.exe .
-        d:\a\YUViewQt\YUViewQt\Qt\bin\windeployqt.exe --release --no-compiler-runtime YUView.exe
+        windeployqt --release --no-compiler-runtime YUView.exe
         cp ../openSSL/*.dll .
         mkdir decoder
         cp ..\libde265.dll decoder

HACKING.md

@@ -1,7 +1,7 @@
 # Compiler Requirements
 
 MSVC: Visual Studio 2013 Update 5 or newer.
-gcc/llvm: C++17 support is required.
+gcc/llvm: C++20 support is required.
 
 # Coding Conventions
 

YUViewApp/YUViewApp.pro

@@ -2,7 +2,7 @@ QT += core gui widgets opengl xml concurrent network
 
 TARGET = YUView
 TEMPLATE = app
-CONFIG += c++17
+CONFIG += c++20
 CONFIG -= debug_and_release
 
 SOURCES += $$files(src/*.cpp, false)

YUViewLib/YUViewLib.pro

@@ -2,7 +2,7 @@ QT += core gui widgets opengl xml concurrent network
 
 TEMPLATE = lib
 CONFIG += staticlib
-CONFIG += c++17
+CONFIG += c++20
 CONFIG -= debug_and_release
 CONFIG += object_parallel_to_source
 

YUViewLib/src/common/Functions.cpp

@@ -198,6 +198,32 @@ std::string toLower(const std::string_view str)
   return lowercaseStr;
 }
 
+std::vector<std::string_view> splitString(const std::string_view str, const char delimiter)
+{
+  std::vector<std::string_view> result;
+  size_t                        start = 0;
+  size_t                        end   = str.find(delimiter);
+  while (end != std::string_view::npos)
+  {
+    result.emplace_back(str.substr(start, end - start));
+    start = end + 1;
+    end   = str.find(delimiter, start);
+  }
+  if (start != str.size())
+    result.emplace_back(str.substr(start));
+  return result;
+}
+
+std::string_view stripWhitespace(std::string_view str)
+{
+  str.remove_prefix(std::min(str.find_first_not_of(" "), str.size()));
+
+  const auto lastNonWhitespace = str.find_last_not_of(" ");
+  if (lastNonWhitespace != std::string_view::npos)
+    str.remove_suffix(str.size() - lastNonWhitespace - 1);
+  return str;
+}
+
 ByteVector readData(std::istream &istream, const size_t nrBytes)
 {
   ByteVector data;
@@ -211,11 +237,12 @@ ByteVector readData(std::istream &istream, const size_t nrBytes)
 std::optional<unsigned> toUnsigned(const std::string_view text)
 {
   unsigned   value{};
-  const auto result = std::from_chars(text.data(), text.data() + text.size(), value);
+  const auto endPointer = text.data() + text.size();
+  const auto result     = std::from_chars(text.data(), endPointer, value);
 
   if (result.ec != std::errc())
     return {};
-  const auto allCharactersParsed = (result.ptr == &(*text.end()));
+  const auto allCharactersParsed = (result.ptr == endPointer);
   if (!allCharactersParsed)
     return {};
 
@@ -225,11 +252,12 @@ std::optional<unsigned> toUnsigned(const std::string_view text)
 std::optional<int> toInt(const std::string_view text)
 {
   int        value{};
-  const auto result = std::from_chars(text.data(), text.data() + text.size(), value);
+  const auto endPointer = text.data() + text.size();
+  const auto result     = std::from_chars(text.data(), endPointer, value);
 
   if (result.ec != std::errc())
     return {};
-  const auto allCharactersParsed = (result.ptr == &(*text.end()));
+  const auto allCharactersParsed = (result.ptr == endPointer);
   if (!allCharactersParsed)
     return {};
 

YUViewLib/src/common/Functions.h

@@ -36,6 +36,8 @@
 
 #include <istream>
 #include <optional>
+#include <string_view>
+#include <vector>
 
 namespace functions
 {
@@ -77,8 +79,12 @@ template <size_t N> QStringList toQStringList(const std::array<std::string_view,
   return list;
 }
 
-std::string toLower(const std::string_view str);
-ByteVector  readData(std::istream &istream, const size_t nrBytes);
+std::string                   toLower(const std::string_view str);
+std::optional<int>            toInt(const std::string_view str);
+std::vector<std::string_view> splitString(const std::string_view str, const char delimiter);
+std::string_view              stripWhitespace(std::string_view str);
+
+ByteVector readData(std::istream &istream, const size_t nrBytes);
 
 template <typename T> unsigned clipToUnsigned(T val)
 {

YUViewLib/src/common/FunctionsGui.cpp

@@ -108,40 +108,6 @@ QIcon functionsGui::convertIcon(QString iconPath)
   return outIcon;
 }
 
-QPixmap functionsGui::convertPixmap(QString pixmapPath)
-{
-  QSettings settings;
-  QString   themeName = settings.value("Theme", "Default").toString();
-
-  // Get the active and inactive colors
-  QStringList colors = functions::getThemeColors(themeName);
-  QRgb        activeColor;
-  if (colors.size() == 4)
-  {
-    QColor active(colors[1]);
-    activeColor = active.rgb();
-  }
-  else
-    activeColor = qRgb(0, 0, 0);
-
-  QImage input(pixmapPath);
-
-  QImage active(input.size(), input.format());
-  for (int y = 0; y < input.height(); y++)
-  {
-    for (int x = 0; x < input.width(); x++)
-    {
-      QRgb in = input.pixel(x, y);
-      if (qAlpha(in) != 0)
-        active.setPixel(x, y, activeColor);
-      else
-        active.setPixel(x, y, in);
-    }
-  }
-
-  return QPixmap::fromImage(active);
-}
-
 QString functionsGui::pixelFormatToString(QImage::Format f)
 {
   if (f == QImage::Format_Invalid)

YUViewLib/src/common/FunctionsGui.h

@@ -108,6 +108,5 @@ void setupUi(void *ui, void (*setupUi)(void *ui, QWidget *widget));
 
 // Return the icon/pixmap from the given file path (inverted if necessary)
 QIcon   convertIcon(QString iconPath);
-QPixmap convertPixmap(QString pixmapPath);
 
 } // namespace functionsGui

YUViewLib/src/common/Typedef.h

@@ -256,50 +256,3 @@ enum recacheIndicator
                  // useless in the cache.
 };
 Q_DECLARE_METATYPE(recacheIndicator)
-
-#if QT_VERSION <= 0x050700
-// copied from newer version of qglobal.h
-template <typename... Args> struct QNonConstOverload
-{
-  template <typename R, typename T>
-  Q_DECL_CONSTEXPR auto operator()(R (T::*ptr)(Args...)) const Q_DECL_NOTHROW->decltype(ptr)
-  {
-    return ptr;
-  }
-  template <typename R, typename T>
-  static Q_DECL_CONSTEXPR auto of(R (T::*ptr)(Args...)) Q_DECL_NOTHROW -> decltype(ptr)
-  {
-    return ptr;
-  }
-};
-template <typename... Args> struct QConstOverload
-{
-  template <typename R, typename T>
-  Q_DECL_CONSTEXPR auto operator()(R (T::*ptr)(Args...) const) const Q_DECL_NOTHROW->decltype(ptr)
-  {
-    return ptr;
-  }
-  template <typename R, typename T>
-  static Q_DECL_CONSTEXPR auto of(R (T::*ptr)(Args...) const) Q_DECL_NOTHROW -> decltype(ptr)
-  {
-    return ptr;
-  }
-};
-template <typename... Args> struct QOverload : QConstOverload<Args...>, QNonConstOverload<Args...>
-{
-  using QConstOverload<Args...>::of;
-  using QConstOverload<Args...>::operator();
-  using QNonConstOverload<Args...>::of;
-  using QNonConstOverload<Args...>::operator();
-  template <typename R>
-  Q_DECL_CONSTEXPR auto operator()(R (*ptr)(Args...)) const Q_DECL_NOTHROW->decltype(ptr)
-  {
-    return ptr;
-  }
-  template <typename R>
-  static Q_DECL_CONSTEXPR auto of(R (*ptr)(Args...)) Q_DECL_NOTHROW -> decltype(ptr)
-  {
-    return ptr;
-  }
-};
-#endif

YUViewLib/src/common/TypedefQtDeprecated.h

@@ -0,0 +1,42 @@
+/*  This file is part of YUView - The YUV player with advanced analytics toolset
+ *   <https://github.com/IENT/YUView>
+ *   Copyright (C) 2015  Institut für Nachrichtentechnik, RWTH Aachen University, GERMANY
+ *
+ *   This program is free software; you can redistribute it and/or modify
+ *   it under the terms of the GNU General Public License as published by
+ *   the Free Software Foundation; either version 3 of the License, or
+ *   (at your option) any later version.
+ *
+ *   In addition, as a special exception, the copyright holders give
+ *   permission to link the code of portions of this program with the
+ *   OpenSSL library under certain conditions as described in each
+ *   individual source file, and distribute linked combinations including
+ *   the two.
+ *
+ *   You must obey the GNU General Public License in all respects for all
+ *   of the code used other than OpenSSL. If you modify file(s) with this
+ *   exception, you may extend this exception to your version of the
+ *   file(s), but you are not obligated to do so. If you do not wish to do
+ *   so, delete this exception statement from your version. If you delete
+ *   this exception statement from all source files in the program, then
+ *   also delete it here.
+ *
+ *   This program is distributed in the hope that it will be useful,
+ *   but WITHOUT ANY WARRANTY; without even the implied warranty of
+ *   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ *   GNU General Public License for more details.
+ *
+ *   You should have received a copy of the GNU General Public License
+ *   along with this program. If not, see <http://www.gnu.org/licenses/>.
+ */
+
+ #pragma once
+
+#include <QCheckBox>
+
+#if QT_VERSION >= QT_VERSION_CHECK(6, 7, 0)
+constexpr auto QCheckBoxStateChanged = &QCheckBox::checkStateChanged;
+#else
+constexpr auto QCheckBoxStateChanged = &QCheckBox::stateChanged;
+#endif
+