ECC Cleanup: Update Doxygen comment

git-svn-id: https://edk2.svn.sourceforge.net/svnroot/edk2/trunk/edk2@6633 6f19259b-4bc3-4df7-8a09-765794883524

5 files changed, 27 insertions, 5 deletions

diff --git a/MdeModulePkg/Universal/SetupBrowserDxe/IfrParse.c b/MdeModulePkg/Universal/SetupBrowserDxe/IfrParse.c
index cde8d6f..fe029a3 100644
--- a/MdeModulePkg/Universal/SetupBrowserDxe/IfrParse.c
+++ b/MdeModulePkg/Universal/SetupBrowserDxe/IfrParse.c
@@ -89,6 +89,21 @@ CreateStatement (
   return Statement;

 }

 

+/**

+  Convert a numeric value to a Unicode String and insert it to String Package.

+  This string is used as the Unicode Name for the EFI Variable. This is to support

+  the deprecated vareqval opcode.

+  

+  @param FormSet        The FormSet.

+  @param Statement      The numeric question whose VarStoreInfo.VarName is the

+                        numeric value which is used to produce the Unicode Name

+                        for the EFI Variable.

+                        

+  If the Statement is NULL, the ASSERT.

+  If the opcode is not Numeric, then ASSERT.

+  

+  @retval EFI_SUCCESS The funtion always succeeds.

+**/

 EFI_STATUS

 UpdateCheckBoxStringToken (

   IN CONST FORM_BROWSER_FORMSET *FormSet,

@@ -115,6 +130,14 @@ UpdateCheckBoxStringToken (
   return EFI_SUCCESS;

 }

 

+/**

+  Check if the next opcode is the EFI_IFR_EXTEND_OP_VAREQNAME.

+  

+  @param OpCodeData     The current opcode.

+                        

+  @retval TRUE Yes.

+  @retval FALSE No.

+**/

 BOOLEAN

 IsNextOpCodeGuidedVarEqName (

   UINT8 *OpCodeData

diff --git a/MdeModulePkg/Universal/SetupBrowserDxe/Print.c b/MdeModulePkg/Universal/SetupBrowserDxe/Print.c
index d8b42f8..7bbe6d4 100644
--- a/MdeModulePkg/Universal/SetupBrowserDxe/Print.c
+++ b/MdeModulePkg/Universal/SetupBrowserDxe/Print.c
@@ -165,7 +165,7 @@ PrintInternal (
 UINTN

 ConsolePrint (

   IN CHAR16   *Fmt,

-  IN ...

+  ...

   )

 {

   VA_LIST Args;

@@ -253,7 +253,7 @@ UINTN
 PrintStringAt (

   IN UINTN     Column,

   IN UINTN     Row,

-  CHAR16       *String

+  IN CHAR16    *String

   )

 {

   return PrintAt (Column, Row, L"%s", String);

diff --git a/MdeModulePkg/Universal/SetupBrowserDxe/Setup.h b/MdeModulePkg/Universal/SetupBrowserDxe/Setup.h
index 0804d51..7366973 100644
--- a/MdeModulePkg/Universal/SetupBrowserDxe/Setup.h
+++ b/MdeModulePkg/Universal/SetupBrowserDxe/Setup.h
@@ -712,7 +712,6 @@ CreateSharedPopUp (
   @param  StringBuffer           The passed in pointer to the buffer which will

                                  hold the typed in string if HotKey is FALSE

   @param  KeyValue               The EFI_KEY value returned if HotKey is TRUE..

-  @param  String                 Pointer to the first string in the list

   @param  ...                    A series of (quantity == NumberOfLines) text

                                  strings which will be used to construct the dialog

                                  box

diff --git a/MdeModulePkg/Universal/SetupBrowserDxe/Ui.c b/MdeModulePkg/Universal/SetupBrowserDxe/Ui.c
index 639854e..9624374 100644
--- a/MdeModulePkg/Universal/SetupBrowserDxe/Ui.c
+++ b/MdeModulePkg/Universal/SetupBrowserDxe/Ui.c
@@ -348,7 +348,7 @@ RefreshForm (
       // Question value may be changed, need invoke its Callback()

       //

       ConfigAccess = Selection->FormSet->ConfigAccess;

-      if ((Question->QuestionFlags & EFI_IFR_FLAG_CALLBACK) && (ConfigAccess != NULL)) {

+      if (((Question->QuestionFlags & EFI_IFR_FLAG_CALLBACK) != 0) && (ConfigAccess != NULL)) {

         ActionRequest = EFI_BROWSER_ACTION_REQUEST_NONE;

 

         HiiValue = &Question->HiiValue;

diff --git a/MdeModulePkg/Universal/SetupBrowserDxe/Ui.h b/MdeModulePkg/Universal/SetupBrowserDxe/Ui.h
index 3244cd1..957a739 100644
--- a/MdeModulePkg/Universal/SetupBrowserDxe/Ui.h
+++ b/MdeModulePkg/Universal/SetupBrowserDxe/Ui.h
@@ -350,7 +350,7 @@ UiWaitForSingleEvent (
   Draw a pop up windows based on the dimension, number of lines and

   strings specified.

 

-  @param RequestedWidth  The width of the pop-up.

+  @param ScreenWidth     The width of the pop-up.

   @param NumberOfLines   The number of lines.

   @param ...             A series of text strings that displayed in the pop-up.