[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