[MPlayer-dev-eng] [PATCH] vstream client support

Joey Parrish joey at nicewarrior.org
Wed Feb 23 02:34:28 CET 2005


On Wed, Feb 23, 2005 at 01:21:04AM +0100, Diego Biurrun wrote:
> Joey Parrish writes:
> > +<listitem><simpara>
> > +  TY: This is a TiVo MPEG stream.  It contains MPEG PES data for audio and
> > +  video streams, as well as extra information like closed captions.  The
> > +  container is not an MPEG program stream, but a closed format created by
> > +  TiVo.  For more information on TiVo stream format, please refer to
> 
> on the
> 
> Also, don't use two spaces between sentences, we use the European
> style with one space throughout.
> 
> Also, you're adding a bit of trailing whitespace here and there, but
> that's unimportant.
> 
> Looks good otherwise.

I've incorporated your suggestions and replied to Reimar with the new
patch.

Thanks,
--Joey

-- 
"There are trivial truths and there are great truths.  The opposite of a
trivial truth is plainly false.  The opposite of a great truth is also true."
  --Neils Bohr




More information about the MPlayer-dev-eng mailing list