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

Måns Rullgård mans
Thu Dec 20 19:35:46 CET 2007


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 ;-)

-- 
M?ns Rullg?rd
mans at mansr.com




More information about the ffmpeg-devel mailing list