X-Git-Url: https://git.proxmox.com/?p=mirror_edk2.git;a=blobdiff_plain;f=MdeModulePkg%2FUniversal%2FSetupBrowserDxe%2FUi.c;h=3dea095f5fd14c312d4361481dac874256614786;hp=dacc472652dcfa6b8c249aa7f98995cc6b14983d;hb=1fcd34f8755c523ec687681c3c6d15ef4c86ccaa;hpb=ce6d12cce0e7abda92ca5b13b323ca0077be6ee5 diff --git a/MdeModulePkg/Universal/SetupBrowserDxe/Ui.c b/MdeModulePkg/Universal/SetupBrowserDxe/Ui.c index dacc472652..3dea095f5f 100644 --- a/MdeModulePkg/Universal/SetupBrowserDxe/Ui.c +++ b/MdeModulePkg/Universal/SetupBrowserDxe/Ui.c @@ -1,8 +1,8 @@ /** @file Utility functions for User Interface functions. -Copyright (c) 2004 - 2009, Intel Corporation -All rights reserved. This program and the accompanying materials +Copyright (c) 2004 - 2011, Intel Corporation. All rights reserved.
+This program and the accompanying materials are licensed and made available under the terms and conditions of the BSD License which accompanies this distribution. The full text of the license may be found at http://opensource.org/licenses/bsd-license.php @@ -12,7 +12,6 @@ WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED. **/ -#include "Ui.h" #include "Setup.h" LIST_ENTRY gMenuOption; @@ -342,7 +341,6 @@ RefreshForm ( UI_MENU_SELECTION *Selection; FORM_BROWSER_STATEMENT *Question; EFI_HII_CONFIG_ACCESS_PROTOCOL *ConfigAccess; - EFI_HII_VALUE *HiiValue; EFI_BROWSER_ACTION_REQUEST ActionRequest; if (gMenuRefreshHead != NULL) { @@ -385,31 +383,14 @@ RefreshForm ( ConfigAccess = Selection->FormSet->ConfigAccess; if (((Question->QuestionFlags & EFI_IFR_FLAG_CALLBACK) != 0) && (ConfigAccess != NULL)) { ActionRequest = EFI_BROWSER_ACTION_REQUEST_NONE; - - HiiValue = &Question->HiiValue; - if (HiiValue->Type == EFI_IFR_TYPE_STRING) { - // - // Create String in HII database for Configuration Driver to retrieve - // - HiiValue->Value.string = NewString ((CHAR16 *) Question->BufferValue, Selection->FormSet->HiiHandle); - } - Status = ConfigAccess->Callback ( ConfigAccess, EFI_BROWSER_ACTION_CHANGING, Question->QuestionId, - HiiValue->Type, - &HiiValue->Value, + Question->HiiValue.Type, + &Question->HiiValue.Value, &ActionRequest ); - - if (HiiValue->Type == EFI_IFR_TYPE_STRING) { - // - // Clean the String in HII Database - // - DeleteString (HiiValue->Value.string, Selection->FormSet->HiiHandle); - } - if (!EFI_ERROR (Status)) { switch (ActionRequest) { case EFI_BROWSER_ACTION_REQUEST_RESET: @@ -679,6 +660,7 @@ UiAddMenuOption ( **/ EFI_STATUS +EFIAPI CreateDialog ( IN UINTN NumberOfLines, IN BOOLEAN HotKey, @@ -979,6 +961,7 @@ CreateSharedPopUp ( **/ VOID +EFIAPI CreateMultiStringPopUp ( IN UINTN RequestedWidth, IN UINTN NumberOfLines, @@ -1028,7 +1011,7 @@ UpdateStatusBar ( ); mInputError = TRUE; } else { - gST->ConOut->SetAttribute (gST->ConOut, FIELD_TEXT_HIGHLIGHT); + gST->ConOut->SetAttribute (gST->ConOut, PcdGet8 (PcdBrowserFieldTextHighlightColor)); for (Index = 0; Index < (GetStringWidth (InputErrorMessage) - 2) / 2; Index++) { PrintAt (gScreenDimensions.LeftColumn + gPromptBlockWidth + Index, gScreenDimensions.BottomRow - 1, L" "); } @@ -1038,7 +1021,7 @@ UpdateStatusBar ( break; case NV_UPDATE_REQUIRED: - if (gClassOfVfr != FORMSET_CLASS_FRONT_PAGE) { + if ((gClassOfVfr & FORMSET_CLASS_FRONT_PAGE) != FORMSET_CLASS_FRONT_PAGE) { if (State) { gST->ConOut->SetAttribute (gST->ConOut, INFO_TEXT); PrintStringAt ( @@ -1050,7 +1033,7 @@ UpdateStatusBar ( gNvUpdateRequired = TRUE; } else { - gST->ConOut->SetAttribute (gST->ConOut, FIELD_TEXT_HIGHLIGHT); + gST->ConOut->SetAttribute (gST->ConOut, PcdGet8 (PcdBrowserFieldTextHighlightColor)); for (Index = 0; Index < (GetStringWidth (NvUpdateMessage) - 2) / 2; Index++) { PrintAt ( (gScreenDimensions.LeftColumn + gPromptBlockWidth + gOptionBlockWidth + Index), @@ -1133,7 +1116,7 @@ GetWidth ( Width -= SUBTITLE_INDENT; } - return Width; + return (UINT16) (Width - LEFT_SKIPPED_COLUMNS); } /** @@ -1669,6 +1652,7 @@ UiDisplayMenu ( UINT8 DigitUint8; UI_MENU_LIST *CurrentMenu; UI_MENU_LIST *MenuList; + FORM_BROWSER_FORM *RefForm; CopyMem (&LocalScreen, &gScreenDimensions, sizeof (EFI_SCREEN_DESCRIPTOR)); @@ -1690,10 +1674,11 @@ UiDisplayMenu ( NextMenuOption = NULL; PreviousMenuOption = NULL; SavedMenuOption = NULL; + RefForm = NULL; ZeroMem (&Key, sizeof (EFI_INPUT_KEY)); - if (gClassOfVfr == FORMSET_CLASS_FRONT_PAGE) { + if ((gClassOfVfr & FORMSET_CLASS_FRONT_PAGE) == FORMSET_CLASS_FRONT_PAGE){ TopRow = LocalScreen.TopRow + FRONT_PAGE_HEADER_HEIGHT + SCROLL_ARROW_HEIGHT; Row = LocalScreen.TopRow + FRONT_PAGE_HEADER_HEIGHT + SCROLL_ARROW_HEIGHT; } else { @@ -1701,7 +1686,7 @@ UiDisplayMenu ( Row = LocalScreen.TopRow + NONE_FRONT_PAGE_HEADER_HEIGHT + SCROLL_ARROW_HEIGHT; } - Col = LocalScreen.LeftColumn; + Col = LocalScreen.LeftColumn + LEFT_SKIPPED_COLUMNS; BottomRow = LocalScreen.BottomRow - STATUS_BAR_HEIGHT - FOOTER_HEIGHT - SCROLL_ARROW_HEIGHT - 1; Selection->TopRow = TopRow; @@ -1772,15 +1757,15 @@ UiDisplayMenu ( UpArrow = FALSE; Row = TopRow; - Temp = SkipValue; - Temp2 = SkipValue; + Temp = (UINTN) SkipValue; + Temp2 = (UINTN) SkipValue; ClearLines ( LocalScreen.LeftColumn, LocalScreen.RightColumn, TopRow - SCROLL_ARROW_HEIGHT, BottomRow + SCROLL_ARROW_HEIGHT, - FIELD_TEXT | FIELD_BACKGROUND + PcdGet8 (PcdBrowserFieldTextColor) | FIELD_BACKGROUND ); UiFreeRefreshList (); @@ -1801,13 +1786,27 @@ UiDisplayMenu ( gST->ConOut->SetAttribute (gST->ConOut, FIELD_TEXT_GRAYED | FIELD_BACKGROUND); } else { if (Statement->Operand == EFI_IFR_SUBTITLE_OP) { - gST->ConOut->SetAttribute (gST->ConOut, SUBTITLE_TEXT | FIELD_BACKGROUND); + gST->ConOut->SetAttribute (gST->ConOut, PcdGet8 (PcdBrowserSubtitleTextColor) | FIELD_BACKGROUND); } } Width = GetWidth (Statement, MenuOption->Handle); OriginalRow = Row; + if (Statement->Operand == EFI_IFR_REF_OP && + ((gClassOfVfr & FORMSET_CLASS_PLATFORM_SETUP) == FORMSET_CLASS_PLATFORM_SETUP) && + MenuOption->Col > 2) { + // + // Print Arrow for Goto button. + // + PrintAt ( + MenuOption->Col - 2, + Row, + L"%c", + GEOMETRICSHAPE_RIGHT_TRIANGLE + ); + } + for (Index = 0; GetLineByWidth (MenuOption->Description, Width, &Index, &OutputString) != 0x0000;) { if ((Temp == 0) && (Row <= BottomRow)) { PrintStringAt (MenuOption->Col, Row, OutputString); @@ -1830,7 +1829,6 @@ UiDisplayMenu ( Temp = 0; Row = OriginalRow; - gST->ConOut->SetAttribute (gST->ConOut, FIELD_TEXT | FIELD_BACKGROUND); Status = ProcessOptions (Selection, MenuOption, FALSE, &OptionString); if (EFI_ERROR (Status)) { // @@ -1878,7 +1876,11 @@ UiDisplayMenu ( MenuRefreshEntry->Selection = Selection; MenuRefreshEntry->CurrentColumn = MenuOption->OptCol; MenuRefreshEntry->CurrentRow = MenuOption->Row; - MenuRefreshEntry->CurrentAttribute = FIELD_TEXT | FIELD_BACKGROUND; + if (MenuOption->GrayOut) { + MenuRefreshEntry->CurrentAttribute = FIELD_TEXT_GRAYED | FIELD_BACKGROUND; + } else { + MenuRefreshEntry->CurrentAttribute = PcdGet8 (PcdBrowserFieldTextColor) | FIELD_BACKGROUND; + } gMenuRefreshHead = MenuRefreshEntry; } else { // @@ -1896,7 +1898,11 @@ UiDisplayMenu ( MenuRefreshEntry->Selection = Selection; MenuRefreshEntry->CurrentColumn = MenuOption->OptCol; MenuRefreshEntry->CurrentRow = MenuOption->Row; - MenuRefreshEntry->CurrentAttribute = FIELD_TEXT | FIELD_BACKGROUND; + if (MenuOption->GrayOut) { + MenuRefreshEntry->CurrentAttribute = FIELD_TEXT_GRAYED | FIELD_BACKGROUND; + } else { + MenuRefreshEntry->CurrentAttribute = PcdGet8 (PcdBrowserFieldTextColor) | FIELD_BACKGROUND; + } } } @@ -1978,6 +1984,7 @@ UiDisplayMenu ( Row = OriginalRow; FreePool (StringPtr); } + gST->ConOut->SetAttribute (gST->ConOut, PcdGet8 (PcdBrowserFieldTextColor) | FIELD_BACKGROUND); // // Need to handle the bottom of the display @@ -2011,7 +2018,7 @@ UiDisplayMenu ( L"%c", ARROW_UP ); - gST->ConOut->SetAttribute (gST->ConOut, FIELD_TEXT | FIELD_BACKGROUND); + gST->ConOut->SetAttribute (gST->ConOut, PcdGet8 (PcdBrowserFieldTextColor) | FIELD_BACKGROUND); } if (DownArrow) { @@ -2022,7 +2029,7 @@ UiDisplayMenu ( L"%c", ARROW_DOWN ); - gST->ConOut->SetAttribute (gST->ConOut, FIELD_TEXT | FIELD_BACKGROUND); + gST->ConOut->SetAttribute (gST->ConOut, PcdGet8 (PcdBrowserFieldTextColor) | FIELD_BACKGROUND); } MenuOption = NULL; @@ -2098,7 +2105,7 @@ UiDisplayMenu ( // gST->ConOut->SetCursorPosition (gST->ConOut, MenuOption->Col, MenuOption->Row); ProcessOptions (Selection, MenuOption, FALSE, &OptionString); - gST->ConOut->SetAttribute (gST->ConOut, FIELD_TEXT | FIELD_BACKGROUND); + gST->ConOut->SetAttribute (gST->ConOut, PcdGet8 (PcdBrowserFieldTextColor) | FIELD_BACKGROUND); if (OptionString != NULL) { if ((MenuOption->ThisTag->Operand == EFI_IFR_DATE_OP) || (MenuOption->ThisTag->Operand == EFI_IFR_TIME_OP) @@ -2142,7 +2149,7 @@ UiDisplayMenu ( if (MenuOption->GrayOut) { gST->ConOut->SetAttribute (gST->ConOut, FIELD_TEXT_GRAYED | FIELD_BACKGROUND); } else if (MenuOption->ThisTag->Operand == EFI_IFR_SUBTITLE_OP) { - gST->ConOut->SetAttribute (gST->ConOut, SUBTITLE_TEXT | FIELD_BACKGROUND); + gST->ConOut->SetAttribute (gST->ConOut, PcdGet8 (PcdBrowserSubtitleTextColor) | FIELD_BACKGROUND); } OriginalRow = MenuOption->Row; @@ -2163,7 +2170,7 @@ UiDisplayMenu ( } MenuOption->Row = OriginalRow; - gST->ConOut->SetAttribute (gST->ConOut, FIELD_TEXT | FIELD_BACKGROUND); + gST->ConOut->SetAttribute (gST->ConOut, PcdGet8 (PcdBrowserFieldTextColor) | FIELD_BACKGROUND); } } } @@ -2193,7 +2200,7 @@ UiDisplayMenu ( // // Set reverse attribute // - gST->ConOut->SetAttribute (gST->ConOut, FIELD_TEXT_HIGHLIGHT | FIELD_BACKGROUND_HIGHLIGHT); + gST->ConOut->SetAttribute (gST->ConOut, PcdGet8 (PcdBrowserFieldTextHighlightColor) | PcdGet8 (PcdBrowserFieldBackgroundHighlightColor)); gST->ConOut->SetCursorPosition (gST->ConOut, MenuOption->Col, MenuOption->Row); // @@ -2203,9 +2210,13 @@ UiDisplayMenu ( // if (gMenuRefreshHead != NULL) { for (MenuRefreshEntry = gMenuRefreshHead; MenuRefreshEntry != NULL; MenuRefreshEntry = MenuRefreshEntry->Next) { - MenuRefreshEntry->CurrentAttribute = FIELD_TEXT | FIELD_BACKGROUND; + if (MenuOption->GrayOut) { + MenuRefreshEntry->CurrentAttribute = FIELD_TEXT_GRAYED | FIELD_BACKGROUND; + } else { + MenuRefreshEntry->CurrentAttribute = PcdGet8 (PcdBrowserFieldTextColor) | FIELD_BACKGROUND; + } if (MenuRefreshEntry->MenuOption == MenuOption) { - MenuRefreshEntry->CurrentAttribute = FIELD_TEXT_HIGHLIGHT | FIELD_BACKGROUND_HIGHLIGHT; + MenuRefreshEntry->CurrentAttribute = PcdGet8 (PcdBrowserFieldTextHighlightColor) | PcdGet8 (PcdBrowserFieldBackgroundHighlightColor); } } } @@ -2277,7 +2288,7 @@ UiDisplayMenu ( // // Clear reverse attribute // - gST->ConOut->SetAttribute (gST->ConOut, FIELD_TEXT | FIELD_BACKGROUND); + gST->ConOut->SetAttribute (gST->ConOut, PcdGet8 (PcdBrowserFieldTextColor) | FIELD_BACKGROUND); } // // Repaint flag will be used when process CfUpdateHelpString, so restore its value @@ -2293,6 +2304,7 @@ UiDisplayMenu ( // // Don't print anything if it is a NULL help token // + ASSERT(MenuOption != NULL); if (MenuOption->ThisTag->Help == 0) { StringPtr = L"\0"; } else { @@ -2369,6 +2381,16 @@ UiDisplayMenu ( // case '+': case '-': + // + // If the screen has no menu items, and the user didn't select UiReset + // ignore the selection and go back to reading keys. + // + if(IsListEmpty (&gMenuOption)) { + ControlFlag = CfReadKey; + break; + } + + ASSERT(MenuOption != NULL); Statement = MenuOption->ThisTag; if ((Statement->Operand == EFI_IFR_DATE_OP) || (Statement->Operand == EFI_IFR_TIME_OP) @@ -2386,6 +2408,8 @@ UiDisplayMenu ( // Repaint = TRUE; NewLine = TRUE; + } else { + Selection->Action = UI_ACTION_REFRESH_FORM; } if (OptionString != NULL) { FreePool (OptionString); @@ -2403,7 +2427,17 @@ UiDisplayMenu ( break; case ' ': - if (gClassOfVfr != FORMSET_CLASS_FRONT_PAGE) { + if ((gClassOfVfr & FORMSET_CLASS_FRONT_PAGE) != FORMSET_CLASS_FRONT_PAGE) { + // + // If the screen has no menu items, and the user didn't select UiReset + // ignore the selection and go back to reading keys. + // + if(IsListEmpty (&gMenuOption)) { + ControlFlag = CfReadKey; + break; + } + + ASSERT(MenuOption != NULL); if (MenuOption->ThisTag->Operand == EFI_IFR_CHECKBOX_OP && !MenuOption->GrayOut) { ScreenOperation = UiSelect; } @@ -2475,11 +2509,9 @@ UiDisplayMenu ( case CfUiSelect: ControlFlag = CfCheckSelection; + ASSERT(MenuOption != NULL); Statement = MenuOption->ThisTag; - if ((Statement->Operand == EFI_IFR_TEXT_OP) || - (Statement->Operand == EFI_IFR_DATE_OP) || - (Statement->Operand == EFI_IFR_TIME_OP) || - (Statement->Operand == EFI_IFR_NUMERIC_OP && Statement->Step != 0)) { + if (Statement->Operand == EFI_IFR_TEXT_OP) { break; } @@ -2494,7 +2526,6 @@ UiDisplayMenu ( // // Goto another Hii Package list // - ControlFlag = CfCheckSelection; Selection->Action = UI_ACTION_REFRESH_FORMSET; StringPtr = GetToken (Statement->RefDevicePath, Selection->FormSet->HiiHandle); @@ -2508,6 +2539,7 @@ UiDisplayMenu ( } BufferSize = StrLen (StringPtr) / 2; DevicePath = AllocatePool (BufferSize); + ASSERT (DevicePath != NULL); // // Convert from Device Path String to DevicePath Buffer in the reverse order. @@ -2549,13 +2581,36 @@ UiDisplayMenu ( // // Goto another Formset, check for uncommitted data // - ControlFlag = CfCheckSelection; Selection->Action = UI_ACTION_REFRESH_FORMSET; CopyMem (&Selection->FormSetGuid, &Statement->RefFormSetId, sizeof (EFI_GUID)); Selection->FormId = Statement->RefFormId; Selection->QuestionId = Statement->RefQuestionId; } else if (Statement->RefFormId != 0) { + // + // Check whether target From is suppressed. + // + RefForm = IdToForm (Selection->FormSet, Statement->RefFormId); + + if ((RefForm != NULL) && (RefForm->SuppressExpression != NULL)) { + Status = EvaluateExpression (Selection->FormSet, RefForm, RefForm->SuppressExpression); + if (EFI_ERROR (Status)) { + return Status; + } + + if (RefForm->SuppressExpression->Result.Value.b) { + // + // Form is suppressed. + // + do { + CreateDialog (4, TRUE, 0, NULL, &Key, gEmptyString, gFormSuppress, gPressEnter, gEmptyString); + } while (Key.UnicodeChar != CHAR_CARRIAGE_RETURN); + + Repaint = TRUE; + break; + } + } + // // Goto another form inside this formset, // @@ -2626,9 +2681,9 @@ UiDisplayMenu ( Selection->Action = UI_ACTION_REFRESH_FORM; } - if (OptionString != NULL) { - FreePool (OptionString); - } + if (OptionString != NULL) { + FreePool (OptionString); + } break; } break; @@ -2663,7 +2718,7 @@ UiDisplayMenu ( break; } - if (gClassOfVfr == FORMSET_CLASS_FRONT_PAGE) { + if ((gClassOfVfr & FORMSET_CLASS_FRONT_PAGE) == FORMSET_CLASS_FRONT_PAGE) { // // We never exit FrontPage, so skip the ESC // @@ -2719,11 +2774,13 @@ UiDisplayMenu ( case CfUiLeft: ControlFlag = CfCheckSelection; + ASSERT(MenuOption != NULL); if ((MenuOption->ThisTag->Operand == EFI_IFR_DATE_OP) || (MenuOption->ThisTag->Operand == EFI_IFR_TIME_OP)) { if (MenuOption->Sequence != 0) { // // In the middle or tail of the Date/Time op-code set, go left. // + ASSERT(NewPos != NULL); NewPos = NewPos->BackLink; } } @@ -2731,11 +2788,13 @@ UiDisplayMenu ( case CfUiRight: ControlFlag = CfCheckSelection; + ASSERT(MenuOption != NULL); if ((MenuOption->ThisTag->Operand == EFI_IFR_DATE_OP) || (MenuOption->ThisTag->Operand == EFI_IFR_TIME_OP)) { if (MenuOption->Sequence != 2) { // // In the middle or tail of the Date/Time op-code set, go left. // + ASSERT(NewPos != NULL); NewPos = NewPos->ForwardLink; } } @@ -2746,6 +2805,7 @@ UiDisplayMenu ( SavedListEntry = TopOfScreen; + ASSERT(NewPos != NULL); if (NewPos->BackLink != &gMenuOption) { NewLine = TRUE; // @@ -2760,6 +2820,7 @@ UiDisplayMenu ( NewPos = NewPos->BackLink; } + Difference = MoveToNextStatement (TRUE, &NewPos); PreviousMenuOption = MENU_OPTION_FROM_LINK (NewPos); DistanceValue = PreviousMenuOption->Skip; @@ -2770,31 +2831,8 @@ UiDisplayMenu ( // checking can be done. // DistanceValue += AdjustDateAndTimePosition (TRUE, &NewPos); - - // - // Check the previous menu entry to see if it was a zero-length advance. If it was, - // don't worry about a redraw. - // - if ((INTN) MenuOption->Row - (INTN) DistanceValue < (INTN) TopRow) { - Repaint = TRUE; - TopOfScreen = NewPos; - } - - Difference = MoveToNextStatement (TRUE, &NewPos); - PreviousMenuOption = MENU_OPTION_FROM_LINK (NewPos); - DistanceValue += PreviousMenuOption->Skip; - - if ((INTN) MenuOption->Row - (INTN) DistanceValue < (INTN) TopRow) { - if (Difference > 0) { - // - // Previous focus MenuOption is above the TopOfScreen, so we need to scroll - // - TopOfScreen = NewPos; - Repaint = TRUE; - SkipValue = 0; - OldSkipValue = 0; - } - } + + ASSERT (MenuOption != NULL); if (Difference < 0) { // // We want to goto previous MenuOption, but finally we go down. @@ -2805,6 +2843,14 @@ UiDisplayMenu ( TopOfScreen = gMenuOption.ForwardLink; Repaint = TRUE; } + } else if ((INTN) MenuOption->Row - (INTN) DistanceValue - Difference < (INTN) TopRow) { + // + // Previous focus MenuOption is above the TopOfScreen, so we need to scroll + // + TopOfScreen = NewPos; + Repaint = TRUE; + SkipValue = 0; + OldSkipValue = 0; } // @@ -2832,6 +2878,7 @@ UiDisplayMenu ( case CfUiPageUp: ControlFlag = CfCheckSelection; + ASSERT(NewPos != NULL); if (NewPos->BackLink == &gMenuOption) { NewLine = FALSE; Repaint = FALSE; @@ -2886,6 +2933,7 @@ UiDisplayMenu ( case CfUiPageDown: ControlFlag = CfCheckSelection; + ASSERT (NewPos != NULL); if (NewPos->ForwardLink == &gMenuOption) { NewLine = FALSE; Repaint = FALSE; @@ -2946,14 +2994,15 @@ UiDisplayMenu ( MenuOption = MENU_OPTION_FROM_LINK (NewPos); NewLine = TRUE; NewPos = NewPos->ForwardLink; - NextMenuOption = MENU_OPTION_FROM_LINK (NewPos); - DistanceValue += NextMenuOption->Skip; DistanceValue += MoveToNextStatement (FALSE, &NewPos); + NextMenuOption = MENU_OPTION_FROM_LINK (NewPos); + // // An option might be multi-line, so we need to reflect that data in the overall skip value // - UpdateOptionSkipLines (Selection, NextMenuOption, &OptionString, SkipValue); + UpdateOptionSkipLines (Selection, NextMenuOption, &OptionString, (UINTN) SkipValue); + DistanceValue += NextMenuOption->Skip; Temp = MenuOption->Row + MenuOption->Skip + DistanceValue - 1; if ((MenuOption->Row + MenuOption->Skip == BottomRow + 1) && @@ -2995,26 +3044,13 @@ UiDisplayMenu ( // // If we have a remainder, skip that many more op-codes until we drain the remainder // - for (; - Difference >= (INTN) SavedMenuOption->Skip; - Difference = Difference - (INTN) SavedMenuOption->Skip - ) { + while (Difference >= (INTN) SavedMenuOption->Skip) { // // Since the Difference is greater than or equal to this op-code's skip value, skip it // + Difference = Difference - (INTN) SavedMenuOption->Skip; TopOfScreen = TopOfScreen->ForwardLink; SavedMenuOption = MENU_OPTION_FROM_LINK (TopOfScreen); - if (Difference < (INTN) SavedMenuOption->Skip) { - Difference = SavedMenuOption->Skip - Difference - 1; - break; - } else { - if (Difference == (INTN) SavedMenuOption->Skip) { - TopOfScreen = TopOfScreen->ForwardLink; - SavedMenuOption = MENU_OPTION_FROM_LINK (TopOfScreen); - Difference = SavedMenuOption->Skip - Difference; - break; - } - } } // // Since we will act on this op-code in the next routine, and increment the @@ -3047,6 +3083,8 @@ UiDisplayMenu ( } else { SkipValue++; } + } else if (SavedMenuOption->Skip == 1) { + SkipValue = 0; } else { SkipValue = 0; TopOfScreen = TopOfScreen->ForwardLink; @@ -3090,6 +3128,7 @@ UiDisplayMenu ( Status = SubmitForm (Selection->FormSet, Selection->Form); if (!EFI_ERROR (Status)) { + ASSERT(MenuOption != NULL); UpdateStatusBar (INPUT_ERROR, MenuOption->ThisTag->QuestionFlags, FALSE); UpdateStatusBar (NV_UPDATE_REQUIRED, MenuOption->ThisTag->QuestionFlags, FALSE); } else {