[NUT-devel] Patch for info packets

Oded Shimon ods15 at ods15.dyndns.org
Thu Feb 23 11:10:13 CET 2006


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




More information about the NUT-devel mailing list