BaseTools: Expression - change from series of if to elif

since the first character of the string cannot be found by multiple if
statements, use elif to optomize the behavior.

Cc: Yonghong Zhu <yonghong.zhu@intel.com>
Cc: Liming Gao <liming.gao@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Jaben Carsey <jaben.carsey@intel.com>
Reviewed-by: Yonghong Zhu <yonghong.zhu@intel.com>

1 files changed, 4 insertions, 4 deletions

diff --git a/BaseTools/Source/Python/Common/Expression.py b/BaseTools/Source/Python/Common/Expression.py
index c7037dd0d0..85c1ce9bbc 100644
--- a/BaseTools/Source/Python/Common/Expression.py
+++ b/BaseTools/Source/Python/Common/Expression.py
@@ -93,18 +93,18 @@ def SplitPcdValueString(String):
     for i, ch in enumerate(String):

         if ch == '(':

             InParenthesis += 1

-        if ch == ')':

+        elif ch == ')':

             if InParenthesis:

                 InParenthesis -= 1

             else:

                 raise BadExpression(ERR_STRING_TOKEN % Item)

-        if ch == '"' and not InSingleQuote:

+        elif ch == '"' and not InSingleQuote:

             if String[i-1] != '\\':

                 InDoubleQuote = not InDoubleQuote

-        if ch == "'" and not InDoubleQuote:

+        elif ch == "'" and not InDoubleQuote:

             if String[i-1] != '\\':

                 InSingleQuote = not InSingleQuote

-        if ch == ',':

+        elif ch == ',':

             if InParenthesis or InSingleQuote or InDoubleQuote:

                 Item += String[i]

                 continue