[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