BaseTools: remove unused setter functions
authorJaben Carsey <jaben.carsey@intel.com>
Fri, 17 Aug 2018 16:17:40 +0000 (00:17 +0800)
committerYonghong Zhu <yonghong.zhu@intel.com>
Mon, 20 Aug 2018 03:34:59 +0000 (11:34 +0800)
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>
BaseTools/Source/Python/Workspace/DecBuildData.py
BaseTools/Source/Python/Workspace/DscBuildData.py
BaseTools/Source/Python/Workspace/InfBuildData.py

index 7eeca9524529b5b6dec743db945fe9a8c043ffb0..45beaebc63efe66737f4124a3ce85d80342b4d1d 100644 (file)
@@ -116,21 +116,6 @@ class DecBuildData(PackageBuildClassObject):
     def _GetArch(self):\r
         return self._Arch\r
 \r
-    ## Set architecture\r
-    #\r
-    #   Changing the default ARCH to another may affect all other information\r
-    # because all information in a platform may be ARCH-related. That's\r
-    # why we need to clear all internal used members, in order to cause all\r
-    # information to be re-retrieved.\r
-    #\r
-    #   @param  Value   The value of ARCH\r
-    #\r
-    def _SetArch(self, Value):\r
-        if self._Arch == Value:\r
-            return\r
-        self._Arch = Value\r
-        self._Clear()\r
-\r
     ## Retrieve all information in [Defines] section\r
     #\r
     #   (Retriving all [Defines] information in one-shot is just to save time.)\r
@@ -469,7 +454,7 @@ class DecBuildData(PackageBuildClassObject):
 \r
 \r
     _Macros = property(_GetMacros)\r
-    Arch = property(_GetArch, _SetArch)\r
+    Arch = property(_GetArch)\r
     PackageName = property(_GetPackageName)\r
     Guid = property(_GetFileGuid)\r
     Version = property(_GetVersion)\r
index c7f07f464646bb4390bbb690a00f1ab600283ac1..a4ad53ee1558b388d8a64a6b38977b825161f4d0 100644 (file)
@@ -308,21 +308,6 @@ class DscBuildData(PlatformBuildClassObject):
     def _GetArch(self):\r
         return self._Arch\r
 \r
-    ## Set architecture\r
-    #\r
-    #   Changing the default ARCH to another may affect all other information\r
-    # because all information in a platform may be ARCH-related. That's\r
-    # why we need to clear all internal used members, in order to cause all\r
-    # information to be re-retrieved.\r
-    #\r
-    #   @param  Value   The value of ARCH\r
-    #\r
-    def _SetArch(self, Value):\r
-        if self._Arch == Value:\r
-            return\r
-        self._Arch = Value\r
-        self._Clear()\r
-\r
     ## Retrieve all information in [Defines] section\r
     #\r
     #   (Retriving all [Defines] information in one-shot is just to save time.)\r
@@ -2849,7 +2834,7 @@ class DscBuildData(PlatformBuildClassObject):
             self._DecPcds, self._GuidDict = GetDeclaredPcd(self, self._Bdb, self._Arch, self._Target, self._Toolchain, PkgSet)\r
         return self._DecPcds\r
     _Macros             = property(_GetMacros)\r
-    Arch                = property(_GetArch, _SetArch)\r
+    Arch                = property(_GetArch)\r
     Platform            = property(_GetPlatformName)\r
     PlatformName        = property(_GetPlatformName)\r
     Guid                = property(_GetFileGuid)\r
index 44eaebf62e50a5ad6ff66502c579f870fb150a11..7c3d712efe636855b0753ed06d7a0076fa93008e 100644 (file)
@@ -173,36 +173,10 @@ class InfBuildData(ModuleBuildClassObject):
     def _GetArch(self):\r
         return self._Arch\r
 \r
-    ## Set architecture\r
-    #\r
-    #   Changing the default ARCH to another may affect all other information\r
-    # because all information in a platform may be ARCH-related. That's\r
-    # why we need to clear all internal used members, in order to cause all\r
-    # information to be re-retrieved.\r
-    #\r
-    #   @param  Value   The value of ARCH\r
-    #\r
-    def _SetArch(self, Value):\r
-        if self._Arch == Value:\r
-            return\r
-        self._Arch = Value\r
-        self._Clear()\r
-\r
     ## Return the name of platform employing this module\r
     def _GetPlatform(self):\r
         return self._Platform\r
 \r
-    ## Change the name of platform employing this module\r
-    #\r
-    #   Changing the default name of platform to another may affect some information\r
-    # because they may be PLATFORM-related. That's why we need to clear all internal\r
-    # used members, in order to cause all information to be re-retrieved.\r
-    #\r
-    def _SetPlatform(self, Value):\r
-        if self._Platform == Value:\r
-            return\r
-        self._Platform = Value\r
-        self._Clear()\r
     def _GetHeaderComments(self):\r
         if not self._HeaderComments:\r
             self._HeaderComments = []\r
@@ -1154,8 +1128,8 @@ class InfBuildData(ModuleBuildClassObject):
             return False\r
 \r
     _Macros = property(_GetMacros)\r
-    Arch = property(_GetArch, _SetArch)\r
-    Platform = property(_GetPlatform, _SetPlatform)\r
+    Arch = property(_GetArch)\r
+    Platform = property(_GetPlatform)\r
 \r
     HeaderComments = property(_GetHeaderComments)\r
     TailComments = property(_GetTailComments)\r