Re: gettext: normal_menu.c

2009-12-03 Thread Carles Pina i Estany

Hello,

On Nov/29/2009, Carles Pina i Estany wrote:

 +  grub_printf (_(Failed to boot default entries.));
 +  grub_putchar ('\n');

This is the only string that I've spotted that it's String\n
other ones are:
\nString
String\n  

Up to the discussion in the grub_printfnl I would change it to
grub_printfnl(_(Failed to boot default entries.)); or grub_printf
(Failed to boot default entries.);

Any feedback for the rest of the patch?

Thanks,

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


___
Grub-devel mailing list
Grub-devel@gnu.org
http://lists.gnu.org/mailman/listinfo/grub-devel


Re: gettext: normal_menu.c

2009-11-28 Thread Carles Pina i Estany

Hi,

I've added some easy changes for normal/menu_entry.c

So better review this patch.

Thanks!

-- 
Carles Pina i Estany
http://pinux.info
2009-11-28  Carles Pina i Estany car...@pina.cat

* normal/menu_text.c (grub_color_menu_high): Gettexttize string.
(print_spaces): New function.
(grub_print_ucs4): New function.
(print_message_indented): New function.
(getstringwidth): New function.
(print_message): Gettexttize strings using print_message_indented.
(run_menu): Replaces grub_printf by dynamic terminal width
print_spaces.
(get_entry_number): Gettextize and uses dynamic terminal width.
(notify_booting, notify_fallback, notify_execution_failure): 
Gettextize.
* normal/menu_entry.c (store_completion): Cleanup the gettextized
string
(run): Likewise.
(grub_menu_entry_run): Likewise.
=== modified file 'normal/menu_entry.c'
--- normal/menu_entry.c	2009-11-23 20:31:57 +
+++ normal/menu_entry.c	2009-11-28 12:14:22 +
@@ -836,7 +836,9 @@ store_completion (const char *item, grub
 	}
 
   grub_gotoxy (0, GRUB_TERM_HEIGHT - 3);
-  grub_printf (   Possible %s are:\n, what);
+  grub_printf (   );
+  grub_printf (_(Possible %s are:), what);
+  grub_printf (\n   );
 }
 
   /* Make sure that the completion buffer has enough room.  */
@@ -997,7 +999,9 @@ run (struct screen *screen)
 }
 
   grub_cls ();
-  grub_printf (_(  Booting a command list\n\n));
+  grub_printf (  );
+  grub_printf (_(Booting a command list));
+  grub_printf (\n\n);
 
 
   /* Execute the script, line for line.  */
@@ -1177,6 +1181,7 @@ grub_menu_entry_run (grub_menu_entry_t e
   grub_cls ();
   grub_print_error ();
   grub_errno = GRUB_ERR_NONE;
-  grub_printf (_(\nPress any key to continue...));
+  grub_putchar ('\n');
+  grub_printf (_(Press any key to continue...));
   (void) grub_getkey ();
 }

=== modified file 'normal/menu_text.c'
--- normal/menu_text.c	2009-11-23 20:59:24 +
+++ normal/menu_text.c	2009-11-28 01:18:23 +
@@ -39,7 +39,8 @@ static grub_uint8_t grub_color_menu_high
 void
 grub_wait_after_message (void)
 {
-  grub_printf (\nPress any key to continue...);
+  grub_putchar('\n');
+  grub_printf (_(Press any key to continue...));
   (void) grub_getkey ();
   grub_putchar ('\n');
 }
@@ -81,28 +82,128 @@ draw_border (void)
 }
 
 static void
+print_spaces (int number_spaces)
+{
+  int i;
+  for (i = 0; i  number_spaces; i++)
+grub_putchar(' ');
+}
+
+static void
+grub_print_ucs4 (const grub_uint32_t* str, const grub_uint32_t* last_position)
+{
+  while (str  last_position)
+{
+  grub_putcode(*str);
+  str++;
+}
+}
+
+static grub_ssize_t
+getstringwidth(grub_uint32_t* str, const grub_uint32_t* last_position)
+{
+  grub_ssize_t width = 0;
+
+  while (str  last_position)
+{
+  width += grub_getcharwidth(*str);
+  str++;
+}
+return width;
+}
+
+static void
+print_message_indented(const char* msg)
+{
+  const int line_len = GRUB_TERM_WIDTH - grub_getcharwidth('m') * 15;
+
+  grub_uint32_t *unicode_msg;
+
+  grub_ssize_t msg_len = grub_strlen(msg);
+
+  unicode_msg = grub_malloc (msg_len * sizeof(*unicode_msg));
+
+  msg_len = grub_utf8_to_ucs4 (unicode_msg, msg_len,
+(grub_uint8_t *) msg, -1, 0);
+
+  if (! unicode_msg)
+{
+  grub_printf(print_message_indented ERROR1: %s,msg);
+  return;
+}
+
+  if (msg_len  0)
+{
+  grub_printf(print_message_indented ERROR2: %s,msg);
+  grub_free (unicode_msg);
+  return;
+}
+  
+  const grub_uint32_t* last_position = unicode_msg + msg_len;
+  
+  grub_uint32_t* current_position = unicode_msg;
+
+  grub_uint32_t* next_new_line = unicode_msg;
+
+  while (current_position  last_position)
+{
+  next_new_line = (grub_uint32_t*)last_position;
+
+  while (getstringwidth(current_position,next_new_line)  line_len 
+  	|| (*next_new_line != ' '  next_new_line  current_position 
+	   next_new_line != last_position))
+{  
+  next_new_line--;
+}
+ 
+  if (next_new_line == current_position)
+{
+  next_new_line = (next_new_line + line_len  last_position) ? 
+   (grub_uint32_t*)last_position :
+		   next_new_line + line_len;
+}
+
+  print_spaces(6);
+  grub_print_ucs4(current_position,next_new_line);
+  grub_putchar('\n');
+  
+  next_new_line++;
+  current_position = next_new_line;
+}
+  grub_free(unicode_msg);
+}
+
+static void
 print_message (int nested, int edit)
 {
   grub_setcolorstate (GRUB_TERM_COLOR_NORMAL);
-
+  
+  grub_putchar ('\n');
   if (edit)
 {
-  grub_printf (\n\
-  Minimum Emacs-like screen editing is supported. TAB lists\n\
-  completions. Press Ctrl-x to boot, Ctrl-c for a command-line\n\
-  or ESC to return menu.);
+  print_message_indented(_(Minimum Emacs-like 

Re: gettext: normal_menu.c

2009-11-28 Thread Robert Millan

Please remember about spaces in our coding style:

On Sat, Nov 28, 2009 at 12:21:28PM +, Carles Pina i Estany wrote:
 +  next_new_line = (grub_uint32_t*)last_position;
   ^ ^
 +  while (getstringwidth(current_position,next_new_line)  line_len 
  ^ ^
etc.  If in doubt, use indent(1).

-- 
Robert Millan

  The DRM opt-in fallacy: Your data belongs to us. We will decide when (and
  how) you may access your data; but nobody's threatening your freedom: we
  still allow you to remove your data and not access it at all.


___
Grub-devel mailing list
Grub-devel@gnu.org
http://lists.gnu.org/mailman/listinfo/grub-devel


Re: gettext: normal_menu.c

2009-11-28 Thread Carles Pina i Estany

Hello,

On Nov/28/2009, Robert Millan wrote:
 
 Please remember about spaces in our coding style:
 
 On Sat, Nov 28, 2009 at 12:21:28PM +, Carles Pina i Estany wrote:
  +  next_new_line = (grub_uint32_t*)last_position;
^ ^
  +  while (getstringwidth(current_position,next_new_line)  line_len 
   ^ ^
 etc.  If in doubt, use indent(1).

Attached the new patch. Comments are welcomed.

The original normal/* files are not indent consistent. So I:
1.- copied the files to old/
2.- applied the patch to the original files and copy to new/
3.- indent old/* and new/*
4.- diff -p -u old/ new/

For next time: what you think to make a patch + ChangeLog entry and
push to trunk and then another push with the changes to the
indent-compliant files?

Cheers,

-- 
Carles Pina i Estany
http://pinux.info
diff -p -u old/menu_entry.c new/menu_entry.c
--- old/menu_entry.c	2009-11-29 00:56:42.020701488 +
+++ new/menu_entry.c	2009-11-29 00:56:06.748700757 +
@@ -833,7 +833,9 @@ store_completion (const char *item, grub
 	}
 
   grub_gotoxy (0, GRUB_TERM_HEIGHT - 3);
-  grub_printf (   Possible %s are:\n, what);
+  grub_printf (   );
+  grub_printf (_(Possible %s are:), what);
+  grub_printf (\n   );
 }
 
   /* Make sure that the completion buffer has enough room.  */
@@ -994,7 +996,9 @@ run (struct screen *screen)
   }
 
   grub_cls ();
-  grub_printf (_(  Booting a command list\n\n));
+  grub_printf (  );
+  grub_printf (_(Booting a command list));
+  grub_printf (\n\n);
 
 
   /* Execute the script, line for line.  */
@@ -1174,6 +1178,7 @@ fail:
   grub_cls ();
   grub_print_error ();
   grub_errno = GRUB_ERR_NONE;
-  grub_printf (_(\nPress any key to continue...));
+  grub_putchar ('\n');
+  grub_printf (_(Press any key to continue...));
   (void) grub_getkey ();
 }
diff -p -u old/menu_text.c new/menu_text.c
--- old/menu_text.c	2009-11-29 00:56:33.700701575 +
+++ new/menu_text.c	2009-11-29 00:56:09.736702258 +
@@ -39,7 +39,8 @@ static grub_uint8_t grub_color_menu_high
 void
 grub_wait_after_message (void)
 {
-  grub_printf (\nPress any key to continue...);
+  grub_putchar ('\n');
+  grub_printf (_(Press any key to continue...));
   (void) grub_getkey ();
   grub_putchar ('\n');
 }
@@ -81,27 +82,130 @@ draw_border (void)
 }
 
 static void
+print_spaces (int number_spaces)
+{
+  int i;
+  for (i = 0; i  number_spaces; i++)
+grub_putchar (' ');
+}
+
+static void
+grub_print_ucs4 (const grub_uint32_t * str,
+		 const grub_uint32_t * last_position)
+{
+  while (str  last_position)
+{
+  grub_putcode (*str);
+  str++;
+}
+}
+
+static grub_ssize_t
+getstringwidth (grub_uint32_t * str, const grub_uint32_t * last_position)
+{
+  grub_ssize_t width = 0;
+
+  while (str  last_position)
+{
+  width += grub_getcharwidth (*str);
+  str++;
+}
+  return width;
+}
+
+static void
+print_message_indented (const char *msg)
+{
+  const int line_len = GRUB_TERM_WIDTH - grub_getcharwidth ('m') * 15;
+
+  grub_uint32_t *unicode_msg;
+
+  grub_ssize_t msg_len = grub_strlen (msg);
+
+  unicode_msg = grub_malloc (msg_len * sizeof (*unicode_msg));
+
+  msg_len = grub_utf8_to_ucs4 (unicode_msg, msg_len,
+			   (grub_uint8_t *) msg, -1, 0);
+
+  if (!unicode_msg)
+{
+  grub_printf (print_message_indented ERROR1: %s, msg);
+  return;
+}
+
+  if (msg_len  0)
+{
+  grub_printf (print_message_indented ERROR2: %s, msg);
+  grub_free (unicode_msg);
+  return;
+}
+
+  const grub_uint32_t *last_position = unicode_msg + msg_len;
+
+  grub_uint32_t *current_position = unicode_msg;
+
+  grub_uint32_t *next_new_line = unicode_msg;
+
+  while (current_position  last_position)
+{
+  next_new_line = (grub_uint32_t *) last_position;
+
+  while (getstringwidth (current_position, next_new_line)  line_len
+	 || (*next_new_line != ' '  next_new_line  current_position 
+		 next_new_line != last_position))
+	{
+	  next_new_line--;
+	}
+
+  if (next_new_line == current_position)
+	{
+	  next_new_line = (next_new_line + line_len  last_position) ?
+	(grub_uint32_t *) last_position : next_new_line + line_len;
+	}
+
+  print_spaces (6);
+  grub_print_ucs4 (current_position, next_new_line);
+  grub_putchar ('\n');
+
+  next_new_line++;
+  current_position = next_new_line;
+}
+  grub_free (unicode_msg);
+}
+
+static void
 print_message (int nested, int edit)
 {
   grub_setcolorstate (GRUB_TERM_COLOR_NORMAL);
 
+  grub_putchar ('\n');
   if (edit)
 {
-  grub_printf (\n\
-  Minimum Emacs-like screen editing is supported. TAB lists\n\
-  completions. Press Ctrl-x to boot, Ctrl-c for a command-line\n\
-  or ESC to return menu.);
+  print_message_indented (_(Minimum Emacs-like screen editing is \
+supported. TAB lists completions. Press Ctrl-x to boot, Ctrl-c for a \
+command-line or ESC to return 

Re: gettext: normal_menu.c

2009-11-28 Thread Vladimir 'φ-coder/phcoder' Serbinenko
Carles Pina i Estany wrote:
 Hello,

 On Nov/28/2009, Robert Millan wrote:
   
 Please remember about spaces in our coding style:

 On Sat, Nov 28, 2009 at 12:21:28PM +, Carles Pina i Estany wrote:
 
 +  next_new_line = (grub_uint32_t*)last_position;
   
^ ^
 
 +  while (getstringwidth(current_position,next_new_line)  line_len 
   
   ^ ^
 etc.  If in doubt, use indent(1).
 

 Attached the new patch. Comments are welcomed.

 The original normal/* files are not indent consistent. So I:
 1.- copied the files to old/
 2.- applied the patch to the original files and copy to new/
 3.- indent old/* and new/*
 4.- diff -p -u old/ new/

 For next time: what you think to make a patch + ChangeLog entry and
 push to trunk and then another push with the changes to the
 indent-compliant files?

   
If you find badly-indented files and you know nobody else is working on
those files right now (e.g. no changes on it in experimental) you can
indent them in the trunk without mentionning it on the list. If you're
aware of people working on these files notify them. Also check that
indent didn't make the things worse and add a Changlog entry like
-MM-DD  .

  * file: Indented.
 Cheers,

   
 

 ___
 Grub-devel mailing list
 Grub-devel@gnu.org
 http://lists.gnu.org/mailman/listinfo/grub-devel


-- 
Regards
Vladimir 'φ-coder/phcoder' Serbinenko




signature.asc
Description: OpenPGP digital signature
___
Grub-devel mailing list
Grub-devel@gnu.org
http://lists.gnu.org/mailman/listinfo/grub-devel


gettext: normal_menu.c

2009-11-27 Thread Carles Pina i Estany

Hi,

Find attached the patch and ChangeLog for normal_menu.c

Suggestions? What to change?

Notice that I used grub_printf to print the spaces/new lines of
formatting.

And print_message_indented receive a long string and prints it
splitting by the spaces and indented 6 spaces.

(15 and 6 in that method could be macrofied or const int spaces = 6,
would be in normal.h ?)

-- 
Carles Pina i Estany
http://pinux.info
2009-11-28  Carles Pina i Estany car...@pina.cat

* normal/menu_text.c (grub_color_menu_high): Gettexttize string.
(print_spaces): New function.
(grub_print_ucs4): New function.
(print_message_indented): New function.
(print_message): Gettexttize strings using print_message_indented.
(run_menu): Replaces grub_printf by dynamic terminal width
print_spaces.
(get_entry_number): Gettextize and uses dynamic terminal width.
(notify_booting, notify_fallback, notify_execution_failure): 
Gettextize.
=== modified file 'normal/menu_text.c'
--- normal/menu_text.c	2009-11-23 20:59:24 +
+++ normal/menu_text.c	2009-11-27 23:58:34 +
@@ -39,7 +39,8 @@ static grub_uint8_t grub_color_menu_high
 void
 grub_wait_after_message (void)
 {
-  grub_printf (\nPress any key to continue...);
+  grub_putchar('\n');
+  grub_printf (_(Press any key to continue...));
   (void) grub_getkey ();
   grub_putchar ('\n');
 }
@@ -81,28 +82,107 @@ draw_border (void)
 }
 
 static void
+print_spaces (int number_spaces)
+{
+  int i;
+  for (i = 0; i  number_spaces; i++)
+grub_putchar(' ');
+}
+
+static void
+grub_print_ucs4 (grub_uint32_t* str, const grub_uint32_t* last_position)
+{
+  while (str  last_position)
+{
+  grub_putcode(*str);
+  str++;
+}
+}
+  
+print_message_indented(const char* msg)
+{
+  const int line_len = GRUB_TERM_WIDTH - 15;
+
+  grub_uint32_t *unicode_msg;
+
+  grub_ssize_t msg_len = grub_strlen(msg);
+
+  unicode_msg = grub_malloc (msg_len * sizeof(*unicode_msg));
+
+  msg_len = grub_utf8_to_ucs4 (unicode_msg, msg_len,
+(grub_uint8_t *) msg, -1, 0);
+
+  if (! unicode_msg)
+{
+  grub_printf(print_message_indented ERROR1: %s,msg);
+  return;
+}
+
+  if (msg_len  0)
+{
+  grub_printf(print_message_indented ERROR2: %s,msg);
+  grub_free (unicode_msg);
+  return;
+}
+  
+  const grub_uint32_t* last_position = unicode_msg + msg_len;
+
+  grub_uint32_t* current_position = unicode_msg;
+
+  grub_uint32_t* next_new_line = unicode_msg;
+
+  while (current_position  last_position)
+{
+  next_new_line = (next_new_line + line_len  last_position) ? 
+   (grub_uint32_t*)last_position :
+		   next_new_line + line_len;
+
+  while (*next_new_line != ' '  next_new_line = unicode_msg 
+  next_new_line != last_position)
+{  
+  next_new_line--;
+}
+
+  print_spaces(6);
+  grub_print_ucs4(current_position,next_new_line);
+  grub_putchar('\n');
+  
+  next_new_line++;
+  current_position = next_new_line;
+}
+  grub_free(unicode_msg);
+}
+
+static void
 print_message (int nested, int edit)
 {
   grub_setcolorstate (GRUB_TERM_COLOR_NORMAL);
-
+  
+  grub_putchar ('\n');
   if (edit)
 {
-  grub_printf (\n\
-  Minimum Emacs-like screen editing is supported. TAB lists\n\
-  completions. Press Ctrl-x to boot, Ctrl-c for a command-line\n\
-  or ESC to return menu.);
+  print_message_indented(_(Minimum Emacs-like screen editing is \
+supported. TAB lists completions. Press Ctrl-x to boot, Ctrl-c for a \
+command-line or ESC to return menu.));
 }
   else
 {
-  grub_printf (_(\n\
-  Use the %C and %C keys to select which entry is highlighted.\n),
-		   (grub_uint32_t) GRUB_TERM_DISP_UP, (grub_uint32_t) GRUB_TERM_DISP_DOWN);
-  grub_printf (\
-  Press enter to boot the selected OS, \'e\' to edit the\n\
-  commands before booting or \'c\' for a command-line.);
+  const char *msg = _(Use the %C and %C keys to select which \
+entry is highlighted.);
+  char* msg_translated = grub_malloc(sizeof(char) * grub_strlen(msg) + 1);
+  
+  grub_sprintf(msg_translated, msg, (grub_uint32_t) GRUB_TERM_DISP_UP, (grub_uint32_t) GRUB_TERM_DISP_DOWN);
+  print_message_indented(msg_translated);
+
+  grub_free (msg_translated);
+
+  print_message_indented(_(Press enter to boot the selected OS, \
+\'e\' to edit the commands before booting or \'c\' for a command-line.));
   if (nested)
-	grub_printf (\n\
-  ESC to return previous menu.);
+{
+	  grub_printf (\n);
+  grub_printf(_(ESC to return previous menu.));
+	}
 }
 }
 
@@ -265,13 +345,16 @@ get_entry_number (const char *name)
 static void
 print_timeout (int timeout, int offset, int second_stage)
 {
-  /* NOTE: Do not remove the trailing space characters.
- They are required to clear the line.  */
-  char 

Re: gettext: normal_menu.c

2009-11-27 Thread Carles Pina i Estany

Hi,

On Nov/28/2009, Carles Pina i Estany wrote:

 Find attached the patch and ChangeLog for normal_menu.c

new version with two improvements:
a) uses grubcharwidth to calculate the width of the string (for the
characters that doesn't have any width, etc.). This has to be consistent
with GRUB_TERM_WIDTH.

b) ensure that strings that doesn't fit in one line will be correctly
handled and indented. This can happen in languages where the separator
is not a space. I've checked if there is some unicode character to say
word separator and I didn't find, so it adds the newline wherever is
fine for the terminal width.

Question: getstringwidth should be in normal/menu_text.c like it's now
or in kern/term.c like grub_getchardwidth.

Some suggestions before committing?

Thanks,

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


___
Grub-devel mailing list
Grub-devel@gnu.org
http://lists.gnu.org/mailman/listinfo/grub-devel


Re: gettext: normal_menu.c

2009-11-27 Thread Carles Pina i Estany

Hi,

On Nov/28/2009, Carles Pina i Estany wrote:
 
 Hi,
 
 On Nov/28/2009, Carles Pina i Estany wrote:
 
  Find attached the patch and ChangeLog for normal_menu.c
 
 new version with two improvements:

really, I should not be sending mails on Friday too late. Now is
attached.

-- 
Carles Pina i Estany
http://pinux.info
=== modified file 'normal/menu_text.c'
--- normal/menu_text.c	2009-11-23 20:59:24 +
+++ normal/menu_text.c	2009-11-28 01:18:23 +
@@ -39,7 +39,8 @@ static grub_uint8_t grub_color_menu_high
 void
 grub_wait_after_message (void)
 {
-  grub_printf (\nPress any key to continue...);
+  grub_putchar('\n');
+  grub_printf (_(Press any key to continue...));
   (void) grub_getkey ();
   grub_putchar ('\n');
 }
@@ -81,28 +82,128 @@ draw_border (void)
 }
 
 static void
+print_spaces (int number_spaces)
+{
+  int i;
+  for (i = 0; i  number_spaces; i++)
+grub_putchar(' ');
+}
+
+static void
+grub_print_ucs4 (const grub_uint32_t* str, const grub_uint32_t* last_position)
+{
+  while (str  last_position)
+{
+  grub_putcode(*str);
+  str++;
+}
+}
+
+static grub_ssize_t
+getstringwidth(grub_uint32_t* str, const grub_uint32_t* last_position)
+{
+  grub_ssize_t width = 0;
+
+  while (str  last_position)
+{
+  width += grub_getcharwidth(*str);
+  str++;
+}
+return width;
+}
+
+static void
+print_message_indented(const char* msg)
+{
+  const int line_len = GRUB_TERM_WIDTH - grub_getcharwidth('m') * 15;
+
+  grub_uint32_t *unicode_msg;
+
+  grub_ssize_t msg_len = grub_strlen(msg);
+
+  unicode_msg = grub_malloc (msg_len * sizeof(*unicode_msg));
+
+  msg_len = grub_utf8_to_ucs4 (unicode_msg, msg_len,
+(grub_uint8_t *) msg, -1, 0);
+
+  if (! unicode_msg)
+{
+  grub_printf(print_message_indented ERROR1: %s,msg);
+  return;
+}
+
+  if (msg_len  0)
+{
+  grub_printf(print_message_indented ERROR2: %s,msg);
+  grub_free (unicode_msg);
+  return;
+}
+  
+  const grub_uint32_t* last_position = unicode_msg + msg_len;
+  
+  grub_uint32_t* current_position = unicode_msg;
+
+  grub_uint32_t* next_new_line = unicode_msg;
+
+  while (current_position  last_position)
+{
+  next_new_line = (grub_uint32_t*)last_position;
+
+  while (getstringwidth(current_position,next_new_line)  line_len 
+  	|| (*next_new_line != ' '  next_new_line  current_position 
+	   next_new_line != last_position))
+{  
+  next_new_line--;
+}
+ 
+  if (next_new_line == current_position)
+{
+  next_new_line = (next_new_line + line_len  last_position) ? 
+   (grub_uint32_t*)last_position :
+		   next_new_line + line_len;
+}
+
+  print_spaces(6);
+  grub_print_ucs4(current_position,next_new_line);
+  grub_putchar('\n');
+  
+  next_new_line++;
+  current_position = next_new_line;
+}
+  grub_free(unicode_msg);
+}
+
+static void
 print_message (int nested, int edit)
 {
   grub_setcolorstate (GRUB_TERM_COLOR_NORMAL);
-
+  
+  grub_putchar ('\n');
   if (edit)
 {
-  grub_printf (\n\
-  Minimum Emacs-like screen editing is supported. TAB lists\n\
-  completions. Press Ctrl-x to boot, Ctrl-c for a command-line\n\
-  or ESC to return menu.);
+  print_message_indented(_(Minimum Emacs-like screen editing is \
+supported. TAB lists completions. Press Ctrl-x to boot, Ctrl-c for a \
+command-line or ESC to return menu.));
 }
   else
 {
-  grub_printf (_(\n\
-  Use the %C and %C keys to select which entry is highlighted.\n),
-		   (grub_uint32_t) GRUB_TERM_DISP_UP, (grub_uint32_t) GRUB_TERM_DISP_DOWN);
-  grub_printf (\
-  Press enter to boot the selected OS, \'e\' to edit the\n\
-  commands before booting or \'c\' for a command-line.);
+  const char *msg = _(Use the %C and %C keys to select which \
+entry is highlighted.);
+  char* msg_translated = grub_malloc(sizeof(char) * grub_strlen(msg) + 1);
+  
+  grub_sprintf(msg_translated, msg, (grub_uint32_t) GRUB_TERM_DISP_UP, (grub_uint32_t) GRUB_TERM_DISP_DOWN);
+  print_message_indented(msg_translated);
+
+  grub_free (msg_translated);
+
+  print_message_indented(_(Press enter to boot the selected OS, \
+\'e\' to edit the commands before booting or \'c\' for a command-line.));
+  
   if (nested)
-	grub_printf (\n\
-  ESC to return previous menu.);
+{
+	  grub_printf (\n);
+  grub_printf(_(ESC to return previous menu.));
+	}
 }
 }
 
@@ -265,13 +366,16 @@ get_entry_number (const char *name)
 static void
 print_timeout (int timeout, int offset, int second_stage)
 {
-  /* NOTE: Do not remove the trailing space characters.
- They are required to clear the line.  */
-  char *msg =The highlighted entry will be booted automatically in %ds.;
+  const char *msg = _(The highlighted entry will be booted automatically in %ds.);
+  const int