[MPlayer-DOCS] CVS: main/help help_mp-en.h,1.232,1.233

Corey Hickey bugfood-ml at fatooh.org
Thu Mar 30 09:14:55 CEST 2006


Reynaldo H. Verdejo CVS wrote:
> CVS change done by Reynaldo H. Verdejo CVS
> 
> Update of /cvsroot/mplayer/main/help
> In directory mail:/var2/tmp/cvs-serv30651/help
> 
> Modified Files:
> 	help_mp-en.h 
> Log Message:
> Part1 of several printf2mp_msg changes in patch from Otvos Attila oattila AT chello DOT hu

I see a few errors, but I'm not sure about most of them.

> +#define MSGTR_LIBMENU_ListMenuEntryDefinitionsNeedAName "[MENU] List menu entry definitions needs a name (line %d).\n"

This should either be:
List menu entry definitions need a name
-or-
List menu entry definition needs a name

I don't know which is intended, though the macro name implies the former.

> +#define MSGTR_LIBMENU_ListMenuNeedAnArgument "[MENU] List menu needs an argument.\n"

Not really a big deal, but the macro name doesn't agree with the string: 
"need" vs. "needs".

> +#define MSGTR_LIBMENU_AChildIsAlreadyRunning "[MENU] A child is alredy running.\n"

alredy --> already

> +#define MSGTR_LIBMENU_PrefMenuNeedAnArgument "[MENU] Pref menu needs an argument.\n"

Again, the macro name says "need" and the string says "needs". I don't 
know if it's worth changing them.


Other than that, it looks ok. Thanks for committing these in chunks. 
Proofreading them is a bit mind-numbing. :)

-Corey




More information about the MPlayer-DOCS mailing list