[Matroska-devel] libmatroska - AddFrame lacks default lacing type
steve.lhomme at free.fr
Mon Nov 3 23:18:16 CET 2003
Steve Lhomme wrote:
>> BTW, not that I think of it, it *does* make sense to have
>> ReadFully=false for an integer. Because in EbmlMaster::Read() it would
>> read the ID+length and leave the value behind. Which is nice if you
>> really don't want to read values and just the data structure.
>> But that also means in your case (reading a Cluster without the Block
>> data only) using ReadFully=false would read nothing, just the
>> structure. So IMO ReadFully should be moved from an integer to a
>> bitfield. With values like :
>> 0 Read no data (no integer content)
>> 1 Read all data
>> 2 Read partial data (ie integers OK, but not Block data)
>> Only the handling of the '2' value needs to be added (in
>> libebml+libmatroska), the rest is backward compatible.
> Should be :
> 0 Read partial data (ie integers OK, but not Block data)
> 1 Read all data
> 2 Read no data (no integer content)
> As 0 and 1 are the current implementation.
This is now committed in libebml and libmatroska.
(I haven't updated the library versions)
More information about the Matroska-devel