]> git.proxmox.com Git - mirror_edk2.git/blobdiff - MdeModulePkg/Library/BootMaintenanceManagerUiLib/UpdatePage.c
MdeModulePkg: Apply uncrustify changes
[mirror_edk2.git] / MdeModulePkg / Library / BootMaintenanceManagerUiLib / UpdatePage.c
index e2b444cc45e22810ef556b0f599e34cffcb4d5aa..ca81b7f352648c30538ff7a77dedac119123c70f 100644 (file)
@@ -29,13 +29,13 @@ CreateUpdateData (
   //\r
   // Create Hii Extend Label OpCode as the start opcode\r
   //\r
-  mStartLabel = (EFI_IFR_GUID_LABEL *) HiiCreateGuidOpCode (mStartOpCodeHandle, &gEfiIfrTianoGuid, NULL, sizeof (EFI_IFR_GUID_LABEL));\r
+  mStartLabel               = (EFI_IFR_GUID_LABEL *)HiiCreateGuidOpCode (mStartOpCodeHandle, &gEfiIfrTianoGuid, NULL, sizeof (EFI_IFR_GUID_LABEL));\r
   mStartLabel->ExtendOpCode = EFI_IFR_EXTEND_OP_LABEL;\r
 \r
   //\r
   // Create Hii Extend Label OpCode as the end opcode\r
   //\r
-  mEndLabel = (EFI_IFR_GUID_LABEL *) HiiCreateGuidOpCode (mEndOpCodeHandle, &gEfiIfrTianoGuid, NULL, sizeof (EFI_IFR_GUID_LABEL));\r
+  mEndLabel               = (EFI_IFR_GUID_LABEL *)HiiCreateGuidOpCode (mEndOpCodeHandle, &gEfiIfrTianoGuid, NULL, sizeof (EFI_IFR_GUID_LABEL));\r
   mEndLabel->ExtendOpCode = EFI_IFR_EXTEND_OP_LABEL;\r
   mEndLabel->Number       = LABEL_END;\r
 }\r
@@ -64,9 +64,8 @@ RefreshUpdateData (
   //\r
   // Create Hii Extend Label OpCode as the start opcode\r
   //\r
-  mStartLabel = (EFI_IFR_GUID_LABEL *) HiiCreateGuidOpCode (mStartOpCodeHandle, &gEfiIfrTianoGuid, NULL, sizeof (EFI_IFR_GUID_LABEL));\r
+  mStartLabel               = (EFI_IFR_GUID_LABEL *)HiiCreateGuidOpCode (mStartOpCodeHandle, &gEfiIfrTianoGuid, NULL, sizeof (EFI_IFR_GUID_LABEL));\r
   mStartLabel->ExtendOpCode = EFI_IFR_EXTEND_OP_LABEL;\r
-\r
 }\r
 \r
 /**\r
@@ -78,7 +77,7 @@ RefreshUpdateData (
 **/\r
 VOID\r
 UpdatePageStart (\r
-  IN BMM_CALLBACK_DATA                *CallbackData\r
+  IN BMM_CALLBACK_DATA  *CallbackData\r
   )\r
 {\r
   RefreshUpdateData ();\r
@@ -109,7 +108,7 @@ UpdatePageStart (
 **/\r
 VOID\r
 UpdatePageEnd (\r
-  IN BMM_CALLBACK_DATA                *CallbackData\r
+  IN BMM_CALLBACK_DATA  *CallbackData\r
   )\r
 {\r
   //\r
@@ -164,8 +163,8 @@ UpdatePageEnd (
 **/\r
 VOID\r
 CleanUpPage (\r
-  IN UINT16                           LabelId,\r
-  IN BMM_CALLBACK_DATA                *CallbackData\r
+  IN UINT16             LabelId,\r
+  IN BMM_CALLBACK_DATA  *CallbackData\r
   )\r
 {\r
   RefreshUpdateData ();\r
@@ -191,11 +190,11 @@ CleanUpPage (
 **/\r
 VOID\r
 UpdateConCOMPage (\r
-  IN BMM_CALLBACK_DATA                *CallbackData\r
+  IN BMM_CALLBACK_DATA  *CallbackData\r
   )\r
 {\r
-  BM_MENU_ENTRY       *NewMenuEntry;\r
-  UINT16              Index;\r
+  BM_MENU_ENTRY  *NewMenuEntry;\r
+  UINT16         Index;\r
 \r
   CallbackData->BmmAskSaveOrNot = TRUE;\r
 \r
@@ -210,14 +209,13 @@ UpdateConCOMPage (
       NewMenuEntry->DisplayStringToken,\r
       STRING_TOKEN (STR_NULL_STRING),\r
       EFI_IFR_FLAG_CALLBACK,\r
-      (UINT16) (TERMINAL_OPTION_OFFSET + Index)\r
+      (UINT16)(TERMINAL_OPTION_OFFSET + Index)\r
       );\r
   }\r
 \r
   UpdatePageEnd (CallbackData);\r
 }\r
 \r
-\r
 /**\r
   Create a list of boot option from global BootOptionMenu. It\r
   allow user to delete the boot option.\r
@@ -227,12 +225,12 @@ UpdateConCOMPage (
 **/\r
 VOID\r
 UpdateBootDelPage (\r
-  IN BMM_CALLBACK_DATA                *CallbackData\r
+  IN BMM_CALLBACK_DATA  *CallbackData\r
   )\r
 {\r
-  BM_MENU_ENTRY   *NewMenuEntry;\r
-  BM_LOAD_CONTEXT *NewLoadContext;\r
-  UINT16          Index;\r
+  BM_MENU_ENTRY    *NewMenuEntry;\r
+  BM_LOAD_CONTEXT  *NewLoadContext;\r
+  UINT16           Index;\r
 \r
   CallbackData->BmmAskSaveOrNot = TRUE;\r
 \r
@@ -240,8 +238,8 @@ UpdateBootDelPage (
 \r
   ASSERT (BootOptionMenu.MenuNumber <= (sizeof (CallbackData->BmmFakeNvData.BootOptionDel) / sizeof (CallbackData->BmmFakeNvData.BootOptionDel[0])));\r
   for (Index = 0; Index < BootOptionMenu.MenuNumber; Index++) {\r
-    NewMenuEntry    = BOpt_GetMenuEntry (&BootOptionMenu, Index);\r
-    NewLoadContext  = (BM_LOAD_CONTEXT *) NewMenuEntry->VariableContext;\r
+    NewMenuEntry   = BOpt_GetMenuEntry (&BootOptionMenu, Index);\r
+    NewLoadContext = (BM_LOAD_CONTEXT *)NewMenuEntry->VariableContext;\r
     if (NewLoadContext->IsLegacy) {\r
       continue;\r
     }\r
@@ -255,15 +253,15 @@ UpdateBootDelPage (
       // deleted, browser maintains old useless info. So clear this info here, and later update this info to browser\r
       // through HiiSetBrowserData function.\r
       //\r
-      CallbackData->BmmFakeNvData.BootOptionDel[Index] = FALSE;\r
+      CallbackData->BmmFakeNvData.BootOptionDel[Index]    = FALSE;\r
       CallbackData->BmmOldFakeNVData.BootOptionDel[Index] = FALSE;\r
     }\r
 \r
     HiiCreateCheckBoxOpCode (\r
       mStartOpCodeHandle,\r
-      (EFI_QUESTION_ID) (BOOT_OPTION_DEL_QUESTION_ID + Index),\r
+      (EFI_QUESTION_ID)(BOOT_OPTION_DEL_QUESTION_ID + Index),\r
       VARSTORE_ID_BOOT_MAINT,\r
-      (UINT16) (BOOT_OPTION_DEL_VAR_OFFSET + Index),\r
+      (UINT16)(BOOT_OPTION_DEL_VAR_OFFSET + Index),\r
       NewMenuEntry->DisplayStringToken,\r
       NewMenuEntry->HelpStringToken,\r
       EFI_IFR_FLAG_CALLBACK,\r
@@ -271,6 +269,7 @@ UpdateBootDelPage (
       NULL\r
       );\r
   }\r
+\r
   UpdatePageEnd (CallbackData);\r
 }\r
 \r
@@ -282,11 +281,11 @@ UpdateBootDelPage (
 **/\r
 VOID\r
 UpdateDrvAddHandlePage (\r
-  IN BMM_CALLBACK_DATA                *CallbackData\r
+  IN BMM_CALLBACK_DATA  *CallbackData\r
   )\r
 {\r
-  BM_MENU_ENTRY *NewMenuEntry;\r
-  UINT16        Index;\r
+  BM_MENU_ENTRY  *NewMenuEntry;\r
+  UINT16         Index;\r
 \r
   CallbackData->BmmAskSaveOrNot = FALSE;\r
 \r
@@ -301,7 +300,7 @@ UpdateDrvAddHandlePage (
       NewMenuEntry->DisplayStringToken,\r
       STRING_TOKEN (STR_NULL_STRING),\r
       EFI_IFR_FLAG_CALLBACK,\r
-      (UINT16) (HANDLE_OPTION_OFFSET + Index)\r
+      (UINT16)(HANDLE_OPTION_OFFSET + Index)\r
       );\r
   }\r
 \r
@@ -317,12 +316,12 @@ UpdateDrvAddHandlePage (
 **/\r
 VOID\r
 UpdateDrvDelPage (\r
-  IN BMM_CALLBACK_DATA                *CallbackData\r
+  IN BMM_CALLBACK_DATA  *CallbackData\r
   )\r
 {\r
-  BM_MENU_ENTRY   *NewMenuEntry;\r
-  BM_LOAD_CONTEXT *NewLoadContext;\r
-  UINT16          Index;\r
+  BM_MENU_ENTRY    *NewMenuEntry;\r
+  BM_LOAD_CONTEXT  *NewLoadContext;\r
+  UINT16           Index;\r
 \r
   CallbackData->BmmAskSaveOrNot = TRUE;\r
 \r
@@ -330,9 +329,9 @@ UpdateDrvDelPage (
 \r
   ASSERT (DriverOptionMenu.MenuNumber <= (sizeof (CallbackData->BmmFakeNvData.DriverOptionDel) / sizeof (CallbackData->BmmFakeNvData.DriverOptionDel[0])));\r
   for (Index = 0; Index < DriverOptionMenu.MenuNumber; Index++) {\r
-    NewMenuEntry            = BOpt_GetMenuEntry (&DriverOptionMenu, Index);\r
+    NewMenuEntry = BOpt_GetMenuEntry (&DriverOptionMenu, Index);\r
 \r
-    NewLoadContext          = (BM_LOAD_CONTEXT *) NewMenuEntry->VariableContext;\r
+    NewLoadContext          = (BM_LOAD_CONTEXT *)NewMenuEntry->VariableContext;\r
     NewLoadContext->Deleted = FALSE;\r
 \r
     if (CallbackData->BmmFakeNvData.DriverOptionDel[Index] && !CallbackData->BmmFakeNvData.DriverOptionDelMark[Index]) {\r
@@ -342,14 +341,15 @@ UpdateDrvDelPage (
       // deleted, browser maintains old useless info. So clear this info here, and later update this info to browser\r
       // through HiiSetBrowserData function.\r
       //\r
-      CallbackData->BmmFakeNvData.DriverOptionDel[Index] = FALSE;\r
+      CallbackData->BmmFakeNvData.DriverOptionDel[Index]    = FALSE;\r
       CallbackData->BmmOldFakeNVData.DriverOptionDel[Index] = FALSE;\r
     }\r
+\r
     HiiCreateCheckBoxOpCode (\r
       mStartOpCodeHandle,\r
-      (EFI_QUESTION_ID) (DRIVER_OPTION_DEL_QUESTION_ID + Index),\r
+      (EFI_QUESTION_ID)(DRIVER_OPTION_DEL_QUESTION_ID + Index),\r
       VARSTORE_ID_BOOT_MAINT,\r
-      (UINT16) (DRIVER_OPTION_DEL_VAR_OFFSET + Index),\r
+      (UINT16)(DRIVER_OPTION_DEL_VAR_OFFSET + Index),\r
       NewMenuEntry->DisplayStringToken,\r
       NewMenuEntry->HelpStringToken,\r
       EFI_IFR_FLAG_CALLBACK,\r
@@ -370,15 +370,15 @@ UpdateDrvDelPage (
 **/\r
 VOID\r
 UpdateDriverAddHandleDescPage (\r
-  IN BMM_CALLBACK_DATA                *CallbackData\r
+  IN BMM_CALLBACK_DATA  *CallbackData\r
   )\r
 {\r
-  BM_MENU_ENTRY *NewMenuEntry;\r
+  BM_MENU_ENTRY  *NewMenuEntry;\r
 \r
-  CallbackData->BmmFakeNvData.DriverAddActive          = 0x01;\r
-  CallbackData->BmmFakeNvData.DriverAddForceReconnect  = 0x00;\r
-  CallbackData->BmmAskSaveOrNot                        = TRUE;\r
-  NewMenuEntry = CallbackData->MenuEntry;\r
+  CallbackData->BmmFakeNvData.DriverAddActive         = 0x01;\r
+  CallbackData->BmmFakeNvData.DriverAddForceReconnect = 0x00;\r
+  CallbackData->BmmAskSaveOrNot                       = TRUE;\r
+  NewMenuEntry                                        = CallbackData->MenuEntry;\r
 \r
   UpdatePageStart (CallbackData);\r
 \r
@@ -392,7 +392,7 @@ UpdateDriverAddHandleDescPage (
 \r
   HiiCreateStringOpCode (\r
     mStartOpCodeHandle,\r
-    (EFI_QUESTION_ID) DRV_ADD_HANDLE_DESC_QUESTION_ID,\r
+    (EFI_QUESTION_ID)DRV_ADD_HANDLE_DESC_QUESTION_ID,\r
     VARSTORE_ID_BOOT_MAINT,\r
     DRV_ADD_HANDLE_DESC_VAR_OFFSET,\r
     STRING_TOKEN (STR_LOAD_OPTION_DESC),\r
@@ -406,7 +406,7 @@ UpdateDriverAddHandleDescPage (
 \r
   HiiCreateCheckBoxOpCode (\r
     mStartOpCodeHandle,\r
-    (EFI_QUESTION_ID) DRV_ADD_RECON_QUESTION_ID,\r
+    (EFI_QUESTION_ID)DRV_ADD_RECON_QUESTION_ID,\r
     VARSTORE_ID_BOOT_MAINT,\r
     DRV_ADD_RECON_VAR_OFFSET,\r
     STRING_TOKEN (STR_LOAD_OPTION_FORCE_RECON),\r
@@ -418,7 +418,7 @@ UpdateDriverAddHandleDescPage (
 \r
   HiiCreateStringOpCode (\r
     mStartOpCodeHandle,\r
-    (EFI_QUESTION_ID) DRIVER_ADD_OPTION_QUESTION_ID,\r
+    (EFI_QUESTION_ID)DRIVER_ADD_OPTION_QUESTION_ID,\r
     VARSTORE_ID_BOOT_MAINT,\r
     DRIVER_ADD_OPTION_VAR_OFFSET,\r
     STRING_TOKEN (STR_OPTIONAL_DATA),\r
@@ -443,20 +443,20 @@ UpdateDriverAddHandleDescPage (
 **/\r
 VOID\r
 UpdateConsolePage (\r
-  IN UINT16                           UpdatePageId,\r
-  IN BM_MENU_OPTION                   *ConsoleMenu,\r
-  IN BMM_CALLBACK_DATA                *CallbackData\r
+  IN UINT16             UpdatePageId,\r
+  IN BM_MENU_OPTION     *ConsoleMenu,\r
+  IN BMM_CALLBACK_DATA  *CallbackData\r
   )\r
 {\r
-  BM_MENU_ENTRY       *NewMenuEntry;\r
-  BM_CONSOLE_CONTEXT  *NewConsoleContext;\r
-  BM_TERMINAL_CONTEXT *NewTerminalContext;\r
-  UINT16              Index;\r
-  UINT16              Index2;\r
-  UINT8               CheckFlags;\r
-  UINT8               *ConsoleCheck;\r
-  EFI_QUESTION_ID     QuestionIdBase;\r
-  UINT16              VariableOffsetBase;\r
+  BM_MENU_ENTRY        *NewMenuEntry;\r
+  BM_CONSOLE_CONTEXT   *NewConsoleContext;\r
+  BM_TERMINAL_CONTEXT  *NewTerminalContext;\r
+  UINT16               Index;\r
+  UINT16               Index2;\r
+  UINT8                CheckFlags;\r
+  UINT8                *ConsoleCheck;\r
+  EFI_QUESTION_ID      QuestionIdBase;\r
+  UINT16               VariableOffsetBase;\r
 \r
   CallbackData->BmmAskSaveOrNot = TRUE;\r
 \r
@@ -467,42 +467,45 @@ UpdateConsolePage (
   VariableOffsetBase = 0;\r
 \r
   switch (UpdatePageId) {\r
-  case FORM_CON_IN_ID:\r
-    ConsoleCheck       = &CallbackData->BmmFakeNvData.ConsoleInCheck[0];\r
-    QuestionIdBase     = CON_IN_DEVICE_QUESTION_ID;\r
-    VariableOffsetBase = CON_IN_DEVICE_VAR_OFFSET;\r
-    break;\r
-\r
-  case FORM_CON_OUT_ID:\r
-    ConsoleCheck       = &CallbackData->BmmFakeNvData.ConsoleOutCheck[0];\r
-    QuestionIdBase     = CON_OUT_DEVICE_QUESTION_ID;\r
-    VariableOffsetBase = CON_OUT_DEVICE_VAR_OFFSET;\r
-    break;\r
-\r
-  case FORM_CON_ERR_ID:\r
-    ConsoleCheck       = &CallbackData->BmmFakeNvData.ConsoleErrCheck[0];\r
-    QuestionIdBase     = CON_ERR_DEVICE_QUESTION_ID;\r
-    VariableOffsetBase = CON_ERR_DEVICE_VAR_OFFSET;\r
-    break;\r
+    case FORM_CON_IN_ID:\r
+      ConsoleCheck       = &CallbackData->BmmFakeNvData.ConsoleInCheck[0];\r
+      QuestionIdBase     = CON_IN_DEVICE_QUESTION_ID;\r
+      VariableOffsetBase = CON_IN_DEVICE_VAR_OFFSET;\r
+      break;\r
+\r
+    case FORM_CON_OUT_ID:\r
+      ConsoleCheck       = &CallbackData->BmmFakeNvData.ConsoleOutCheck[0];\r
+      QuestionIdBase     = CON_OUT_DEVICE_QUESTION_ID;\r
+      VariableOffsetBase = CON_OUT_DEVICE_VAR_OFFSET;\r
+      break;\r
+\r
+    case FORM_CON_ERR_ID:\r
+      ConsoleCheck       = &CallbackData->BmmFakeNvData.ConsoleErrCheck[0];\r
+      QuestionIdBase     = CON_ERR_DEVICE_QUESTION_ID;\r
+      VariableOffsetBase = CON_ERR_DEVICE_VAR_OFFSET;\r
+      break;\r
   }\r
+\r
   ASSERT (ConsoleCheck != NULL);\r
 \r
   for (Index = 0; ((Index < ConsoleMenu->MenuNumber) && \\r
-       (Index < MAX_MENU_NUMBER)) ; Index++) {\r
-    CheckFlags = 0;\r
-    NewMenuEntry = BOpt_GetMenuEntry (ConsoleMenu, Index);\r
-    NewConsoleContext = (BM_CONSOLE_CONTEXT *) NewMenuEntry->VariableContext;\r
+                   (Index < MAX_MENU_NUMBER)); Index++)\r
+  {\r
+    CheckFlags        = 0;\r
+    NewMenuEntry      = BOpt_GetMenuEntry (ConsoleMenu, Index);\r
+    NewConsoleContext = (BM_CONSOLE_CONTEXT *)NewMenuEntry->VariableContext;\r
     if (NewConsoleContext->IsActive) {\r
-      CheckFlags |= EFI_IFR_CHECKBOX_DEFAULT;\r
+      CheckFlags         |= EFI_IFR_CHECKBOX_DEFAULT;\r
       ConsoleCheck[Index] = TRUE;\r
     } else {\r
       ConsoleCheck[Index] = FALSE;\r
     }\r
+\r
     HiiCreateCheckBoxOpCode (\r
       mStartOpCodeHandle,\r
-      (EFI_QUESTION_ID) (QuestionIdBase + Index),\r
+      (EFI_QUESTION_ID)(QuestionIdBase + Index),\r
       VARSTORE_ID_BOOT_MAINT,\r
-      (UINT16) (VariableOffsetBase + Index),\r
+      (UINT16)(VariableOffsetBase + Index),\r
       NewMenuEntry->DisplayStringToken,\r
       NewMenuEntry->HelpStringToken,\r
       EFI_IFR_FLAG_CALLBACK,\r
@@ -512,26 +515,29 @@ UpdateConsolePage (
   }\r
 \r
   for (Index2 = 0; ((Index2 < TerminalMenu.MenuNumber) && \\r
-       (Index2 < MAX_MENU_NUMBER)); Index2++) {\r
-    CheckFlags          = 0;\r
-    NewMenuEntry        = BOpt_GetMenuEntry (&TerminalMenu, Index2);\r
-    NewTerminalContext  = (BM_TERMINAL_CONTEXT *) NewMenuEntry->VariableContext;\r
+                    (Index2 < MAX_MENU_NUMBER)); Index2++)\r
+  {\r
+    CheckFlags         = 0;\r
+    NewMenuEntry       = BOpt_GetMenuEntry (&TerminalMenu, Index2);\r
+    NewTerminalContext = (BM_TERMINAL_CONTEXT *)NewMenuEntry->VariableContext;\r
 \r
     ASSERT (Index < MAX_MENU_NUMBER);\r
     if (((NewTerminalContext->IsConIn != 0) && (UpdatePageId == FORM_CON_IN_ID)) ||\r
         ((NewTerminalContext->IsConOut != 0) && (UpdatePageId == FORM_CON_OUT_ID)) ||\r
         ((NewTerminalContext->IsStdErr != 0) && (UpdatePageId == FORM_CON_ERR_ID))\r
-        ) {\r
-      CheckFlags |= EFI_IFR_CHECKBOX_DEFAULT;\r
+        )\r
+    {\r
+      CheckFlags         |= EFI_IFR_CHECKBOX_DEFAULT;\r
       ConsoleCheck[Index] = TRUE;\r
     } else {\r
       ConsoleCheck[Index] = FALSE;\r
     }\r
+\r
     HiiCreateCheckBoxOpCode (\r
       mStartOpCodeHandle,\r
-      (EFI_QUESTION_ID) (QuestionIdBase + Index),\r
+      (EFI_QUESTION_ID)(QuestionIdBase + Index),\r
       VARSTORE_ID_BOOT_MAINT,\r
-      (UINT16) (VariableOffsetBase + Index),\r
+      (UINT16)(VariableOffsetBase + Index),\r
       NewMenuEntry->DisplayStringToken,\r
       NewMenuEntry->HelpStringToken,\r
       EFI_IFR_FLAG_CALLBACK,\r
@@ -556,58 +562,60 @@ UpdateConsolePage (
 **/\r
 VOID\r
 UpdateOrderPage (\r
-  IN UINT16                           UpdatePageId,\r
-  IN BM_MENU_OPTION                   *OptionMenu,\r
-  IN BMM_CALLBACK_DATA                *CallbackData\r
+  IN UINT16             UpdatePageId,\r
+  IN BM_MENU_OPTION     *OptionMenu,\r
+  IN BMM_CALLBACK_DATA  *CallbackData\r
   )\r
 {\r
-  BM_MENU_ENTRY     *NewMenuEntry;\r
-  UINT16            Index;\r
-  UINT16            OptionIndex;\r
-  VOID              *OptionsOpCodeHandle;\r
-  BOOLEAN           BootOptionFound;\r
-  UINT32            *OptionOrder;\r
-  EFI_QUESTION_ID   QuestionId;\r
-  UINT16            VarOffset;\r
+  BM_MENU_ENTRY    *NewMenuEntry;\r
+  UINT16           Index;\r
+  UINT16           OptionIndex;\r
+  VOID             *OptionsOpCodeHandle;\r
+  BOOLEAN          BootOptionFound;\r
+  UINT32           *OptionOrder;\r
+  EFI_QUESTION_ID  QuestionId;\r
+  UINT16           VarOffset;\r
 \r
   CallbackData->BmmAskSaveOrNot = TRUE;\r
   UpdatePageStart (CallbackData);\r
 \r
   OptionOrder = NULL;\r
-  QuestionId = 0;\r
-  VarOffset = 0;\r
+  QuestionId  = 0;\r
+  VarOffset   = 0;\r
   switch (UpdatePageId) {\r
+    case FORM_BOOT_CHG_ID:\r
+      //\r
+      // If the BootOptionOrder in the BmmFakeNvData are same with the date in the BmmOldFakeNVData,\r
+      // means all Boot Options has been save in BootOptionMenu, we can get the date from the menu.\r
+      // else means browser maintains some uncommitted date which are not saved in BootOptionMenu,\r
+      // so we should not get the data from BootOptionMenu to show it.\r
+      //\r
+      if (CompareMem (CallbackData->BmmFakeNvData.BootOptionOrder, CallbackData->BmmOldFakeNVData.BootOptionOrder, sizeof (CallbackData->BmmFakeNvData.BootOptionOrder)) == 0) {\r
+        GetBootOrder (CallbackData);\r
+      }\r
 \r
-  case FORM_BOOT_CHG_ID:\r
-    //\r
-    // If the BootOptionOrder in the BmmFakeNvData are same with the date in the BmmOldFakeNVData,\r
-    // means all Boot Options has been save in BootOptionMenu, we can get the date from the menu.\r
-    // else means browser maintains some uncommitted date which are not saved in BootOptionMenu,\r
-    // so we should not get the data from BootOptionMenu to show it.\r
-    //\r
-    if (CompareMem (CallbackData->BmmFakeNvData.BootOptionOrder, CallbackData->BmmOldFakeNVData.BootOptionOrder, sizeof (CallbackData->BmmFakeNvData.BootOptionOrder)) == 0) {\r
-      GetBootOrder (CallbackData);\r
-    }\r
-    OptionOrder = CallbackData->BmmFakeNvData.BootOptionOrder;\r
-    QuestionId = BOOT_OPTION_ORDER_QUESTION_ID;\r
-    VarOffset = BOOT_OPTION_ORDER_VAR_OFFSET;\r
-    break;\r
+      OptionOrder = CallbackData->BmmFakeNvData.BootOptionOrder;\r
+      QuestionId  = BOOT_OPTION_ORDER_QUESTION_ID;\r
+      VarOffset   = BOOT_OPTION_ORDER_VAR_OFFSET;\r
+      break;\r
 \r
-  case FORM_DRV_CHG_ID:\r
-    //\r
-    // If the DriverOptionOrder in the BmmFakeNvData are same with the date in the BmmOldFakeNVData,\r
-    // means all Driver Options has been save in DriverOptionMenu, we can get the DriverOptionOrder from the menu.\r
-    // else means browser maintains some uncommitted date which are not saved in DriverOptionMenu,\r
-    // so we should not get the data from DriverOptionMenu to show it.\r
-    //\r
-    if (CompareMem (CallbackData->BmmFakeNvData.DriverOptionOrder, CallbackData->BmmOldFakeNVData.DriverOptionOrder, sizeof (CallbackData->BmmFakeNvData.DriverOptionOrder)) == 0) {\r
-      GetDriverOrder (CallbackData);\r
-    }\r
-    OptionOrder = CallbackData->BmmFakeNvData.DriverOptionOrder;\r
-    QuestionId = DRIVER_OPTION_ORDER_QUESTION_ID;\r
-    VarOffset = DRIVER_OPTION_ORDER_VAR_OFFSET;\r
-    break;\r
+    case FORM_DRV_CHG_ID:\r
+      //\r
+      // If the DriverOptionOrder in the BmmFakeNvData are same with the date in the BmmOldFakeNVData,\r
+      // means all Driver Options has been save in DriverOptionMenu, we can get the DriverOptionOrder from the menu.\r
+      // else means browser maintains some uncommitted date which are not saved in DriverOptionMenu,\r
+      // so we should not get the data from DriverOptionMenu to show it.\r
+      //\r
+      if (CompareMem (CallbackData->BmmFakeNvData.DriverOptionOrder, CallbackData->BmmOldFakeNVData.DriverOptionOrder, sizeof (CallbackData->BmmFakeNvData.DriverOptionOrder)) == 0) {\r
+        GetDriverOrder (CallbackData);\r
+      }\r
+\r
+      OptionOrder = CallbackData->BmmFakeNvData.DriverOptionOrder;\r
+      QuestionId  = DRIVER_OPTION_ORDER_QUESTION_ID;\r
+      VarOffset   = DRIVER_OPTION_ORDER_VAR_OFFSET;\r
+      break;\r
   }\r
+\r
   ASSERT (OptionOrder != NULL);\r
 \r
   OptionsOpCodeHandle = HiiAllocateOpCodeHandle ();\r
@@ -617,12 +625,13 @@ UpdateOrderPage (
   for (OptionIndex = 0; (OptionOrder[OptionIndex] != 0 && OptionIndex < MAX_MENU_NUMBER); OptionIndex++) {\r
     BootOptionFound = FALSE;\r
     for (Index = 0; Index < OptionMenu->MenuNumber; Index++) {\r
-      NewMenuEntry   = BOpt_GetMenuEntry (OptionMenu, Index);\r
-      if ((UINT32) (NewMenuEntry->OptionNumber + 1) == OptionOrder[OptionIndex]) {\r
+      NewMenuEntry = BOpt_GetMenuEntry (OptionMenu, Index);\r
+      if ((UINT32)(NewMenuEntry->OptionNumber + 1) == OptionOrder[OptionIndex]) {\r
         BootOptionFound = TRUE;\r
         break;\r
       }\r
     }\r
+\r
     if (BootOptionFound) {\r
       HiiCreateOneOfOptionOpCode (\r
         OptionsOpCodeHandle,\r
@@ -654,7 +663,6 @@ UpdateOrderPage (
   HiiFreeOpCodeHandle (OptionsOpCodeHandle);\r
 \r
   UpdatePageEnd (CallbackData);\r
-\r
 }\r
 \r
 /**\r
@@ -665,26 +673,26 @@ UpdateOrderPage (
 **/\r
 VOID\r
 UpdateConModePage (\r
-  IN BMM_CALLBACK_DATA                *CallbackData\r
+  IN BMM_CALLBACK_DATA  *CallbackData\r
   )\r
 {\r
-  UINTN                         Mode;\r
-  UINTN                         Index;\r
-  UINTN                         Col;\r
-  UINTN                         Row;\r
-  CHAR16                        ModeString[50];\r
-  CHAR16                        *PStr;\r
-  UINTN                         MaxMode;\r
-  UINTN                         ValidMode;\r
-  EFI_STRING_ID                 *ModeToken;\r
-  EFI_STATUS                    Status;\r
-  VOID                          *OptionsOpCodeHandle;\r
+  UINTN                            Mode;\r
+  UINTN                            Index;\r
+  UINTN                            Col;\r
+  UINTN                            Row;\r
+  CHAR16                           ModeString[50];\r
+  CHAR16                           *PStr;\r
+  UINTN                            MaxMode;\r
+  UINTN                            ValidMode;\r
+  EFI_STRING_ID                    *ModeToken;\r
+  EFI_STATUS                       Status;\r
+  VOID                             *OptionsOpCodeHandle;\r
   EFI_SIMPLE_TEXT_OUTPUT_PROTOCOL  *ConOut;\r
 \r
   ConOut    = gST->ConOut;\r
   Index     = 0;\r
   ValidMode = 0;\r
-  MaxMode   = (UINTN) (ConOut->Mode->MaxMode);\r
+  MaxMode   = (UINTN)(ConOut->Mode->MaxMode);\r
 \r
   CallbackData->BmmAskSaveOrNot = TRUE;\r
 \r
@@ -698,6 +706,7 @@ UpdateConModePage (
     if (EFI_ERROR (Status)) {\r
       continue;\r
     }\r
+\r
     ValidMode++;\r
   }\r
 \r
@@ -708,8 +717,8 @@ UpdateConModePage (
   OptionsOpCodeHandle = HiiAllocateOpCodeHandle ();\r
   ASSERT (OptionsOpCodeHandle != NULL);\r
 \r
-  ModeToken           = AllocateZeroPool (sizeof (EFI_STRING_ID) * ValidMode);\r
-  ASSERT(ModeToken != NULL);\r
+  ModeToken = AllocateZeroPool (sizeof (EFI_STRING_ID) * ValidMode);\r
+  ASSERT (ModeToken != NULL);\r
 \r
   //\r
   // Determin which mode should be the first entry in menu\r
@@ -730,7 +739,7 @@ UpdateConModePage (
     //\r
     UnicodeValueToStringS (ModeString, sizeof (ModeString), 0, Col, 0);\r
     PStr = &ModeString[0];\r
-    StrnCatS (PStr, ARRAY_SIZE (ModeString), L" x ", StrLen(L" x ") + 1);\r
+    StrnCatS (PStr, ARRAY_SIZE (ModeString), L" x ", StrLen (L" x ") + 1);\r
     PStr = PStr + StrLen (PStr);\r
     UnicodeValueToStringS (\r
       PStr,\r
@@ -748,7 +757,7 @@ UpdateConModePage (
         ModeToken[Index],\r
         EFI_IFR_OPTION_DEFAULT,\r
         EFI_IFR_TYPE_NUM_SIZE_16,\r
-        (UINT16) Mode\r
+        (UINT16)Mode\r
         );\r
     } else {\r
       HiiCreateOneOfOptionOpCode (\r
@@ -756,15 +765,16 @@ UpdateConModePage (
         ModeToken[Index],\r
         0,\r
         EFI_IFR_TYPE_NUM_SIZE_16,\r
-        (UINT16) Mode\r
+        (UINT16)Mode\r
         );\r
     }\r
+\r
     Index++;\r
   }\r
 \r
   HiiCreateOneOfOpCode (\r
     mStartOpCodeHandle,\r
-    (EFI_QUESTION_ID) CON_MODE_QUESTION_ID,\r
+    (EFI_QUESTION_ID)CON_MODE_QUESTION_ID,\r
     VARSTORE_ID_BOOT_MAINT,\r
     CON_MODE_VAR_OFFSET,\r
     STRING_TOKEN (STR_CON_MODE_SETUP),\r
@@ -781,46 +791,47 @@ UpdateConModePage (
   UpdatePageEnd (CallbackData);\r
 }\r
 \r
- /**\r
 Create the dynamic page which allows user to set the property such as Baud Rate, Data Bits,\r
 Parity, Stop Bits, Terminal Type.\r
+/**\r
+ Create the dynamic page which allows user to set the property such as Baud Rate, Data Bits,\r
+ Parity, Stop Bits, Terminal Type.\r
 \r
 @param CallbackData    The BMM context data.\r
+ @param CallbackData    The BMM context data.\r
 \r
 **/\r
 VOID\r
 UpdateTerminalPage (\r
-  IN BMM_CALLBACK_DATA                *CallbackData\r
+  IN BMM_CALLBACK_DATA  *CallbackData\r
   )\r
 {\r
-  UINT8               Index;\r
-  UINT8               CheckFlags;\r
-  BM_MENU_ENTRY       *NewMenuEntry;\r
-  VOID                *OptionsOpCodeHandle;\r
-  UINTN               CurrentTerminal;\r
+  UINT8          Index;\r
+  UINT8          CheckFlags;\r
+  BM_MENU_ENTRY  *NewMenuEntry;\r
+  VOID           *OptionsOpCodeHandle;\r
+  UINTN          CurrentTerminal;\r
 \r
   CallbackData->BmmAskSaveOrNot = TRUE;\r
 \r
   UpdatePageStart (CallbackData);\r
 \r
   CurrentTerminal = CallbackData->CurrentTerminal;\r
-  NewMenuEntry = BOpt_GetMenuEntry (\r
-                  &TerminalMenu,\r
-                  CurrentTerminal\r
-                  );\r
+  NewMenuEntry    = BOpt_GetMenuEntry (\r
+                      &TerminalMenu,\r
+                      CurrentTerminal\r
+                      );\r
 \r
   if (NewMenuEntry == NULL) {\r
-    return ;\r
+    return;\r
   }\r
 \r
   OptionsOpCodeHandle = HiiAllocateOpCodeHandle ();\r
   ASSERT (OptionsOpCodeHandle != NULL);\r
 \r
-  for (Index = 0; Index < sizeof (BaudRateList) / sizeof (BaudRateList [0]); Index++) {\r
+  for (Index = 0; Index < sizeof (BaudRateList) / sizeof (BaudRateList[0]); Index++) {\r
     CheckFlags = 0;\r
     if (BaudRateList[Index].Value == 115200) {\r
       CheckFlags |= EFI_IFR_OPTION_DEFAULT;\r
     }\r
+\r
     HiiCreateOneOfOptionOpCode (\r
       OptionsOpCodeHandle,\r
       BaudRateList[Index].StringToken,\r
@@ -832,9 +843,9 @@ UpdateTerminalPage (
 \r
   HiiCreateOneOfOpCode (\r
     mStartOpCodeHandle,\r
-    (EFI_QUESTION_ID) (COM_BAUD_RATE_QUESTION_ID + CurrentTerminal),\r
+    (EFI_QUESTION_ID)(COM_BAUD_RATE_QUESTION_ID + CurrentTerminal),\r
     VARSTORE_ID_BOOT_MAINT,\r
-    (UINT16) (COM_BAUD_RATE_VAR_OFFSET + CurrentTerminal),\r
+    (UINT16)(COM_BAUD_RATE_VAR_OFFSET + CurrentTerminal),\r
     STRING_TOKEN (STR_COM_BAUD_RATE),\r
     STRING_TOKEN (STR_COM_BAUD_RATE),\r
     EFI_IFR_FLAG_CALLBACK,\r
@@ -865,9 +876,9 @@ UpdateTerminalPage (
 \r
   HiiCreateOneOfOpCode (\r
     mStartOpCodeHandle,\r
-    (EFI_QUESTION_ID) (COM_DATA_RATE_QUESTION_ID + CurrentTerminal),\r
+    (EFI_QUESTION_ID)(COM_DATA_RATE_QUESTION_ID + CurrentTerminal),\r
     VARSTORE_ID_BOOT_MAINT,\r
-    (UINT16) (COM_DATA_RATE_VAR_OFFSET + CurrentTerminal),\r
+    (UINT16)(COM_DATA_RATE_VAR_OFFSET + CurrentTerminal),\r
     STRING_TOKEN (STR_COM_DATA_BITS),\r
     STRING_TOKEN (STR_COM_DATA_BITS),\r
     EFI_IFR_FLAG_CALLBACK,\r
@@ -897,9 +908,9 @@ UpdateTerminalPage (
 \r
   HiiCreateOneOfOpCode (\r
     mStartOpCodeHandle,\r
-    (EFI_QUESTION_ID) (COM_PARITY_QUESTION_ID + CurrentTerminal),\r
+    (EFI_QUESTION_ID)(COM_PARITY_QUESTION_ID + CurrentTerminal),\r
     VARSTORE_ID_BOOT_MAINT,\r
-    (UINT16) (COM_PARITY_VAR_OFFSET + CurrentTerminal),\r
+    (UINT16)(COM_PARITY_VAR_OFFSET + CurrentTerminal),\r
     STRING_TOKEN (STR_COM_PARITY),\r
     STRING_TOKEN (STR_COM_PARITY),\r
     EFI_IFR_FLAG_CALLBACK,\r
@@ -929,9 +940,9 @@ UpdateTerminalPage (
 \r
   HiiCreateOneOfOpCode (\r
     mStartOpCodeHandle,\r
-    (EFI_QUESTION_ID) (COM_STOP_BITS_QUESTION_ID + CurrentTerminal),\r
+    (EFI_QUESTION_ID)(COM_STOP_BITS_QUESTION_ID + CurrentTerminal),\r
     VARSTORE_ID_BOOT_MAINT,\r
-    (UINT16) (COM_STOP_BITS_VAR_OFFSET + CurrentTerminal),\r
+    (UINT16)(COM_STOP_BITS_VAR_OFFSET + CurrentTerminal),\r
     STRING_TOKEN (STR_COM_STOP_BITS),\r
     STRING_TOKEN (STR_COM_STOP_BITS),\r
     EFI_IFR_FLAG_CALLBACK,\r
@@ -952,7 +963,7 @@ UpdateTerminalPage (
 \r
     HiiCreateOneOfOptionOpCode (\r
       OptionsOpCodeHandle,\r
-      (EFI_STRING_ID) TerminalType[Index],\r
+      (EFI_STRING_ID)TerminalType[Index],\r
       CheckFlags,\r
       EFI_IFR_TYPE_NUM_SIZE_8,\r
       Index\r
@@ -961,9 +972,9 @@ UpdateTerminalPage (
 \r
   HiiCreateOneOfOpCode (\r
     mStartOpCodeHandle,\r
-    (EFI_QUESTION_ID) (COM_TERMINAL_QUESTION_ID + CurrentTerminal),\r
+    (EFI_QUESTION_ID)(COM_TERMINAL_QUESTION_ID + CurrentTerminal),\r
     VARSTORE_ID_BOOT_MAINT,\r
-    (UINT16) (COM_TERMINAL_VAR_OFFSET + CurrentTerminal),\r
+    (UINT16)(COM_TERMINAL_VAR_OFFSET + CurrentTerminal),\r
     STRING_TOKEN (STR_COM_TERMI_TYPE),\r
     STRING_TOKEN (STR_COM_TERMI_TYPE),\r
     EFI_IFR_FLAG_CALLBACK,\r
@@ -977,13 +988,14 @@ UpdateTerminalPage (
   ASSERT (OptionsOpCodeHandle != NULL);\r
 \r
   for (Index = 0; Index < ARRAY_SIZE (mFlowControlType); Index++) {\r
-  CheckFlags = 0;\r
+    CheckFlags = 0;\r
     if (Index == 0) {\r
       CheckFlags |= EFI_IFR_OPTION_DEFAULT;\r
     }\r
+\r
     HiiCreateOneOfOptionOpCode (\r
       OptionsOpCodeHandle,\r
-      (EFI_STRING_ID) mFlowControlType[Index],\r
+      (EFI_STRING_ID)mFlowControlType[Index],\r
       CheckFlags,\r
       EFI_IFR_TYPE_NUM_SIZE_8,\r
       mFlowControlValue[Index]\r
@@ -992,9 +1004,9 @@ UpdateTerminalPage (
 \r
   HiiCreateOneOfOpCode (\r
     mStartOpCodeHandle,\r
-    (EFI_QUESTION_ID) (COM_FLOWCONTROL_QUESTION_ID + CurrentTerminal),\r
+    (EFI_QUESTION_ID)(COM_FLOWCONTROL_QUESTION_ID + CurrentTerminal),\r
     VARSTORE_ID_BOOT_MAINT,\r
-    (UINT16) (COM_FLOWCONTROL_VAR_OFFSET + CurrentTerminal),\r
+    (UINT16)(COM_FLOWCONTROL_VAR_OFFSET + CurrentTerminal),\r
     STRING_TOKEN (STR_COM_FLOW_CONTROL),\r
     STRING_TOKEN (STR_COM_FLOW_CONTROL),\r
     EFI_IFR_FLAG_CALLBACK,\r
@@ -1017,20 +1029,21 @@ Update add boot/driver option page.
 \r
 **/\r
 VOID\r
-UpdateOptionPage(\r
-  IN   BMM_CALLBACK_DATA        *CallbackData,\r
-  IN   EFI_FORM_ID              FormId,\r
-  IN   EFI_DEVICE_PATH_PROTOCOL *DevicePath\r
+UpdateOptionPage (\r
+  IN   BMM_CALLBACK_DATA         *CallbackData,\r
+  IN   EFI_FORM_ID               FormId,\r
+  IN   EFI_DEVICE_PATH_PROTOCOL  *DevicePath\r
   )\r
 {\r
-  CHAR16                *String;\r
-  EFI_STRING_ID         StringToken;\r
+  CHAR16         *String;\r
+  EFI_STRING_ID  StringToken;\r
 \r
   String = NULL;\r
 \r
-  if (DevicePath != NULL){\r
-    String = ExtractFileNameFromDevicePath(DevicePath);\r
+  if (DevicePath != NULL) {\r
+    String = ExtractFileNameFromDevicePath (DevicePath);\r
   }\r
+\r
   if (String == NULL) {\r
     String = HiiGetString (CallbackData->BmmHiiHandle, STRING_TOKEN (STR_NULL_STRING), NULL);\r
     ASSERT (String != NULL);\r
@@ -1039,14 +1052,14 @@ UpdateOptionPage(
   StringToken = HiiSetString (CallbackData->BmmHiiHandle, 0, String, NULL);\r
   FreePool (String);\r
 \r
-  if(FormId == FORM_BOOT_ADD_ID){\r
+  if (FormId == FORM_BOOT_ADD_ID) {\r
     if (!CallbackData->BmmFakeNvData.BootOptionChanged) {\r
       ZeroMem (CallbackData->BmmFakeNvData.BootOptionalData, sizeof (CallbackData->BmmFakeNvData.BootOptionalData));\r
       ZeroMem (CallbackData->BmmFakeNvData.BootDescriptionData, sizeof (CallbackData->BmmFakeNvData.BootDescriptionData));\r
       ZeroMem (CallbackData->BmmOldFakeNVData.BootOptionalData, sizeof (CallbackData->BmmOldFakeNVData.BootOptionalData));\r
       ZeroMem (CallbackData->BmmOldFakeNVData.BootDescriptionData, sizeof (CallbackData->BmmOldFakeNVData.BootDescriptionData));\r
     }\r
-  } else if (FormId == FORM_DRV_ADD_FILE_ID){\r
+  } else if (FormId == FORM_DRV_ADD_FILE_ID) {\r
     if (!CallbackData->BmmFakeNvData.DriverOptionChanged) {\r
       ZeroMem (CallbackData->BmmFakeNvData.DriverOptionalData, sizeof (CallbackData->BmmFakeNvData.DriverOptionalData));\r
       ZeroMem (CallbackData->BmmFakeNvData.DriverDescriptionData, sizeof (CallbackData->BmmFakeNvData.DriverDescriptionData));\r
@@ -1055,7 +1068,7 @@ UpdateOptionPage(
     }\r
   }\r
 \r
-  RefreshUpdateData();\r
+  RefreshUpdateData ();\r
   mStartLabel->Number = FormId;\r
 \r
   HiiCreateSubTitleOpCode (\r
@@ -1070,8 +1083,8 @@ UpdateOptionPage(
     CallbackData->BmmHiiHandle,\r
     &mBootMaintGuid,\r
     FormId,\r
-    mStartOpCodeHandle,// Label FormId\r
-    mEndOpCodeHandle   // LABEL_END\r
+    mStartOpCodeHandle, // Label FormId\r
+    mEndOpCodeHandle    // LABEL_END\r
     );\r
 }\r
 \r
@@ -1085,34 +1098,34 @@ UpdateOptionPage(
 **/\r
 VOID\r
 UpdatePageBody (\r
-  IN UINT16                           UpdatePageId,\r
-  IN BMM_CALLBACK_DATA                *CallbackData\r
+  IN UINT16             UpdatePageId,\r
+  IN BMM_CALLBACK_DATA  *CallbackData\r
   )\r
 {\r
   CleanUpPage (UpdatePageId, CallbackData);\r
   switch (UpdatePageId) {\r
-  case FORM_CON_IN_ID:\r
-    UpdateConsolePage (UpdatePageId, &ConsoleInpMenu, CallbackData);\r
-    break;\r
+    case FORM_CON_IN_ID:\r
+      UpdateConsolePage (UpdatePageId, &ConsoleInpMenu, CallbackData);\r
+      break;\r
 \r
-  case FORM_CON_OUT_ID:\r
-    UpdateConsolePage (UpdatePageId, &ConsoleOutMenu, CallbackData);\r
-    break;\r
+    case FORM_CON_OUT_ID:\r
+      UpdateConsolePage (UpdatePageId, &ConsoleOutMenu, CallbackData);\r
+      break;\r
 \r
-  case FORM_CON_ERR_ID:\r
-    UpdateConsolePage (UpdatePageId, &ConsoleErrMenu, CallbackData);\r
-    break;\r
+    case FORM_CON_ERR_ID:\r
+      UpdateConsolePage (UpdatePageId, &ConsoleErrMenu, CallbackData);\r
+      break;\r
 \r
-  case FORM_BOOT_CHG_ID:\r
-    UpdateOrderPage (UpdatePageId, &BootOptionMenu, CallbackData);\r
-    break;\r
+    case FORM_BOOT_CHG_ID:\r
+      UpdateOrderPage (UpdatePageId, &BootOptionMenu, CallbackData);\r
+      break;\r
 \r
-  case FORM_DRV_CHG_ID:\r
-    UpdateOrderPage (UpdatePageId, &DriverOptionMenu, CallbackData);\r
-    break;\r
+    case FORM_DRV_CHG_ID:\r
+      UpdateOrderPage (UpdatePageId, &DriverOptionMenu, CallbackData);\r
+      break;\r
 \r
-  default:\r
-    break;\r
+    default:\r
+      break;\r
   }\r
 }\r
 \r
@@ -1125,8 +1138,8 @@ UpdatePageBody (
 **/\r
 VOID\r
 UpdatePageId (\r
-  BMM_CALLBACK_DATA              *Private,\r
-  UINT16                         NewPageId\r
+  BMM_CALLBACK_DATA  *Private,\r
+  UINT16             NewPageId\r
   )\r
 {\r
   if ((NewPageId < FILE_OPTION_OFFSET) && (NewPageId >= HANDLE_OPTION_OFFSET)) {\r
@@ -1144,7 +1157,7 @@ UpdatePageId (
   }\r
 \r
   if ((NewPageId > 0) && (NewPageId < MAXIMUM_FORM_ID)) {\r
-    Private->BmmPreviousPageId  = Private->BmmCurrentPageId;\r
-    Private->BmmCurrentPageId   = NewPageId;\r
+    Private->BmmPreviousPageId = Private->BmmCurrentPageId;\r
+    Private->BmmCurrentPageId  = NewPageId;\r
   }\r
 }\r