[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