should i18n.h include misc.h if internal ?

[Carles Pina i Estany]

Hello,

Imagine that I'm going to gettexttize commands/true.c . I need to add
#include <grub/i18n.h> , but this include defines:

# define _(str) grub_gettext(str)

so I need #include <grub/misc.h>

Any problem with this patch:

=== modified file 'include/grub/i18n.h'
--- include/grub/i18n.h 2009-11-23 20:34:42 +0000
+++ include/grub/i18n.h 2009-11-23 23:46:19 +0000
@@ -24,6 +24,7 @@
 # include <libintl.h>
 # define _(str) gettext(str)
 #else
+# include <grub/misc.h>
 # define _(str) grub_gettext(str)
 #endif

Other option would be to move these:
----
const char *EXPORT_FUNC(grub_gettext_dummy) (const char *s);
extern const char *(*EXPORT_VAR(grub_gettext)) (const char *s);
----

from misc.h to i18n.h (internal). I haven't tested but if it works it
includes less things of course and could be better if it's not needed.

Thanks,

-- 
Carles Pina i Estany
        http://pinux.info