[FFmpeg-devel] [PATCH] IFF: Merge lots of duplicate code into a single function
Sun May 16 18:06:12 CEST 2010
Ronald S. Bultje a ?crit :
> 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.
Hi Ronald, do you like it better now? I have also added returning
an error if byterun1 is malformed in some way.
Also I forgot to add documentation for buf_end => Fixed.
Besides this, I added Stefano's hint about get_buffer returning err
instead -1 here, since I had to introduce err variable here anyway.
Please review this new one, if it's OK I will adapt HAM patch to this.
:-) Basty/CDGS (-:
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 6111 bytes
Desc: not available
More information about the ffmpeg-devel