[FFmpeg-devel] [PATCH v3 2/2] rtp: rfc4175: add handler for YCbCr-4:2:2

Damien Riegel damien.riegel at savoirfairelinux.com
Thu Mar 2 06:08:34 EET 2017


On Wed, Mar 01, 2017 at 12:12:42AM +0100, Thomas Volkert wrote:
> 
> On 28.02.2017 19:39, Damien Riegel wrote:
> > This adds partial support for the RFC 4175 (raw video over RTP). The
> > only supported formats are the YCbCr-4:2:2 8 bit because it's natively
> > supported by FFmpeg with pixel format UYVY, and 10 bit which requires
> > the vrawdepay codec to convert the payload in a format handled by
> > FFmpeg.
> >
> > Signed-off-by: Damien Riegel <damien.riegel at savoirfairelinux.com>
> > ---
> > Changes in v3:
> >  - rename rawvideo to rfc4175
> >
> >  
> 
> >  libavformat/rtpdec_rfc4175.c | 236 +++++++++++++++++++++++++++++++++++++++++++
> >  
> 
> I would use a more meaningful file name here - I think we always tried
> to avoid using the rfc number of the RTP payload format definition as
> part of the file name. We rather used the codec name or something
> similar. (There is only exception - RFC 2190 for H.263)

I changed it because of a suggestion in previous version (or maybe I got
that wrong). I can change it back to "rawvideo", or to whatever name is
preferred.

Thanks,
-- 
Damien


More information about the ffmpeg-devel mailing list