]> git.proxmox.com Git - mirror_edk2.git/blobdiff - ShellPkg/Application/Shell/ShellEnvVar.c
ShellPkg: Apply uncrustify changes
[mirror_edk2.git] / ShellPkg / Application / Shell / ShellEnvVar.c
index 04fed3bf9141b02ae74b7e8d5f0d91e599b46eb3..b97cfe91b96bd10593758cbe2681e1d72a08c815 100644 (file)
@@ -14,7 +14,7 @@
 //\r
 // The list is used to cache the environment variables.\r
 //\r
-ENV_VAR_LIST                   gShellEnvVarList;\r
+ENV_VAR_LIST  gShellEnvVarList;\r
 \r
 /**\r
   Reports whether an environment variable is Volatile or Non-Volatile.\r
@@ -27,8 +27,8 @@ ENV_VAR_LIST                   gShellEnvVarList;
 **/\r
 EFI_STATUS\r
 IsVolatileEnv (\r
-  IN CONST CHAR16 *EnvVarName,\r
-  OUT BOOLEAN     *Volatile\r
+  IN CONST CHAR16  *EnvVarName,\r
+  OUT BOOLEAN      *Volatile\r
   )\r
 {\r
   EFI_STATUS  Status;\r
@@ -38,29 +38,35 @@ IsVolatileEnv (
 \r
   ASSERT (Volatile != NULL);\r
 \r
-  Size = 0;\r
+  Size   = 0;\r
   Buffer = NULL;\r
 \r
   //\r
   // get the variable\r
   //\r
-  Status = gRT->GetVariable((CHAR16*)EnvVarName,\r
-                            &gShellVariableGuid,\r
-                            &Attribs,\r
-                            &Size,\r
-                            Buffer);\r
+  Status = gRT->GetVariable (\r
+                  (CHAR16 *)EnvVarName,\r
+                  &gShellVariableGuid,\r
+                  &Attribs,\r
+                  &Size,\r
+                  Buffer\r
+                  );\r
   if (Status == EFI_BUFFER_TOO_SMALL) {\r
-    Buffer = AllocateZeroPool(Size);\r
+    Buffer = AllocateZeroPool (Size);\r
     if (Buffer == NULL) {\r
       return EFI_OUT_OF_RESOURCES;\r
     }\r
-    Status = gRT->GetVariable((CHAR16*)EnvVarName,\r
-                              &gShellVariableGuid,\r
-                              &Attribs,\r
-                              &Size,\r
-                              Buffer);\r
-    FreePool(Buffer);\r
+\r
+    Status = gRT->GetVariable (\r
+                    (CHAR16 *)EnvVarName,\r
+                    &gShellVariableGuid,\r
+                    &Attribs,\r
+                    &Size,\r
+                    Buffer\r
+                    );\r
+    FreePool (Buffer);\r
   }\r
+\r
   //\r
   // not found means volatile\r
   //\r
@@ -68,6 +74,7 @@ IsVolatileEnv (
     *Volatile = TRUE;\r
     return EFI_SUCCESS;\r
   }\r
+\r
   if (EFI_ERROR (Status)) {\r
     return Status;\r
   }\r
@@ -75,7 +82,7 @@ IsVolatileEnv (
   //\r
   // check for the Non Volatile bit\r
   //\r
-  *Volatile = !(BOOLEAN) ((Attribs & EFI_VARIABLE_NON_VOLATILE) == EFI_VARIABLE_NON_VOLATILE);\r
+  *Volatile = !(BOOLEAN)((Attribs & EFI_VARIABLE_NON_VOLATILE) == EFI_VARIABLE_NON_VOLATILE);\r
   return EFI_SUCCESS;\r
 }\r
 \r
@@ -85,30 +92,33 @@ IsVolatileEnv (
   @param[in] List               The pointer to pointer to list.\r
 **/\r
 VOID\r
-FreeEnvironmentVariableList(\r
-  IN LIST_ENTRY *List\r
+FreeEnvironmentVariableList (\r
+  IN LIST_ENTRY  *List\r
   )\r
 {\r
-  ENV_VAR_LIST *Node;\r
+  ENV_VAR_LIST  *Node;\r
 \r
   ASSERT (List != NULL);\r
   if (List == NULL) {\r
     return;\r
   }\r
 \r
-  for ( Node = (ENV_VAR_LIST*)GetFirstNode(List)\r
-      ; !IsListEmpty(List)\r
-      ; Node = (ENV_VAR_LIST*)GetFirstNode(List)\r
-     ){\r
-    ASSERT(Node != NULL);\r
-    RemoveEntryList(&Node->Link);\r
+  for ( Node = (ENV_VAR_LIST *)GetFirstNode (List)\r
+        ; !IsListEmpty (List)\r
+        ; Node = (ENV_VAR_LIST *)GetFirstNode (List)\r
+        )\r
+  {\r
+    ASSERT (Node != NULL);\r
+    RemoveEntryList (&Node->Link);\r
     if (Node->Key != NULL) {\r
-      FreePool(Node->Key);\r
+      FreePool (Node->Key);\r
     }\r
+\r
     if (Node->Val != NULL) {\r
-      FreePool(Node->Val);\r
+      FreePool (Node->Val);\r
     }\r
-    FreePool(Node);\r
+\r
+    FreePool (Node);\r
   }\r
 }\r
 \r
@@ -121,18 +131,18 @@ FreeEnvironmentVariableList(
   @retval EFI_SUCCESS           the list was created successfully.\r
 **/\r
 EFI_STATUS\r
-GetEnvironmentVariableList(\r
-  IN OUT LIST_ENTRY *ListHead\r
+GetEnvironmentVariableList (\r
+  IN OUT LIST_ENTRY  *ListHead\r
   )\r
 {\r
-  CHAR16            *VariableName;\r
-  UINTN             NameSize;\r
-  UINTN             NameBufferSize;\r
-  EFI_STATUS        Status;\r
-  EFI_GUID          Guid;\r
-  UINTN             ValSize;\r
-  UINTN             ValBufferSize;\r
-  ENV_VAR_LIST      *VarList;\r
+  CHAR16        *VariableName;\r
+  UINTN         NameSize;\r
+  UINTN         NameBufferSize;\r
+  EFI_STATUS    Status;\r
+  EFI_GUID      Guid;\r
+  UINTN         ValSize;\r
+  UINTN         ValBufferSize;\r
+  ENV_VAR_LIST  *VarList;\r
 \r
   if (ListHead == NULL) {\r
     return (EFI_INVALID_PARAMETER);\r
@@ -140,34 +150,36 @@ GetEnvironmentVariableList(
 \r
   Status = EFI_SUCCESS;\r
 \r
-  ValBufferSize = INIT_DATA_BUFFER_SIZE;\r
+  ValBufferSize  = INIT_DATA_BUFFER_SIZE;\r
   NameBufferSize = INIT_NAME_BUFFER_SIZE;\r
-  VariableName = AllocateZeroPool(NameBufferSize);\r
+  VariableName   = AllocateZeroPool (NameBufferSize);\r
   if (VariableName == NULL) {\r
     return (EFI_OUT_OF_RESOURCES);\r
   }\r
+\r
   *VariableName = CHAR_NULL;\r
 \r
-  while (!EFI_ERROR(Status)) {\r
+  while (!EFI_ERROR (Status)) {\r
     NameSize = NameBufferSize;\r
-    Status = gRT->GetNextVariableName(&NameSize, VariableName, &Guid);\r
-    if (Status == EFI_NOT_FOUND){\r
+    Status   = gRT->GetNextVariableName (&NameSize, VariableName, &Guid);\r
+    if (Status == EFI_NOT_FOUND) {\r
       Status = EFI_SUCCESS;\r
       break;\r
     } else if (Status == EFI_BUFFER_TOO_SMALL) {\r
       NameBufferSize = NameSize > NameBufferSize * 2 ? NameSize : NameBufferSize * 2;\r
-      SHELL_FREE_NON_NULL(VariableName);\r
-      VariableName = AllocateZeroPool(NameBufferSize);\r
+      SHELL_FREE_NON_NULL (VariableName);\r
+      VariableName = AllocateZeroPool (NameBufferSize);\r
       if (VariableName == NULL) {\r
         Status = EFI_OUT_OF_RESOURCES;\r
         break;\r
       }\r
+\r
       NameSize = NameBufferSize;\r
-      Status = gRT->GetNextVariableName(&NameSize, VariableName, &Guid);\r
+      Status   = gRT->GetNextVariableName (&NameSize, VariableName, &Guid);\r
     }\r
 \r
-    if (!EFI_ERROR(Status) && CompareGuid(&Guid, &gShellVariableGuid)){\r
-      VarList = AllocateZeroPool(sizeof(ENV_VAR_LIST));\r
+    if (!EFI_ERROR (Status) && CompareGuid (&Guid, &gShellVariableGuid)) {\r
+      VarList = AllocateZeroPool (sizeof (ENV_VAR_LIST));\r
       if (VarList == NULL) {\r
         Status = EFI_OUT_OF_RESOURCES;\r
       } else {\r
@@ -177,12 +189,13 @@ GetEnvironmentVariableList(
         //\r
         VarList->Val = AllocateZeroPool (ValSize + sizeof (CHAR16));\r
         if (VarList->Val == NULL) {\r
-            SHELL_FREE_NON_NULL(VarList);\r
-            Status = EFI_OUT_OF_RESOURCES;\r
-            break;\r
+          SHELL_FREE_NON_NULL (VarList);\r
+          Status = EFI_OUT_OF_RESOURCES;\r
+          break;\r
         }\r
-        Status = SHELL_GET_ENVIRONMENT_VARIABLE_AND_ATTRIBUTES(VariableName, &VarList->Atts, &ValSize, VarList->Val);\r
-        if (Status == EFI_BUFFER_TOO_SMALL){\r
+\r
+        Status = SHELL_GET_ENVIRONMENT_VARIABLE_AND_ATTRIBUTES (VariableName, &VarList->Atts, &ValSize, VarList->Val);\r
+        if (Status == EFI_BUFFER_TOO_SMALL) {\r
           ValBufferSize = ValSize > ValBufferSize * 2 ? ValSize : ValBufferSize * 2;\r
           SHELL_FREE_NON_NULL (VarList->Val);\r
           //\r
@@ -190,34 +203,36 @@ GetEnvironmentVariableList(
           //\r
           VarList->Val = AllocateZeroPool (ValBufferSize + sizeof (CHAR16));\r
           if (VarList->Val == NULL) {\r
-            SHELL_FREE_NON_NULL(VarList);\r
+            SHELL_FREE_NON_NULL (VarList);\r
             Status = EFI_OUT_OF_RESOURCES;\r
             break;\r
           }\r
 \r
           ValSize = ValBufferSize;\r
-          Status = SHELL_GET_ENVIRONMENT_VARIABLE_AND_ATTRIBUTES(VariableName, &VarList->Atts, &ValSize, VarList->Val);\r
+          Status  = SHELL_GET_ENVIRONMENT_VARIABLE_AND_ATTRIBUTES (VariableName, &VarList->Atts, &ValSize, VarList->Val);\r
         }\r
-        if (!EFI_ERROR(Status)) {\r
-          VarList->Key = AllocateCopyPool(StrSize(VariableName), VariableName);\r
+\r
+        if (!EFI_ERROR (Status)) {\r
+          VarList->Key = AllocateCopyPool (StrSize (VariableName), VariableName);\r
           if (VarList->Key == NULL) {\r
-            SHELL_FREE_NON_NULL(VarList->Val);\r
-            SHELL_FREE_NON_NULL(VarList);\r
+            SHELL_FREE_NON_NULL (VarList->Val);\r
+            SHELL_FREE_NON_NULL (VarList);\r
             Status = EFI_OUT_OF_RESOURCES;\r
           } else {\r
-            InsertTailList(ListHead, &VarList->Link);\r
+            InsertTailList (ListHead, &VarList->Link);\r
           }\r
         } else {\r
-          SHELL_FREE_NON_NULL(VarList->Val);\r
-          SHELL_FREE_NON_NULL(VarList);\r
+          SHELL_FREE_NON_NULL (VarList->Val);\r
+          SHELL_FREE_NON_NULL (VarList);\r
         }\r
       } // if (VarList == NULL) ... else ...\r
     } // compare guid\r
   } // while\r
+\r
   SHELL_FREE_NON_NULL (VariableName);\r
 \r
-  if (EFI_ERROR(Status)) {\r
-    FreeEnvironmentVariableList(ListHead);\r
+  if (EFI_ERROR (Status)) {\r
+    FreeEnvironmentVariableList (ListHead);\r
   }\r
 \r
   return (Status);\r
@@ -236,51 +251,56 @@ GetEnvironmentVariableList(
   @retval EFI_SUCCESS           the list was Set successfully.\r
 **/\r
 EFI_STATUS\r
-SetEnvironmentVariableList(\r
-  IN LIST_ENTRY *ListHead\r
+SetEnvironmentVariableList (\r
+  IN LIST_ENTRY  *ListHead\r
   )\r
 {\r
-  ENV_VAR_LIST      VarList;\r
-  ENV_VAR_LIST      *Node;\r
-  EFI_STATUS        Status;\r
-  UINTN             Size;\r
+  ENV_VAR_LIST  VarList;\r
+  ENV_VAR_LIST  *Node;\r
+  EFI_STATUS    Status;\r
+  UINTN         Size;\r
 \r
-  InitializeListHead(&VarList.Link);\r
+  InitializeListHead (&VarList.Link);\r
 \r
   //\r
   // Delete all the current environment variables\r
   //\r
-  Status = GetEnvironmentVariableList(&VarList.Link);\r
-  ASSERT_EFI_ERROR(Status);\r
-\r
-  for ( Node = (ENV_VAR_LIST*)GetFirstNode(&VarList.Link)\r
-      ; !IsNull(&VarList.Link, &Node->Link)\r
-      ; Node = (ENV_VAR_LIST*)GetNextNode(&VarList.Link, &Node->Link)\r
-     ){\r
+  Status = GetEnvironmentVariableList (&VarList.Link);\r
+  ASSERT_EFI_ERROR (Status);\r
+\r
+  for ( Node = (ENV_VAR_LIST *)GetFirstNode (&VarList.Link)\r
+        ; !IsNull (&VarList.Link, &Node->Link)\r
+        ; Node = (ENV_VAR_LIST *)GetNextNode (&VarList.Link, &Node->Link)\r
+        )\r
+  {\r
     if (Node->Key != NULL) {\r
-      Status = SHELL_DELETE_ENVIRONMENT_VARIABLE(Node->Key);\r
+      Status = SHELL_DELETE_ENVIRONMENT_VARIABLE (Node->Key);\r
     }\r
-    ASSERT_EFI_ERROR(Status);\r
+\r
+    ASSERT_EFI_ERROR (Status);\r
   }\r
 \r
-  FreeEnvironmentVariableList(&VarList.Link);\r
+  FreeEnvironmentVariableList (&VarList.Link);\r
 \r
   //\r
   // set all the variables from the list\r
   //\r
-  for ( Node = (ENV_VAR_LIST*)GetFirstNode(ListHead)\r
-      ; !IsNull(ListHead, &Node->Link)\r
-      ; Node = (ENV_VAR_LIST*)GetNextNode(ListHead, &Node->Link)\r
-     ){\r
+  for ( Node = (ENV_VAR_LIST *)GetFirstNode (ListHead)\r
+        ; !IsNull (ListHead, &Node->Link)\r
+        ; Node = (ENV_VAR_LIST *)GetNextNode (ListHead, &Node->Link)\r
+        )\r
+  {\r
     Size = StrSize (Node->Val) - sizeof (CHAR16);\r
     if (Node->Atts & EFI_VARIABLE_NON_VOLATILE) {\r
-      Status = SHELL_SET_ENVIRONMENT_VARIABLE_NV(Node->Key, Size, Node->Val);\r
+      Status = SHELL_SET_ENVIRONMENT_VARIABLE_NV (Node->Key, Size, Node->Val);\r
     } else {\r
       Status = SHELL_SET_ENVIRONMENT_VARIABLE_V (Node->Key, Size, Node->Val);\r
     }\r
-    ASSERT_EFI_ERROR(Status);\r
+\r
+    ASSERT_EFI_ERROR (Status);\r
   }\r
-  FreeEnvironmentVariableList(ListHead);\r
+\r
+  FreeEnvironmentVariableList (ListHead);\r
 \r
   return (Status);\r
 }\r
@@ -299,8 +319,8 @@ SetEnvironmentVariableList(
   @sa SetEnvironmentVariableList\r
 **/\r
 EFI_STATUS\r
-SetEnvironmentVariables(\r
-  IN CONST CHAR16 **Environment\r
+SetEnvironmentVariables (\r
+  IN CONST CHAR16  **Environment\r
   )\r
 {\r
   CONST CHAR16  *CurrentString;\r
@@ -318,61 +338,65 @@ SetEnvironmentVariables(
   // Build a list identical to the ones used for get/set list functions above\r
   //\r
   for ( CurrentCount = 0\r
-      ;\r
-      ; CurrentCount++\r
-     ){\r
+        ;\r
+        ; CurrentCount++\r
+        )\r
+  {\r
     CurrentString = Environment[CurrentCount];\r
     if (CurrentString == NULL) {\r
       break;\r
     }\r
-    ASSERT(StrStr(CurrentString, L"=") != NULL);\r
-    Node = AllocateZeroPool(sizeof(ENV_VAR_LIST));\r
+\r
+    ASSERT (StrStr (CurrentString, L"=") != NULL);\r
+    Node = AllocateZeroPool (sizeof (ENV_VAR_LIST));\r
     if (Node == NULL) {\r
-      SetEnvironmentVariableList(&VarList->Link);\r
+      SetEnvironmentVariableList (&VarList->Link);\r
       return (EFI_OUT_OF_RESOURCES);\r
     }\r
 \r
-    Node->Key = AllocateZeroPool((StrStr(CurrentString, L"=") - CurrentString + 1) * sizeof(CHAR16));\r
+    Node->Key = AllocateZeroPool ((StrStr (CurrentString, L"=") - CurrentString + 1) * sizeof (CHAR16));\r
     if (Node->Key == NULL) {\r
-      SHELL_FREE_NON_NULL(Node);\r
-      SetEnvironmentVariableList(&VarList->Link);\r
+      SHELL_FREE_NON_NULL (Node);\r
+      SetEnvironmentVariableList (&VarList->Link);\r
       return (EFI_OUT_OF_RESOURCES);\r
     }\r
 \r
     //\r
     // Copy the string into the Key, leaving the last character allocated as NULL to terminate\r
     //\r
-    StrnCpyS( Node->Key,\r
-              StrStr(CurrentString, L"=") - CurrentString + 1,\r
-              CurrentString,\r
-              StrStr(CurrentString, L"=") - CurrentString\r
-              );\r
+    StrnCpyS (\r
+      Node->Key,\r
+      StrStr (CurrentString, L"=") - CurrentString + 1,\r
+      CurrentString,\r
+      StrStr (CurrentString, L"=") - CurrentString\r
+      );\r
 \r
     //\r
     // ValueSize = TotalSize - already removed size - size for '=' + size for terminator (the last 2 items cancel each other)\r
     //\r
-    Node->Val = AllocateCopyPool(StrSize(CurrentString) - StrSize(Node->Key), CurrentString + StrLen(Node->Key) + 1);\r
+    Node->Val = AllocateCopyPool (StrSize (CurrentString) - StrSize (Node->Key), CurrentString + StrLen (Node->Key) + 1);\r
     if (Node->Val == NULL) {\r
-      SHELL_FREE_NON_NULL(Node->Key);\r
-      SHELL_FREE_NON_NULL(Node);\r
-      SetEnvironmentVariableList(&VarList->Link);\r
+      SHELL_FREE_NON_NULL (Node->Key);\r
+      SHELL_FREE_NON_NULL (Node);\r
+      SetEnvironmentVariableList (&VarList->Link);\r
       return (EFI_OUT_OF_RESOURCES);\r
     }\r
 \r
     Node->Atts = EFI_VARIABLE_BOOTSERVICE_ACCESS;\r
 \r
     if (VarList == NULL) {\r
-      VarList = AllocateZeroPool(sizeof(ENV_VAR_LIST));\r
+      VarList = AllocateZeroPool (sizeof (ENV_VAR_LIST));\r
       if (VarList == NULL) {\r
-        SHELL_FREE_NON_NULL(Node->Key);\r
-        SHELL_FREE_NON_NULL(Node->Val);\r
-        SHELL_FREE_NON_NULL(Node);\r
+        SHELL_FREE_NON_NULL (Node->Key);\r
+        SHELL_FREE_NON_NULL (Node->Val);\r
+        SHELL_FREE_NON_NULL (Node);\r
         return (EFI_OUT_OF_RESOURCES);\r
       }\r
-      InitializeListHead(&VarList->Link);\r
+\r
+      InitializeListHead (&VarList->Link);\r
     }\r
-    InsertTailList(&VarList->Link, &Node->Link);\r
 \r
+    InsertTailList (&VarList->Link, &Node->Link);\r
   } // for loop\r
 \r
   //\r
@@ -380,7 +404,7 @@ SetEnvironmentVariables(
   // this function also frees the memory and deletes all pre-existing\r
   // shell-guid based environment variables.\r
   //\r
-  return (SetEnvironmentVariableList(&VarList->Link));\r
+  return (SetEnvironmentVariableList (&VarList->Link));\r
 }\r
 \r
 /**\r
@@ -400,28 +424,30 @@ SetEnvironmentVariables(
 **/\r
 EFI_STATUS\r
 ShellFindEnvVarInList (\r
-  IN  CONST CHAR16    *Key,\r
-  OUT CHAR16          **Value,\r
-  OUT UINTN           *ValueSize,\r
-  OUT UINT32          *Atts OPTIONAL\r
+  IN  CONST CHAR16  *Key,\r
+  OUT CHAR16        **Value,\r
+  OUT UINTN         *ValueSize,\r
+  OUT UINT32        *Atts OPTIONAL\r
   )\r
 {\r
-  ENV_VAR_LIST      *Node;\r
+  ENV_VAR_LIST  *Node;\r
 \r
-  if (Key == NULL || Value == NULL || ValueSize == NULL) {\r
+  if ((Key == NULL) || (Value == NULL) || (ValueSize == NULL)) {\r
     return SHELL_INVALID_PARAMETER;\r
   }\r
 \r
-  for ( Node = (ENV_VAR_LIST*)GetFirstNode(&gShellEnvVarList.Link)\r
-      ; !IsNull(&gShellEnvVarList.Link, &Node->Link)\r
-      ; Node = (ENV_VAR_LIST*)GetNextNode(&gShellEnvVarList.Link, &Node->Link)\r
-     ){\r
-    if (Node->Key != NULL && StrCmp(Key, Node->Key) == 0) {\r
-      *Value      = AllocateCopyPool(StrSize(Node->Val), Node->Val);\r
-      *ValueSize  = StrSize(Node->Val);\r
+  for ( Node = (ENV_VAR_LIST *)GetFirstNode (&gShellEnvVarList.Link)\r
+        ; !IsNull (&gShellEnvVarList.Link, &Node->Link)\r
+        ; Node = (ENV_VAR_LIST *)GetNextNode (&gShellEnvVarList.Link, &Node->Link)\r
+        )\r
+  {\r
+    if ((Node->Key != NULL) && (StrCmp (Key, Node->Key) == 0)) {\r
+      *Value     = AllocateCopyPool (StrSize (Node->Val), Node->Val);\r
+      *ValueSize = StrSize (Node->Val);\r
       if (Atts != NULL) {\r
         *Atts = Node->Atts;\r
       }\r
+\r
       return EFI_SUCCESS;\r
     }\r
   }\r
@@ -444,17 +470,17 @@ ShellFindEnvVarInList (
 **/\r
 EFI_STATUS\r
 ShellAddEnvVarToList (\r
-  IN CONST CHAR16     *Key,\r
-  IN CONST CHAR16     *Value,\r
-  IN UINTN            ValueSize,\r
-  IN UINT32           Atts\r
+  IN CONST CHAR16  *Key,\r
+  IN CONST CHAR16  *Value,\r
+  IN UINTN         ValueSize,\r
+  IN UINT32        Atts\r
   )\r
 {\r
-  ENV_VAR_LIST      *Node;\r
-  CHAR16            *LocalKey;\r
-  CHAR16            *LocalValue;\r
+  ENV_VAR_LIST  *Node;\r
+  CHAR16        *LocalKey;\r
+  CHAR16        *LocalValue;\r
 \r
-  if (Key == NULL || Value == NULL || ValueSize == 0) {\r
+  if ((Key == NULL) || (Value == NULL) || (ValueSize == 0)) {\r
     return EFI_INVALID_PARAMETER;\r
   }\r
 \r
@@ -466,14 +492,15 @@ ShellAddEnvVarToList (
   //\r
   // Update the variable value if it exists in gShellEnvVarList.\r
   //\r
-  for ( Node = (ENV_VAR_LIST*)GetFirstNode(&gShellEnvVarList.Link)\r
-      ; !IsNull(&gShellEnvVarList.Link, &Node->Link)\r
-      ; Node = (ENV_VAR_LIST*)GetNextNode(&gShellEnvVarList.Link, &Node->Link)\r
-     ){\r
-    if (Node->Key != NULL && StrCmp(Key, Node->Key) == 0) {\r
+  for ( Node = (ENV_VAR_LIST *)GetFirstNode (&gShellEnvVarList.Link)\r
+        ; !IsNull (&gShellEnvVarList.Link, &Node->Link)\r
+        ; Node = (ENV_VAR_LIST *)GetNextNode (&gShellEnvVarList.Link, &Node->Link)\r
+        )\r
+  {\r
+    if ((Node->Key != NULL) && (StrCmp (Key, Node->Key) == 0)) {\r
       Node->Atts = Atts;\r
-      SHELL_FREE_NON_NULL(Node->Val);\r
-      Node->Val  = LocalValue;\r
+      SHELL_FREE_NON_NULL (Node->Val);\r
+      Node->Val = LocalValue;\r
       return EFI_SUCCESS;\r
     }\r
   }\r
@@ -482,21 +509,23 @@ ShellAddEnvVarToList (
   // If the environment variable key doesn't exist in list just insert\r
   // a new node.\r
   //\r
-  LocalKey = AllocateCopyPool (StrSize(Key), Key);\r
+  LocalKey = AllocateCopyPool (StrSize (Key), Key);\r
   if (LocalKey == NULL) {\r
     FreePool (LocalValue);\r
     return EFI_OUT_OF_RESOURCES;\r
   }\r
-  Node = (ENV_VAR_LIST*)AllocateZeroPool (sizeof(ENV_VAR_LIST));\r
+\r
+  Node = (ENV_VAR_LIST *)AllocateZeroPool (sizeof (ENV_VAR_LIST));\r
   if (Node == NULL) {\r
     FreePool (LocalKey);\r
     FreePool (LocalValue);\r
     return EFI_OUT_OF_RESOURCES;\r
   }\r
-  Node->Key = LocalKey;\r
-  Node->Val = LocalValue;\r
+\r
+  Node->Key  = LocalKey;\r
+  Node->Val  = LocalValue;\r
   Node->Atts = Atts;\r
-  InsertTailList(&gShellEnvVarList.Link, &Node->Link);\r
+  InsertTailList (&gShellEnvVarList.Link, &Node->Link);\r
 \r
   return EFI_SUCCESS;\r
 }\r
@@ -512,24 +541,25 @@ ShellAddEnvVarToList (
 **/\r
 EFI_STATUS\r
 ShellRemvoeEnvVarFromList (\r
-  IN CONST CHAR16           *Key\r
+  IN CONST CHAR16  *Key\r
   )\r
 {\r
-  ENV_VAR_LIST      *Node;\r
+  ENV_VAR_LIST  *Node;\r
 \r
   if (Key == NULL) {\r
     return EFI_INVALID_PARAMETER;\r
   }\r
 \r
-  for ( Node = (ENV_VAR_LIST*)GetFirstNode(&gShellEnvVarList.Link)\r
-      ; !IsNull(&gShellEnvVarList.Link, &Node->Link)\r
-      ; Node = (ENV_VAR_LIST*)GetNextNode(&gShellEnvVarList.Link, &Node->Link)\r
-     ){\r
-    if (Node->Key != NULL && StrCmp(Key, Node->Key) == 0) {\r
-      SHELL_FREE_NON_NULL(Node->Key);\r
-      SHELL_FREE_NON_NULL(Node->Val);\r
-      RemoveEntryList(&Node->Link);\r
-      SHELL_FREE_NON_NULL(Node);\r
+  for ( Node = (ENV_VAR_LIST *)GetFirstNode (&gShellEnvVarList.Link)\r
+        ; !IsNull (&gShellEnvVarList.Link, &Node->Link)\r
+        ; Node = (ENV_VAR_LIST *)GetNextNode (&gShellEnvVarList.Link, &Node->Link)\r
+        )\r
+  {\r
+    if ((Node->Key != NULL) && (StrCmp (Key, Node->Key) == 0)) {\r
+      SHELL_FREE_NON_NULL (Node->Key);\r
+      SHELL_FREE_NON_NULL (Node->Val);\r
+      RemoveEntryList (&Node->Link);\r
+      SHELL_FREE_NON_NULL (Node);\r
       return EFI_SUCCESS;\r
     }\r
   }\r
@@ -547,9 +577,9 @@ ShellInitEnvVarList (
   VOID\r
   )\r
 {\r
-  EFI_STATUS    Status;\r
+  EFI_STATUS  Status;\r
 \r
-  InitializeListHead(&gShellEnvVarList.Link);\r
+  InitializeListHead (&gShellEnvVarList.Link);\r
   Status = GetEnvironmentVariableList (&gShellEnvVarList.Link);\r
 \r
   return Status;\r
@@ -565,8 +595,7 @@ ShellFreeEnvVarList (
   )\r
 {\r
   FreeEnvironmentVariableList (&gShellEnvVarList.Link);\r
-  InitializeListHead(&gShellEnvVarList.Link);\r
+  InitializeListHead (&gShellEnvVarList.Link);\r
 \r
   return;\r
 }\r
-\r