[MPlayer-dev-eng] [PATCH] multi language part 1/2
Diego Biurrun
diego at biurrun.de
Wed Dec 10 09:02:45 CET 2008
On Tue, Dec 09, 2008 at 11:43:24PM +0100, Ötvös Attila wrote:
> Ezzel a dátummal: Tuesday 09 December 2008 20:49:41 Diego Biurrun ezt írta:
>
> > This patch is huge and does many different things, as your patches
> > always seem to. Could you please split this into small self-contained
> > pieces? Otherwise this will be impossible to manage and review.
>
> Ok. I splited patch:
>
> 01-01-mp_msg_fixed.patch:
> This patch fixed double " string terminator and
> change MGSTR_* to MSGTR_*
> (12x help_mp-*.h)
I already made this change.
> 01-03-remove-static-string-cat.patch:
> This patch removed static string cat
Why?
> 01-04-remove-spaces-string.patch:
> This patch removed spaces string from gui/mplayer/gtk/menu.c
Why?
> 01-05-remove-static-string-cat-about.patch:
> This patch removed static string cat from gui/mplayer/gtk/about.c
Why?
Note that I'm not against these patches, I'd just like to hear some
reasoning for them.
Diego
More information about the MPlayer-dev-eng
mailing list