[FFmpeg-devel] [PATCH] IFF: Merge lots of duplicate code into a single function

Sebastian Vater cdgs.basty
Sun May 16 16:21:26 CEST 2010


Ronald S. Bultje a ?crit :
> Hi,
>
> On Fri, May 14, 2010 at 9:48 PM, Sebastian Vater
> <cdgs.basty at googlemail.com> wrote:
> [..]
>   
>> + * @return returns pointer to the source stream's end of line
>>     
>
> I find the return value a bit iffy, I'd expect it to return bytes
> consumed (and then callers do buf += decodebyterun()). What do others
> think? I don't want to bikeshed over this but just want some input
> from other devs.
>   

What's the current status regarding this? Should I change that now or
can the patch go in?

Remember that this one is required to apply the HAM support patch.

-- 

Best regards,
                   :-) Basty/CDGS (-:




More information about the ffmpeg-devel mailing list