[Mplayer-cvslog] CVS: main/libmpcodecs ve_lavc.c,1.97,1.98

Diego Biurrun diego at biurrun.de
Tue Aug 10 01:46:44 CEST 2004


The Wanderer writes:
> Diego Biurrun wrote:
> 
> > attila writes:
> > 
> >> On Tue, Jul 20, 2004 at 02:06:33PM +0200, Michael Niedermayer CVS
> >> wrote:
> >>
> >>> CVS change done by Michael Niedermayer CVS
> >>>                  lavc_param_psnr=0; // disable block
> >>> +                mp_msg(MSGT_MENCODER,MSGL_ERR,"Can't open %s for writing. Check your permissions.\n",filename);
> >> 
> >>Hmm... i'd say here s/your/the/, but i'm not good at english :(
> >>Diego ? Wanderer ?
> > 
> > "your" is OK IMO, but "its" should be even better.
> 
> Yes, "your" works - it's saying "check the permissions which you are
> using". The usage is perhaps a little nonstandard, but flows better than
> some of the alternatives. It is, however, somewhat less precise than
> might be desirable; I don't think "its" sounds as good, but it does have
> the advantage of being more specific.

Committed "its".

Diego




More information about the MPlayer-cvslog mailing list