[Matroska-devel] libmatroska - AddFrame lacks default lacing type

Steve Lhomme steve.lhomme at free.fr
Fri Oct 31 14:08:05 CET 2003


Steve Lhomme wrote:

[edit]
> BTW, not that I think of it, it *does* make sense to have 

s/not/now/

> 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.




More information about the Matroska-devel mailing list