On Sat, Dec 03, 2016 at 13:23:26 +0100, Michael Niedermayer wrote: > > Please add a comment like /* gop_size */ here and similar ones below. > > Otherwise it'll be rather mysterious what's the point of all these avio_rb* > > calls. > > done locally
Are the remaining avio_rb*() calls good for anything except documenting the now unused fields? I know the code is just called once for the headers, but replacing N avio_rb*() calls with a single avio_skip() would be way more efficient, right? Moritz _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel