[MPlayer-dev-eng] NUT cleanup

[Rich Felker]

On Tue, Sep 06, 2005 at 09:00:03PM +0200, Michael Niedermayer wrote:
> Hi
> 
> On Tue, Sep 06, 2005 at 01:36:11PM -0400, Rich Felker wrote:
> > On Tue, Sep 06, 2005 at 07:05:19PM +0200, Michael Niedermayer wrote:
> [...]
> > > > It has a slight muxer implementation advantage - in my muxer it decides 
> > > > global timestamp by looking at 'stream[0].last_pts' - with this limitation, 
> > > > i can be sure that the first stream is the most "important" one...
> > > > And, well, I don't see it as a big limitation.. not much of a big advantage 
> > > > either though.
> > > 
> > > many muxers will have to remap stream ids because they are user / input file
> > > supplied, many muxers simple will ignore the rule ...
> > 
> > Again, IMO the nut library code should treat stream id's as zero-based
> > within each class, and just remap them to a single group of id's for
> > internal nut purposes. But maybe there's a reason this is bad?
> 
> stream ids should uniquely identify a stream otherwise you must keep track

OK.

> of streamid/class pairs everyhwere but its not too difficult to remap things
> around so iam fine with this if you think it has some advantage

Well most players want to show a zero-based list of streams of each
type anyway to the user; otherwise it's hard to know what stream id's
are valid for what. However the more I read your emails I agree, this
belongs in the player not the demuxer/spec.

> > > ok, if rich agrees iam fine with this though iam slightly in favor of having
> > > the forward_ptr in the checksumed area
> > 
> > I don't really care one way or the other, but I see no advantage to
> > including the forward_ptr in the checksum. So if omitting it makes
> > implementations simpler, let's omit it.
> 
> ok

ok

Rich