[Mplayer-cvslog] CVS: main/DOCS/tech slave.txt,1.12,1.13
Diego Biurrun
diego at biurrun.de
Tue Aug 24 11:48:54 CEST 2004
The Wanderer writes:
> Joey Parrish CVS wrote:
>
> > +sub_select
> > + Cycle through the subtitle set. Supported subtitle sources are -sub
> > + options on the command line, VOBSubs, DVD subtitles, and OGG text
> > + streams.
>
> Miscapitalization of "Ogg" and "VOBsub"; also, how are VOBsubs distinct
> from "DVD subtitles"?
>
> While I'm at it: "available subtitles" might be a little more clear than
> "subtitle set". It took me a moment to figure out unambiguously what was
> meant.
>
> From a slightly later commit:
>
> > +vo_rootwin
> > + Toggle playback on the root window. This runtime toggle is only
> > + supported by the directx driver at the moment.
>
> I'd suggest "is currently supported only by the DirectX driver", or
> "currently works only with -vo directx" if the capitalization is because
> this is an explicit reference to that video-out method.
All applied.
Diego
More information about the MPlayer-cvslog
mailing list