[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