[FFmpeg-devel] got RTP encoding to support XVID codec

[Nash Tsai]

Hi Aurelien,

I think this is a good 1 line fix, but it also mean that ffmpeg has no
interest to distinguish MPEG4 and XVID encoder, although XVID is
really just a part of MEPG4?

Nash

On Fri, Mar 12, 2010 at 8:18 PM, Aurelien Jacobs <aurel at gnuage.org> wrote:
> On Fri, Mar 12, 2010 at 05:45:20PM +0800, Nash Tsai wrote:
>> Hi,
>>
>> I've got RTP encoding to support XVID and works great with VLC media
>> player with its output SDP, will be great if you can accept this
>> patch.
>>
>> [...]
>>
>> Index: libavformat/sdp.c
>> ===================================================================
>> --- libavformat/sdp.c (revision 22452)
>> +++ libavformat/sdp.c (working copy)
>> @@ -226,6 +226,7 @@
>> ? ? ? ? ? ? ?av_strlcatf(buff, size, "a=rtpmap:%d H263-2000/90000\r\n", payload_type);
>> ? ? ? ? ? ? ?break;
>> ? ? ? ? ?case CODEC_ID_MPEG4:
>> + ? ? ? ?case CODEC_ID_XVID:
>
> I wonder what this CODEC_ID_XVID could be useful for ?
> The attched patch may be a better fix (CODEC_ID_XVID is not used
> anywhere else).
>
> Aurel
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at mplayerhq.hu
> https://lists.mplayerhq.hu/mailman/listinfo/ffmpeg-devel
>