[FFmpeg-devel] [PATCH/v3.1] rmdec.c/rm.h changes to allow rtsp
Alex Beregszaszi
alex
Sun Jul 22 17:13:59 CEST 2007
Hi,
> > maybe you want to merge get_str() and get_str8()
> > a 1 line difference doesnt merit having a second function ...
>
>
> Makes sense. How about $attached?
>
> (I can also make get_str{.8}() macros to get_str_i() to save another 2x3
> lines for their function declarations, I think you originally intended that,
> but then the if(len_size==1) would no longer be inlined, making it slightly
> slower...)
I would change to get_str8 and get_str16 (make them a macro) and call
the real function get_str. What about moving this into aviobuf.c ?
--
Alex
More information about the ffmpeg-devel
mailing list