[NUT-devel] CVS: main/DOCS/tech mpcf.txt,1.143,1.144

[Rich Felker]

On Sun, Mar 12, 2006 at 10:41:43PM +0100, Michael Niedermayer wrote:
> Hi
> 
> On Sun, Mar 12, 2006 at 06:54:21PM +0200, Oded Shimon wrote:
> > On Sun, Mar 12, 2006 at 06:43:13PM +0200, Oded Shimon wrote:
> > > On Sun, Mar 12, 2006 at 03:56:59PM +0100, Michael Niedermayer wrote:
> > > > Hi
> > > > 
> > > > On Sun, Mar 12, 2006 at 02:30:48PM +0100, Oded Shimon CVS wrote:
> > > > > CVS change done by Oded Shimon CVS
> > > > > 
> > > > > Update of /cvsroot/mplayer/main/DOCS/tech
> > > > > In directory mail:/var2/tmp/cvs-serv5137/DOCS/tech
> > > > > 
> > > > > Modified Files:
> > > > > 	mpcf.txt 
> > > > > Log Message:
> > > > > better max for frame code values, still allowing compact implementation
> > > > 
> > > > iam against this change, having a few reserved values
> > > > 1. allows us to use these values in the future (using exactly 8 or 16 bit
> > > >    means that hw demuxers might not be easily able to be extended)
> > > > 2. allows implementations to internally use the unused cases for flags
> > > >    and special cases
> > > > 3. the extended values added are not usefull in practice
> > > > 3A. you have 254 frame codes, some are needed for "exceptional" cases
> > > >     so >250 streams will hardly be hardcodeable in reality
> > > > 3B. data_size_mul/lsb > 256 make little >16384 make no sense
> > 
> > BTW, regarding that, they are useful for storing mp3 packets in a single 
> > byte of overhead...
> 
> mp3 doesnt have packets > 16384 bytes

He probably meant the >256 case.

Rich