fix(video): fix invalid VideoSource ID allocation

This commit fixes the an invalid ID allocation by VideoSource, before
this commit all VideoSources receives the same ID causing VideoFrame to
believe all frames belong to the same video source.

Additionally, this commit addresses issues with deprecated pixel
formats and silences libswscale warnings.

src/video/videoframe.cpp

@@ -97,6 +97,46 @@ VideoFrame::VideoFrame(IDType sourceID, AVFrame* sourceFrame, QRect dimensions,
       sourceFrameKey(getFrameKey(dimensions.size(), pixFmt, sourceFrame->linesize[0])),
       freeSourceFrame(freeSourceFrame)
 {
+
+    // We override the pixel format in the case a deprecated one is used
+    switch(pixFmt)
+    {
+    case AV_PIX_FMT_YUVJ420P:
+    {
+        pixFmt = AV_PIX_FMT_YUV420P;
+        sourceFrame->color_range = AVCOL_RANGE_MPEG;
+        break;
+    }
+
+    case AV_PIX_FMT_YUVJ411P:
+    {
+        pixFmt = AV_PIX_FMT_YUV411P;
+        sourceFrame->color_range = AVCOL_RANGE_MPEG;
+        break;
+    }
+
+    case AV_PIX_FMT_YUVJ422P:
+    {
+        pixFmt = AV_PIX_FMT_YUV422P;
+        sourceFrame->color_range = AVCOL_RANGE_MPEG;
+        break;
+    }
+
+    case AV_PIX_FMT_YUVJ444P:
+    {
+        pixFmt = AV_PIX_FMT_YUV444P;
+        sourceFrame->color_range = AVCOL_RANGE_MPEG;
+        break;
+    }
+
+    case AV_PIX_FMT_YUVJ440P:
+    {
+        pixFmt = AV_PIX_FMT_YUV440P;
+        sourceFrame->color_range = AVCOL_RANGE_MPEG;
+        break;
+    }
+    }
+
     frameBuffer[sourceFrameKey] = sourceFrame;
 }
 

src/video/videosource.h

@@ -38,7 +38,7 @@ class VideoSource : public QObject
     using AtomicIDType = std::atomic_uint_fast64_t;
 
 public:
-    VideoSource() : id(sourceIDs.fetch_add(std::memory_order_relaxed)){}
+    VideoSource() : id(sourceIDs++){}
 
     virtual ~VideoSource() = default;
     /**