[FFmpeg-devel] [PATCH] don't set is_streamed when it's not

Reimar Döffinger Reimar.Doeffinger
Thu Dec 20 19:30:04 CET 2007


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.
And if like in this case there is no allocation function it will be used like that.

Greetings,
Reimar D?ffinger




More information about the ffmpeg-devel mailing list