BaseTools: change the Division Operator in the expression

PEP 238 -- Changing the Division Operator
x/y to return a reasonable approximation of the mathematical result
    of the division ("true division")
x//y to return the floor ("floor division")

Cc: Liming Gao <liming.gao@intel.com>
Cc: Yonghong Zhu <yonghong.zhu@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Yunhua Feng <yunhuax.feng@intel.com>
Reviewed-by: Liming Gao <liming.gao@intel.com>

2 files changed, 11 insertions, 4 deletions

diff --git a/BaseTools/Source/Python/Common/Expression.py b/BaseTools/Source/Python/Common/Expression.py
index 9a98236a8c..5e4e66a10e 100644
--- a/BaseTools/Source/Python/Common/Expression.py
+++ b/BaseTools/Source/Python/Common/Expression.py
@@ -425,6 +425,13 @@ class ValueExpression(BaseExpression):
                 else:

                     Val = Val3

                 continue

+            #

+            # PEP 238 -- Changing the Division Operator

+            # x/y to return a reasonable approximation of the mathematical result of the division ("true division")

+            # x//y to return the floor ("floor division")

+            #

+            if Op == '/':

+                Op = '//'

             try:

                 Val = self.Eval(Op, Val, EvalFunc())

             except WrnExpression as Warn:

@@ -898,7 +905,7 @@ class ValueExpressionEx(ValueExpression):
                     if TmpValue.bit_length() == 0:

                         PcdValue = '{0x00}'

                     else:

-                        for I in range((TmpValue.bit_length() + 7) / 8):

+                        for I in range((TmpValue.bit_length() + 7) // 8):

                             TmpList.append('0x%02x' % ((TmpValue >> I * 8) & 0xff))

                         PcdValue = '{' + ', '.join(TmpList) + '}'

                 except:

diff --git a/BaseTools/Source/Python/Common/Misc.py b/BaseTools/Source/Python/Common/Misc.py
index ea0e62245f..71ab6da529 100644
--- a/BaseTools/Source/Python/Common/Misc.py
+++ b/BaseTools/Source/Python/Common/Misc.py
@@ -1310,7 +1310,7 @@ def ParseDevPathValue (Value):
 

 def ParseFieldValue (Value):

     if isinstance(Value, type(0)):

-        return Value, (Value.bit_length() + 7) / 8

+        return Value, (Value.bit_length() + 7) // 8

     if not isinstance(Value, type('')):

         raise BadExpression('Type %s is %s' %(Value, type(Value)))

     Value = Value.strip()

@@ -1431,12 +1431,12 @@ def ParseFieldValue (Value):
             raise BadExpression("invalid hex value: %s" % Value)

         if Value == 0:

             return 0, 1

-        return Value, (Value.bit_length() + 7) / 8

+        return Value, (Value.bit_length() + 7) // 8

     if Value[0].isdigit():

         Value = int(Value, 10)

         if Value == 0:

             return 0, 1

-        return Value, (Value.bit_length() + 7) / 8

+        return Value, (Value.bit_length() + 7) // 8

     if Value.lower() == 'true':

         return 1, 1

     if Value.lower() == 'false':