[FFmpeg-devel] [PATCH] don't set is_streamed when it's not
Fri Dec 21 00:17:32 CET 2007
M?ns Rullg?rd wrote:
> Reimar D?ffinger <Reimar.Doeffinger at stud.uni-karlsruhe.de> writes:
> > Hello,
> > On Thu, Dec 20, 2007 at 01:16:11PM -0500, Rich Felker wrote:
> >> On Thu, Dec 20, 2007 at 10:33:38AM +0100, Michael Niedermayer
> >> wrote:
> > [...]
> >> > What is the advantage of having one get and one set function for
> >> > each field in ByteIOContext compared to direct access to
> >> > ByteIOContext? Will you also propose to add such get/set
> >> > functions for each field in AVCodecContext and AVFormatContext ?
> >> I agree with Michael here. Just have a policy of not reordering
> >> fields and adding new ones only at the end. Then direct access is
> >> possible and ABI is never broken. All this wrapper function mess
> >> reaks of C++ and has no practical benefit.
> > Well, it has the disadvantage that it can be used without calling an
> > allocation function, thus making sure that adding new fields will
> > break, and worse it will break in a very hard to find way.
> But it won't be our code that breaks ;-)
Sure, but we will still get the bug reports about latest libavcodec.dll
crashing where previous version worked fine.
More information about the ffmpeg-devel