[MPlayer-dev-eng] Re: [PATCH] Clean up demuxers

[Daniel Egger]

Hija,

> There is now 2 special demuxer number :)

Wouldn't be a problem, the idea is scalable. :)

> And this look like a very bad solution.

Not as far as I can see.

> Anyway using an array like you have done isn't the right way.

Convince me otherwise. :)

> You should put the functions inside the demuxer_t struct itself.
> Then during opening, the demuxer itself fill this fields with the
> correct function. It's the same thing but it doesn't need any array
> or things like that. Even cleaner :)

Sound like you thought about it; unfortunately I've no idea what exactly
you would change so please elaborate or change it and I'll see. :)

-- 
Servus,
       Daniel