[FFmpeg-devel] [PATCH]Basic XSUB encoder (take 5)

Robert Swain robert.swain
Mon Feb 9 02:58:54 CET 2009


2009/2/8 Michael Niedermayer <michaelni at gmx.at>:
> On Sat, Feb 07, 2009 at 03:33:12PM +0100, Bj?rn Axelsson wrote:
>> On Fri, 6 Feb 2009, Michael Niedermayer wrote:
>>
>> [...]
>>
>> > a patch adding a check in avcodec_encode_subtitle() that start_display_time is
>> > 0 is welcome!
>>
>> Patch attached (subtitle_check_start_display_time.diff).
>
> ok

Applied.

>> [...]
>>
>> > > +    if (h->num_rects == 0 || h->rects == NULL)
>> >
>> > !h->rects
>> >
>> >
>> > > +        return -1;
>> >
>> > also this check can be moved to avcodec_encode_subtitle() i think
>>
>> Patch attached (subtitle_check_rects.diff).
>
> ok

Applied.

Regards,
Rob




More information about the ffmpeg-devel mailing list