[MPlayer-dev-eng] [PATCH] tivo vstream

[Joey Parrish]

On Sat, Aug 21, 2004 at 03:44:32PM +0900, Attila Kinali wrote:
> On Fri, Aug 13, 2004 at 02:35:40AM +0200, Diego Biurrun wrote:
> > Joey Parrish writes:
> > > On Thu, Aug 12, 2004 at 01:08:24PM +0200, Diego Biurrun wrote:
> > > > > +  --disable-vstream      build with TiVo vstream [enable]
> > > > 
> > > > I would suggest setting it to disabled by default.  Only very few
> > > > people have a TiVo.
> > > 
> > > I believe that it doesn't actually do anything but increase
> > > executable size unless you use tivo:// URL.  So I disagree.
> > > Is there another reason to disable?
> > 
> > No other reason except avoiding bloat.  We have a tie then.  What do
> > the others think?
> 
> There is no better reason. Please keep in mind that a bigger
> executable always means a slower executable (can you spell
> cache miss ?). Thus unused stuff should stay out of the binary.

Yes, I have changed my mind after reviewing vstream more heavily.  I am
talking to the author to try to get it to work better as a library on
its own.  When that happens, I will write a simple MPlayer streaming
interface to it.

For now, it doesn't belong in CVS, so forget about it.

--Joey
> 
> 
> 			Attila Kinali
> 
> _______________________________________________
> MPlayer-dev-eng mailing list
> MPlayer-dev-eng at mplayerhq.hu
> http://mplayerhq.hu/mailman/listinfo/mplayer-dev-eng
> 
> 

-- 
"The Hell Law says that Hell is reserved exclusively for them that believe
in it.  Further, the lowest Rung in Hell is reserved for them that believe
in it on the supposition that they'll go there if they don't."
  HBT; The Gospel According to Fred, 3:1