From e75fb15393e9a0343e457aa25e4bd23e7c56d083 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Adrien=20B=C3=A9raud?= <adrien.beraud@savoirfairelinux.com>
Date: Mon, 4 Feb 2019 10:13:06 -0500
Subject: [PATCH] videogenerator: use AVPixelFormat

Change-Id: I6989c6708ccfab2a2bcdc274c353e3efdc1ffc8c
---
 src/media/media_decoder.cpp              | 2 +-
 src/media/media_decoder.h                | 2 +-
 src/media/video/video_base.h             | 2 +-
 src/media/video/video_input.cpp          | 8 ++++----
 src/media/video/video_input.h            | 2 +-
 src/media/video/video_mixer.cpp          | 2 +-
 src/media/video/video_mixer.h            | 2 +-
 src/media/video/video_receive_thread.cpp | 2 +-
 src/media/video/video_receive_thread.h   | 2 +-
 9 files changed, 12 insertions(+), 12 deletions(-)

diff --git a/src/media/media_decoder.cpp b/src/media/media_decoder.cpp
index 90eee43950..4c68aaee16 100644
--- a/src/media/media_decoder.cpp
+++ b/src/media/media_decoder.cpp
@@ -449,7 +449,7 @@ MediaDecoder::getTimeBase() const
             (unsigned)avStream_->time_base.den};
 }
 
-int MediaDecoder::getPixelFormat() const
+AVPixelFormat MediaDecoder::getPixelFormat() const
 { return decoderCtx_->pix_fmt; }
 
 int
diff --git a/src/media/media_decoder.h b/src/media/media_decoder.h
index b5ccd577a9..e8893c319e 100644
--- a/src/media/media_decoder.h
+++ b/src/media/media_decoder.h
@@ -96,7 +96,7 @@ class MediaDecoder {
         std::string getDecoderName() const;
 
         rational<double> getFps() const;
-        int getPixelFormat() const;
+        AVPixelFormat getPixelFormat() const;
 
         void setOptions(const std::map<std::string, std::string>& options);
 #ifdef RING_ACCEL
diff --git a/src/media/video/video_base.h b/src/media/video/video_base.h
index 17063e0870..f9325e1433 100644
--- a/src/media/video/video_base.h
+++ b/src/media/video/video_base.h
@@ -72,7 +72,7 @@ public:
 
     virtual int getWidth() const = 0;
     virtual int getHeight() const = 0;
-    virtual int getPixelFormat() const = 0;
+    virtual AVPixelFormat getPixelFormat() const = 0;
 
     std::shared_ptr<VideoFrame> obtainLastFrame();
 
diff --git a/src/media/video/video_input.cpp b/src/media/video/video_input.cpp
index 9b76941524..168795c0dc 100644
--- a/src/media/video/video_input.cpp
+++ b/src/media/video/video_input.cpp
@@ -114,7 +114,7 @@ void VideoInput::process()
     for (auto& buffer : buffers_) {
         if (buffer.status == BUFFER_FULL && buffer.index == publish_index_) {
             auto& frame = getNewFrame();
-            int format = getPixelFormat();
+            AVPixelFormat format = getPixelFormat();
 
             buffer.status = BUFFER_PUBLISHED;
             frame.setFromMemory((uint8_t*)buffer.data, format, decOpts_.width, decOpts_.height,
@@ -593,14 +593,14 @@ int VideoInput::getWidth() const
 int VideoInput::getHeight() const
 { return decOpts_.height; }
 
-int VideoInput::getPixelFormat() const
+AVPixelFormat VideoInput::getPixelFormat() const
 {
     int format;
     std::stringstream ss;
     ss << decOpts_.format;
     ss >> format;
 
-    return format;
+    return (AVPixelFormat)format;
 }
 #else
 int VideoInput::getWidth() const
@@ -609,7 +609,7 @@ int VideoInput::getWidth() const
 int VideoInput::getHeight() const
 { return decoder_->getHeight(); }
 
-int VideoInput::getPixelFormat() const
+AVPixelFormat VideoInput::getPixelFormat() const
 { return decoder_->getPixelFormat(); }
 #endif
 
diff --git a/src/media/video/video_input.h b/src/media/video/video_input.h
index 906f0f6f3d..36a1c26ea0 100644
--- a/src/media/video/video_input.h
+++ b/src/media/video/video_input.h
@@ -76,7 +76,7 @@ public:
     // as VideoGenerator
     int getWidth() const;
     int getHeight() const;
-    int getPixelFormat() const;
+    AVPixelFormat getPixelFormat() const;
     DeviceParams getParams() const;
     MediaStream getInfo() const;
 
diff --git a/src/media/video/video_mixer.cpp b/src/media/video/video_mixer.cpp
index 84c5fa6ec3..1700fb20fc 100644
--- a/src/media/video/video_mixer.cpp
+++ b/src/media/video/video_mixer.cpp
@@ -229,7 +229,7 @@ int
 VideoMixer::getHeight() const
 { return height_; }
 
-int
+AVPixelFormat
 VideoMixer::getPixelFormat() const
 { return AV_PIX_FMT_YUYV422; }
 
diff --git a/src/media/video/video_mixer.h b/src/media/video/video_mixer.h
index 89881e5bdc..6f988a9874 100644
--- a/src/media/video/video_mixer.h
+++ b/src/media/video/video_mixer.h
@@ -46,7 +46,7 @@ public:
 
     int getWidth() const override;
     int getHeight() const override;
-    int getPixelFormat() const override;
+    AVPixelFormat getPixelFormat() const override;
 
     // as VideoFramePassiveReader
     void update(Observable<std::shared_ptr<MediaFrame>>* ob, const std::shared_ptr<MediaFrame>& v) override;
diff --git a/src/media/video/video_receive_thread.cpp b/src/media/video/video_receive_thread.cpp
index b4f6eb67c2..33336bf9eb 100644
--- a/src/media/video/video_receive_thread.cpp
+++ b/src/media/video/video_receive_thread.cpp
@@ -243,7 +243,7 @@ int VideoReceiveThread::getWidth() const
 int VideoReceiveThread::getHeight() const
 { return dstHeight_; }
 
-int VideoReceiveThread::getPixelFormat() const
+AVPixelFormat VideoReceiveThread::getPixelFormat() const
 { return videoDecoder_->getPixelFormat(); }
 
 MediaStream
diff --git a/src/media/video/video_receive_thread.h b/src/media/video/video_receive_thread.h
index a5ede14bb5..82c35ab1d2 100644
--- a/src/media/video/video_receive_thread.h
+++ b/src/media/video/video_receive_thread.h
@@ -59,7 +59,7 @@ public:
     // as VideoGenerator
     int getWidth() const;
     int getHeight() const;
-    int getPixelFormat() const;
+    AVPixelFormat getPixelFormat() const;
     MediaStream getInfo() const;
     void triggerKeyFrameRequest();
 
-- 
GitLab