On Wed, Feb 22, 2006 at 03:15:24PM +0100, Michael Niedermayer wrote:
On Tue, Feb 21, 2006 at 07:26:07PM -0500, Rich Felker wrote:
- "TotalTime" - total length of the stream in msecs
hmmmmm, global info packets dont have chapter_len, index is optional and you remove this umm ...
I thought it was our intent that index would not be "optional". IIRC I suggested defining in the spec a 'NUT stream authored for permenant storage' or something similar, and requiring such a stream to conform to several conditions such as: repeated headers, index present, ...
The idea is that a partially written, truncated, streamed, etc. NUT file would not be a a "NUT stream authored for permenant storage" and would not be subject to these requirements, but NUTlint would check these things and indicate to the user that the file is not conformant to the more restrictive requirements.
BTW in the case of a truncated file where the index is lost, knowing TotalTime is not useful anyway since it will be wrong.
and the chapter lengthes? its a PITA to go and read the index to find the last chapter length
I'm ok with bringing back 'totaltime' or having some other solution for having the length for the last chapter... Other than this, can I commit the patch? Are there any comments left? - ods15