Revision: 18606
          http://sourceforge.net/p/edk2/code/18606
Author:   ydong10
Date:     2015-10-15 01:03:47 +0000 (Thu, 15 Oct 2015)
Log Message:
-----------
BaseTools VfrCompiler: In order to keep consistent, add an optional ";" for 
condition op-code.

Current grammar for suppressif opcode not consistent in statement and option 
case, this patch fixed this issue. The same case also existed for other 
condition opcodes.

Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Eric Dong <[email protected]>
Reviewed-by: Liming Gao <[email protected]>

Modified Paths:
--------------
    trunk/edk2/BaseTools/Source/C/VfrCompile/VfrSyntax.g

Modified: trunk/edk2/BaseTools/Source/C/VfrCompile/VfrSyntax.g
===================================================================
--- trunk/edk2/BaseTools/Source/C/VfrCompile/VfrSyntax.g        2015-10-15 
00:57:45 UTC (rev 18605)
+++ trunk/edk2/BaseTools/Source/C/VfrCompile/VfrSyntax.g        2015-10-15 
01:03:47 UTC (rev 18606)
@@ -3258,7 +3258,7 @@
   Prompt "=" "STRING_TOKEN" "\(" S:Number "\)" ","     << IIObj.SetError 
(_STOSID(S->getText(), S->getLine())); >>
   { FLAGS "=" flagsField ( "\|" flagsField )* "," }
   vfrStatementExpression[0]
-  E:EndIf                                              << CRT_END_OP (E); >>
+  E:EndIf {";"}                                        << CRT_END_OP (E); >>
   ;
 
 vfrStatementNoSubmitIf :
@@ -3267,7 +3267,7 @@
   Prompt "=" "STRING_TOKEN" "\(" S:Number "\)" ","     << NSIObj.SetError 
(_STOSID(S->getText(), S->getLine())); >>
   { FLAGS "=" flagsField ( "\|" flagsField )* "," }
   vfrStatementExpression[0]
-  E:EndIf                                              << CRT_END_OP (E); >>
+  E:EndIf {";"}                                        << CRT_END_OP (E); >>
   ;
 
 vfrStatementWarningIf :
@@ -3276,7 +3276,7 @@
   Prompt "=" "STRING_TOKEN" "\(" S:Number "\)" ","     << WIObj.SetWarning 
(_STOSID(S->getText(), S->getLine())); >>
   {Timeout "=" T:Number ","                            << WIObj.SetTimeOut 
(_STOU8(T->getText(), T->getLine())); >>}
   vfrStatementExpression[0]
-  E:EndIf                                              << CRT_END_OP (E); >>
+  E:EndIf {";"}                                        << CRT_END_OP (E); >>
   ;
 
 vfrStatementDisableIfQuest :
@@ -3286,7 +3286,7 @@
   L:DisableIf                                          << 
DIObj.SetLineNo(L->getLine()); >>
   vfrStatementExpression[0] ";"
   vfrStatementQuestionOptionList
-  E:EndIf                                              << CRT_END_OP (E); >>
+  E:EndIf {";"}                                        << CRT_END_OP (E); >>
   ;
 
 vfrStatementRefresh :
@@ -3316,7 +3316,7 @@
   { FLAGS "=" flagsField ( "\|" flagsField )* "," }
   vfrStatementExpression[0] ";"
   vfrStatementQuestionOptionList
-  E:EndIf                                              << CRT_END_OP (E); >>
+  E:EndIf {";"}                                        << CRT_END_OP (E); >>
   ;
 
 vfrStatementGrayOutIfQuest :
@@ -3325,7 +3325,7 @@
   { FLAGS "=" flagsField ( "\|" flagsField )* "," }
   vfrStatementExpression[0] ";"
   vfrStatementQuestionOptionList
-  E:EndIf                                              << CRT_END_OP (E); >>
+  E:EndIf {";"}                                        << CRT_END_OP (E); >>
   ;
 
 vfrStatementOptions :


------------------------------------------------------------------------------
_______________________________________________
edk2-commits mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/edk2-commits

Reply via email to