Simple menu: really avoid disabled entries syslinux-3.71-pre2
authorH. Peter Anvin <hpa@zytor.com>
Thu, 3 Jul 2008 01:34:49 +0000 (18:34 -0700)
committerH. Peter Anvin <hpa@zytor.com>
Thu, 3 Jul 2008 01:34:49 +0000 (18:34 -0700)
Really, really try to avoid stepping on disabled entries...

NEWS
com32/menu/menumain.c

diff --git a/NEWS b/NEWS
index 6df01f6..81ab3fa 100644 (file)
--- a/NEWS
+++ b/NEWS
@@ -2,6 +2,12 @@ Starting with 1.47, changes marked with SYSLINUX/PXELINUX/ISOLINUX
 apply to that specific program only; other changes apply to all of
 them.
 
 apply to that specific program only; other changes apply to all of
 them.
 
+Changes in 3.71:
+       * Workaround for a VESA BIOS which tries to make DOS system
+         calls(!!)
+       * Simple menu: fix navigation around disabled entries
+         (or at least try to...)
+
 Changes in 3.70:
        * PXELINUX: Support enhanced capabilities when running on top
          of gPXE (http://www.etherboot.org/).  In particular, support
 Changes in 3.70:
        * PXELINUX: Support enhanced capabilities when running on top
          of gPXE (http://www.etherboot.org/).  In particular, support
index f79d00e..01b4fb4 100644 (file)
@@ -747,13 +747,12 @@ run_menu(void)
   }
 
   while ( !done ) {
   }
 
   while ( !done ) {
-    if ( entry <= 0 ) {
+    if (entry <= 0) {
       entry = 0;
       while (entry < cm->nentries && is_disabled(cm->menu_entries[entry]))
        entry++;
     }
       entry = 0;
       while (entry < cm->nentries && is_disabled(cm->menu_entries[entry]))
        entry++;
     }
-
-    if ( entry >= cm->nentries ) {
+    if (entry >= cm->nentries) {
       entry = cm->nentries-1;
       while (entry > 0 && is_disabled(cm->menu_entries[entry]))
        entry--;
       entry = cm->nentries-1;
       while (entry > 0 && is_disabled(cm->menu_entries[entry]))
        entry--;
@@ -871,28 +870,23 @@ run_menu(void)
 
     case KEY_UP:
     case KEY_CTRL('P'):
 
     case KEY_UP:
     case KEY_CTRL('P'):
-      while (entry > 0 && entry-- && is_disabled(cm->menu_entries[entry])) {
-       if ( entry < top )
+      while (entry > 0) {
+       entry--;
+       if (entry < top)
          top -= MENU_ROWS;
          top -= MENU_ROWS;
-      }
-
-      if ( entry == 0 ) {
-        while (is_disabled(cm->menu_entries[entry]))
-          entry++;
+       if (!is_disabled(cm->menu_entries[entry]))
+         break;
       }
       break;
 
     case KEY_DOWN:
     case KEY_CTRL('N'):
       }
       break;
 
     case KEY_DOWN:
     case KEY_CTRL('N'):
-      while (entry < cm->nentries-1 && entry++ &&
-            is_disabled(cm->menu_entries[entry])) {
-       if ( entry >= top+MENU_ROWS )
+      while (entry < cm->nentries-1) {
+       entry++;
+       if (entry >= top+MENU_ROWS)
          top += MENU_ROWS;
          top += MENU_ROWS;
-      }
-
-      if ( entry >= cm->nentries-1 ) {
-        while (is_disabled(cm->menu_entries[entry]))
-          entry--;
+       if (!is_disabled(cm->menu_entries[entry]))
+         break;
       }
       break;
 
       }
       break;
 
@@ -902,6 +896,11 @@ run_menu(void)
     case '<':
       entry -= MENU_ROWS;
       top   -= MENU_ROWS;
     case '<':
       entry -= MENU_ROWS;
       top   -= MENU_ROWS;
+      while (entry > 0 && is_disabled(cm->menu_entries[entry])) {
+         entry--;
+         if (entry < top)
+           top -= MENU_ROWS;
+      }
       break;
 
     case KEY_PGDN:
       break;
 
     case KEY_PGDN:
@@ -911,20 +910,29 @@ run_menu(void)
     case ' ':
       entry += MENU_ROWS;
       top   += MENU_ROWS;
     case ' ':
       entry += MENU_ROWS;
       top   += MENU_ROWS;
+      while (entry < cm->nentries-1 && is_disabled(cm->menu_entries[entry])) {
+       entry++;
+       if (entry >= top+MENU_ROWS)
+         top += MENU_ROWS;
+      }
       break;
 
     case '-':
       break;
 
     case '-':
-      do {
+      while (entry > 0) {
        entry--;
        top--;
        entry--;
        top--;
-      } while (entry > 0 && is_disabled(cm->menu_entries[entry]));
+       if (!is_disabled(cm->menu_entries[entry]))
+         break;
+      }
       break;
 
     case '+':
       break;
 
     case '+':
-      do {
+      while (entry < cm->nentries-1) {
        entry++;
        top++;
        entry++;
        top++;
-      } while (entry < cm->nentries-1 && is_disabled(cm->menu_entries[entry]));
+       if (!is_disabled(cm->menu_entries[entry]))
+         break;
+      }
       break;
 
     case KEY_CTRL('A'):
       break;
 
     case KEY_CTRL('A'):