2006-09-22 Marco Gerards <marco@gnu.org>

	* normal/script.c (grub_script_create_cmdmenu): Skip leading
	newlines.

diff --git a/ChangeLog b/ChangeLog
index e8ccd3bdd1bb09acda12d849b4bad7c4c47e6118..2882cd84005d78e0f0f24516ec2fb01fb048f30b 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,8 @@
+2006-09-22  Marco Gerards  <marco@gnu.org>
+
+       * normal/script.c (grub_script_create_cmdmenu): Skip leading
+       newlines.
+
 2006-09-22  Marco Gerards  <marco@gnu.org>
 
        * commands/echo.c: New file.

diff --git a/normal/script.c b/normal/script.c
index dc4c2f12aaf322cf63da57396e5c33f3dec7808e..2873f25ff7273df62dec66165534ead022d42a85 100644 (file)
--- a/normal/script.c
+++ b/normal/script.c
@@ -214,6 +214,11 @@ grub_script_create_cmdmenu (struct grub_parser_param *state,
   struct grub_script_cmd_menuentry *cmd;
   int i;
 
+  /* Skip leading newlines to make the sourcecode better readable when
+     using the editor.  */
+  while (*sourcecode == '\n')
+    sourcecode++;
+
   /* Having trailing returns can some some annoying conflicts, remove
      them.  XXX: Can the parser be improved to handle this?  */
   for (i = grub_strlen (sourcecode) - 1; i > 0; i--)
@@ -226,6 +231,7 @@ grub_script_create_cmdmenu (struct grub_parser_param *state,
   cmd = grub_script_malloc (state, sizeof (*cmd));
   cmd->cmd.exec = grub_script_execute_menuentry;
   cmd->cmd.next = 0;
+  /* XXX: Check if this memory is properly free'ed.  */
   cmd->sourcecode = sourcecode;
   cmd->title = title;
   cmd->options = options;