[MPlayer-DOCS] CVS: main/DOCS/xml/en mencoder.xml,1.64,1.65

[Guillaume POIRIER]

Hi,

On 5/15/05, The Wanderer <inverseparadox at comcast.net> wrote:
> Guillaume Poirier CVS wrote:
> 
> > Log Message:
> > some more fixes suggested by The Wanderer and Rich
> 
> Still two things (one old, one new):
> 
> > Index: mencoder.xml
> > ===================================================================
> > RCS file: /cvsroot/mplayer/main/DOCS/xml/en/mencoder.xml,v
> > retrieving revision 1.64
> > retrieving revision 1.65
> > diff -u -r1.64 -r1.65
> > --- mencoder.xml      14 May 2005 15:44:52 -0000      1.64
> > +++ mencoder.xml      14 May 2005 19:37:54 -0000      1.65
> > @@ -666,7 +666,7 @@
> >    grouped as a frame, even though they are not intended to be shown
> >    at the same moment in time.
> >    The MPEG-2 standard used on DVD and digital TV provides both a
> > -  way to encode the original progressive frames, and to store in
> > +  way to encode the original progressive frames, and store in
> 
> ...you still have the unwanted comma here; is there some specific reason
> you're retaining it which I don't know about?
> 
> You also still have "both a way to encode ... and store", instead of
> either "a way to both encode ... and store" or "a way both to encode ...
> and to store", here. Am I being unclear about what it is I'm trying to
> convey, or is there something else involved which I'm not seeing?

Well, I won't say you're unclear, because this is not the case. I must
admit I REALLY underestimated how tricky this part was.

Thanks a lot for double-checking.

Before doing another unnecessary wrong correction, here's the full
sentence that I'll commit tonight. Please tell me if it's all right
now:
  The MPEG-2 standard used on DVD and digital TV provides a
  way to both encode the original progressive frames and store in
  the header of each frame the number of fields for which it should
  be shown.

Cheers,
Guillaume