On 20 June 2010 14:13, Ronald S. Bultje <[email protected]> wrote: > Hi, > > On Sun, Jun 20, 2010 at 3:46 PM, Josh Allmann <[email protected]> > wrote: >> $subj > > skip_spaces() is small enough that I think a static inline might make > more sense (in lavf/internal.h).
Fixed. Can you also check and make sure that > the rtp*.c files do not have copies of this functions lying around? I > thought this was copied 1-2 times... ffserver has a copy, but I grepped for skip_spaces -- if it is named differently, I missed it. There is still quite a bit of boilerplate in the sdp/fmtp parsing code shared among rtp*, Martin added another todo for that on the wiki. Josh _______________________________________________ FFmpeg-soc mailing list [email protected] https://lists.mplayerhq.hu/mailman/listinfo/ffmpeg-soc
