VideoCodec: remove setImageParameters from with, height and pixelFormat

Clement Champetier · Clement Champetier · commit e83abbd1365e · 2015-01-13T18:26:47.000+01:00
Only one way so set the video codec: from a videoFrameDesc.
diff --git a/src/AvTranscoder/codec/VideoCodec.cpp b/src/AvTranscoder/codec/VideoCodec.cpp
@@ -38,14 +38,9 @@ std::pair< size_t, size_t > VideoCodec::getTimeBase() const
 
 void VideoCodec::setImageParameters( const VideoFrameDesc& videoFrameDesc )
 {
-	setImageParameters( videoFrameDesc.getWidth(), videoFrameDesc.getHeight(), videoFrameDesc.getPixelFormat() );
-}
-
-void VideoCodec::setImageParameters( const size_t width, const size_t height, const AVPixelFormat& pixel )
-{
-	_avCodecContext->width = width;
-	_avCodecContext->height = height;
-	_avCodecContext->pix_fmt = pixel;
+	_avCodecContext->width = videoFrameDesc.getWidth();
+	_avCodecContext->height = videoFrameDesc.getHeight();
+	_avCodecContext->pix_fmt = videoFrameDesc.getPixelFormat();
 }
 
 void VideoCodec::setTimeBase( const size_t num, const size_t den, const size_t ticksPerFrame )
diff --git a/src/AvTranscoder/codec/VideoCodec.hpp b/src/AvTranscoder/codec/VideoCodec.hpp
@@ -20,7 +20,6 @@ class AvExport VideoCodec : public ICodec
 	std::pair< size_t, size_t > getTimeBase() const;
 	
 	void setImageParameters( const VideoFrameDesc& videoFrameDesc );
-	void setImageParameters( const size_t width, const size_t height, const AVPixelFormat& pixel );
 
 	void setTimeBase( const size_t num, const size_t den, const size_t ticksPerFrame = 1 );
 };
