[MPlayer-dev-eng] [PATCH] gui/menu: optimize language table.
Ingo Brückl
ib at wupperonline.de
Sun Feb 14 00:10:31 CET 2016
Reimar Döffinger wrote on Sat, 13 Feb 2016 23:50:48 +0100:
> Store fixed size 4 entry directly
> instead of via pointer.
> diff --git a/gui/dialog/menu.c b/gui/dialog/menu.c
> index de6f45f..fcb56fb 100644
> +++ b/gui/dialog/menu.c
> @@ -230,7 +230,7 @@ GtkWidget * AddSeparator( GtkWidget * Menu )
> typedef struct
> {
> int id;
> - const char * id2;
> + const char id2[4];
> const char * name;
> } Languages_t;
I can sense Coverity on strcasecmp(Languages[i].id2, p). After fixing id2[3]
seems enough.
Ingo
More information about the MPlayer-dev-eng
mailing list