BaseTools: make static functions when self is not needed

remove self, and add @staticmethod to functions

Cc: Liming Gao <liming.gao@intel.com>
Cc: Yonghong Zhu <yonghong.zhu@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, 12 insertions, 13 deletions

diff --git a/BaseTools/Source/Python/GenFds/FdfParser.py b/BaseTools/Source/Python/GenFds/FdfParser.py
index 5dbcb96bbd..9b7e898570 100644
--- a/BaseTools/Source/Python/GenFds/FdfParser.py
+++ b/BaseTools/Source/Python/GenFds/FdfParser.py
@@ -704,7 +704,8 @@ class FdfParser:
         # Preprocess done.

         self.Rewind()

         

-    def __GetIfListCurrentItemStat(self, IfList):

+    @staticmethod

+    def __GetIfListCurrentItemStat(IfList):

         if len(IfList) == 0:

             return True

         

@@ -2625,13 +2626,12 @@ class FdfParser:
     #

     #   Check whether reloc strip flag can be set for a file type.

     #

-    #   @param  self        The object pointer

     #   @param  FileType    The file type to check with

     #   @retval True        This type could have relocation strip flag

     #   @retval False       No way to have it

     #

-

-    def __FileCouldHaveRelocFlag (self, FileType):

+    @staticmethod

+    def __FileCouldHaveRelocFlag (FileType):

         if FileType in ('SEC', 'PEI_CORE', 'PEIM', 'PEI_DXE_COMBO'):

             return True

         else:

@@ -2641,13 +2641,12 @@ class FdfParser:
     #

     #   Check whether reloc strip flag can be set for a section type.

     #

-    #   @param  self        The object pointer

     #   @param  SectionType The section type to check with

     #   @retval True        This type could have relocation strip flag

     #   @retval False       No way to have it

     #

-

-    def __SectionCouldHaveRelocFlag (self, SectionType):

+    @staticmethod

+    def __SectionCouldHaveRelocFlag (SectionType):

         if SectionType in ('TE', 'PE32'):

             return True

         else:

@@ -3986,12 +3985,12 @@ class FdfParser:
     #

     #   Get whether a section could be optional

     #

-    #   @param  self        The object pointer

     #   @param  SectionType The section type to check

     #   @retval True        section could be optional

     #   @retval False       section never optional

     #

-    def __RuleSectionCouldBeOptional(self, SectionType):

+    @staticmethod

+    def __RuleSectionCouldBeOptional(SectionType):

         if SectionType in ("DXE_DEPEX", "UI", "VERSION", "PEI_DEPEX", "RAW", "SMM_DEPEX"):

             return True

         else:

@@ -4001,12 +4000,12 @@ class FdfParser:
     #

     #   Get whether a section could have build number information

     #

-    #   @param  self        The object pointer

     #   @param  SectionType The section type to check

     #   @retval True        section could have build number information

     #   @retval False       section never have build number information

     #

-    def __RuleSectionCouldHaveBuildNum(self, SectionType):

+    @staticmethod

+    def __RuleSectionCouldHaveBuildNum(SectionType):

         if SectionType in ("VERSION"):

             return True

         else:

@@ -4016,12 +4015,12 @@ class FdfParser:
     #

     #   Get whether a section could have string

     #

-    #   @param  self        The object pointer

     #   @param  SectionType The section type to check

     #   @retval True        section could have string

     #   @retval False       section never have string

     #

-    def __RuleSectionCouldHaveString(self, SectionType):

+    @staticmethod

+    def __RuleSectionCouldHaveString(SectionType):

         if SectionType in ("UI", "VERSION"):

             return True

         else: