BaseTools: Refactor to remove functionally equivalent functions

IsSupportedArch and IsBinaryModule return the same value under the same
curcimstances.  Remove newer one with fewer callers and send them to the
other function.

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, 1 insertions, 7 deletions

diff --git a/BaseTools/Source/Python/Workspace/InfBuildData.py b/BaseTools/Source/Python/Workspace/InfBuildData.py
index 1325b4f029..d666c092a6 100644
--- a/BaseTools/Source/Python/Workspace/InfBuildData.py
+++ b/BaseTools/Source/Python/Workspace/InfBuildData.py
@@ -578,12 +578,7 @@ class InfBuildData(ModuleBuildClassObject):
             EdkLogger.error('build', RESOURCE_NOT_AVAILABLE, ExtraData=ErrorInfo, File=self.MetaFile)

 

         return Binaries

-    ## Check whether it exists the binaries with current ARCH in AsBuild INF

-    def _IsSupportedArch(self):

-        if self._GetBinaries() and not self._GetSourceFiles():

-            return True

-        else:

-            return False

+

     ## Retrieve source files

     def _GetSourceFiles(self):

         # Ignore all source files in a binary build mode

@@ -1170,4 +1165,3 @@ class InfBuildData(ModuleBuildClassObject):
     Depex                   = property(_GetDepex)

     DepexExpression         = property(_GetDepexExpression)

     IsBinaryModule          = property(_IsBinaryModule)

-    IsSupportedArch         = property(_IsSupportedArch)