Remove the wrong @return doxygen comment.

[mirror_edk2.git] / MdeModulePkg / Universal / SetupBrowserDxe / IfrParse.c

diff --git a/MdeModulePkg/Universal/SetupBrowserDxe/IfrParse.c b/MdeModulePkg/Universal/SetupBrowserDxe/IfrParse.c
index 9108acb473ea6ded418216d2e23f1675ea62ee8d..7e5f6684ce39dce2c821d74635282a746b0a6edd 100644 (file)
--- a/MdeModulePkg/Universal/SetupBrowserDxe/IfrParse.c
+++ b/MdeModulePkg/Universal/SetupBrowserDxe/IfrParse.c
@@ -1,4 +1,6 @@
 /** @file
 /** @file

+Parser for IFR binary encoding.
+

 Copyright (c) 2007 - 2008, 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
 Copyright (c) 2007 - 2008, 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


@@ -8,15 +10,6 @@ http://opensource.org/licenses/bsd-license.php
 THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS,
 WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.
 
 THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS,
 WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.
 

-Module Name:
-
-  IfrParse.c
-
-Abstract:
-
-  Parser for IFR binary encoding.
-
-

 **/
 
 #include "Setup.h"
 **/
 
 #include "Setup.h"


@@ -337,7 +330,7 @@ InitializeRequestElement (
     StrLen = UnicodeSPrint (RequestElement, 30 * sizeof (CHAR16), L"&%s", Question->VariableName);
   }
 
     StrLen = UnicodeSPrint (RequestElement, 30 * sizeof (CHAR16), L"&%s", Question->VariableName);
   }
 

-  if ((Question->Operand == EFI_IFR_PASSWORD_OP) && (Question->QuestionFlags & EFI_IFR_FLAG_CALLBACK)) {
+  if ((Question->Operand == EFI_IFR_PASSWORD_OP) && ((Question->QuestionFlags & EFI_IFR_FLAG_CALLBACK) == EFI_IFR_FLAG_CALLBACK)) {

     //
     // Password with CALLBACK flag is stored in encoded format,
     // so don't need to append it to <ConfigRequest>
     //
     // Password with CALLBACK flag is stored in encoded format,
     // so don't need to append it to <ConfigRequest>


@@ -371,12 +364,10 @@ InitializeRequestElement (
 
 
 /**
 
 
 /**

-  Free resources of a Expression
+  Free resources of a Expression.

 
   @param  FormSet                Pointer of the Expression
 
 
   @param  FormSet                Pointer of the Expression
 

-  @return None.
-

 **/
 VOID
 DestroyExpression (
 **/
 VOID
 DestroyExpression (


@@ -402,12 +393,10 @@ DestroyExpression (
 
 
 /**
 
 
 /**

-  Free resources of a storage
+  Free resources of a storage.

 
   @param  Storage                Pointer of the storage
 
 
   @param  Storage                Pointer of the storage
 

-  @return None.
-

 **/
 VOID
 DestroyStorage (
 **/
 VOID
 DestroyStorage (


@@ -444,12 +433,10 @@ DestroyStorage (
 
 
 /**
 
 
 /**

-  Free resources of a Statement
+  Free resources of a Statement.

 
   @param  Statement              Pointer of the Statement
 
 
   @param  Statement              Pointer of the Statement
 

-  @return None.
-

 **/
 VOID
 DestroyStatement (
 **/
 VOID
 DestroyStatement (


@@ -511,11 +498,9 @@ DestroyStatement (
 
 
 /**
 
 
 /**

-  Free resources of a Form
+  Free resources of a Form.

 
 

-  @param  Form                   Pointer of the Form
-
-  @return None.
+  @param  Form                   Pointer of the Form.

 
 **/
 VOID
 
 **/
 VOID


@@ -557,12 +542,10 @@ DestroyForm (
 
 
 /**
 
 
 /**

-  Free resources allocated for a FormSet
+  Free resources allocated for a FormSet.

 
   @param  FormSet                Pointer of the FormSet
 
 
   @param  FormSet                Pointer of the FormSet
 

-  @return None.
-

 **/
 VOID
 DestroyFormSet (
 **/
 VOID
 DestroyFormSet (


@@ -657,8 +640,6 @@ IsExpressionOpCode (
   @param  NumberOfStatement      Number of Statemens(Questions)
   @param  NumberOfExpression     Number of Expression OpCodes
 
   @param  NumberOfStatement      Number of Statemens(Questions)
   @param  NumberOfExpression     Number of Expression OpCodes
 

-  @return None.
-

 **/
 VOID
 CountOpCodes (
 **/
 VOID
 CountOpCodes (


@@ -791,7 +772,7 @@ ParseOpCodes (
     //
     // If scope bit set, push onto scope stack
     //
     //
     // If scope bit set, push onto scope stack
     //

-    if (Scope) {
+    if (Scope != 0) {

       PushScope (Operand);
     }
 
       PushScope (Operand);
     }
 


@@ -1091,7 +1072,7 @@ ParseOpCodes (
       CurrentStatement = CreateStatement (OpCodeData, FormSet, CurrentForm);
       CurrentStatement->Flags = ((EFI_IFR_SUBTITLE *) OpCodeData)->Flags;
 
       CurrentStatement = CreateStatement (OpCodeData, FormSet, CurrentForm);
       CurrentStatement->Flags = ((EFI_IFR_SUBTITLE *) OpCodeData)->Flags;
 

-      if (Scope) {
+      if (Scope != 0) {

         mInScopeSubtitle = TRUE;
       }
       break;
         mInScopeSubtitle = TRUE;
       }
       break;


@@ -1187,7 +1168,7 @@ ParseOpCodes (
 
       InitializeRequestElement (FormSet, CurrentStatement);
 
 
       InitializeRequestElement (FormSet, CurrentStatement);
 

-      if ((Operand == EFI_IFR_ONE_OF_OP) && Scope) {
+      if ((Operand == EFI_IFR_ONE_OF_OP) && Scope != 0) {

         SuppressForOption = TRUE;
       }
       break;
         SuppressForOption = TRUE;
       }
       break;


@@ -1208,7 +1189,7 @@ ParseOpCodes (
       CurrentStatement->HiiValue.Type = EFI_IFR_TYPE_OTHER;
       CurrentStatement->BufferValue = AllocateZeroPool (CurrentStatement->StorageWidth);
 
       CurrentStatement->HiiValue.Type = EFI_IFR_TYPE_OTHER;
       CurrentStatement->BufferValue = AllocateZeroPool (CurrentStatement->StorageWidth);
 

-      if (Scope) {
+      if (Scope != 0) {

         SuppressForOption = TRUE;
       }
       break;
         SuppressForOption = TRUE;
       }
       break;


@@ -1320,7 +1301,7 @@ ParseOpCodes (
       //
       InsertTailList (&CurrentStatement->DefaultListHead, &CurrentDefault->Link);
 
       //
       InsertTailList (&CurrentStatement->DefaultListHead, &CurrentDefault->Link);
 

-      if (Scope) {
+      if (Scope != 0) {

         InScopeDefault = TRUE;
       }
       break;
         InScopeDefault = TRUE;
       }
       break;