Carsey, Jaben 
							
						 
					 
					
						
						
						
						
							
						
						
							992fbe3519 
							
						 
					 
					
						
						
							
							BaseTools: AutoGen - use dafultdict instead of dict  
						
						... 
						
						
						
						since we manually make each entry a set(), just use defaultdict(set)
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> 
						
						
					 
					
						2018-04-26 14:44:35 +08:00 
						 
				 
			
				
					
						
							
							
								Carsey, Jaben 
							
						 
					 
					
						
						
						
						
							
						
						
							dc2c30ef92 
							
						 
					 
					
						
						
							
							BaseTools: remove duplicate variable  
						
						... 
						
						
						
						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> 
						
						
					 
					
						2018-04-26 14:44:34 +08:00 
						 
				 
			
				
					
						
							
							
								Carsey, Jaben 
							
						 
					 
					
						
						
						
						
							
						
						
							1d5fde83d0 
							
						 
					 
					
						
						
							
							BaseTools: remove dict from DscBuildData  
						
						... 
						
						
						
						the dict is not needed as BaseTools can check the set
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> 
						
						
					 
					
						2018-04-26 14:43:26 +08:00 
						 
				 
			
				
					
						
							
							
								Carsey, Jaben 
							
						 
					 
					
						
						
						
						
							
						
						
							dca689d579 
							
						 
					 
					
						
						
							
							BaseTools: Remove extra .keys()  
						
						... 
						
						
						
						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> 
						
						
					 
					
						2018-04-26 14:43:25 +08:00 
						 
				 
			
				
					
						
							
							
								Carsey, Jaben 
							
						 
					 
					
						
						
						
						
							
						
						
							6fcd911c32 
							
						 
					 
					
						
						
							
							BaseTools: AutoGen - remove unused variables.  
						
						... 
						
						
						
						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> 
						
						
					 
					
						2018-04-26 14:43:25 +08:00 
						 
				 
			
				
					
						
							
							
								Carsey, Jaben 
							
						 
					 
					
						
						
						
						
							
						
						
							fe0a9af881 
							
						 
					 
					
						
						
							
							BaseTools: AutoGen - remove dictionary populated, but never accessed  
						
						... 
						
						
						
						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> 
						
						
					 
					
						2018-04-26 14:43:24 +08:00 
						 
				 
			
				
					
						
							
							
								Carsey, Jaben 
							
						 
					 
					
						
						
						
						
							
						
						
							9759febdb8 
							
						 
					 
					
						
						
							
							BaseTools: Workspace - refactor GetStructurePcdInfo  
						
						... 
						
						
						
						the function doesn't use self and can be static
defaultdict replaces dict and removes the dict initialization code
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> 
						
						
					 
					
						2018-04-26 14:42:20 +08:00 
						 
				 
			
				
					
						
							
							
								Carsey, Jaben 
							
						 
					 
					
						
						
						
						
							
						
						
							b813843cca 
							
						 
					 
					
						
						
							
							BaseTools: simplify if call  
						
						... 
						
						
						
						the variable is a string type since we just used .strip() on it.
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> 
						
						
					 
					
						2018-04-26 14:36:05 +08:00 
						 
				 
			
				
					
						
							
							
								Carsey, Jaben 
							
						 
					 
					
						
						
						
						
							
						
						
							6b1f2cf41a 
							
						 
					 
					
						
						
							
							BaseTools: Autogen - replace string constants with those from DataType  
						
						... 
						
						
						
						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> 
						
						
					 
					
						2018-04-26 14:36:05 +08:00 
						 
				 
			
				
					
						
							
							
								Carsey, Jaben 
							
						 
					 
					
						
						
						
						
							
						
						
							3e4faa268e 
							
						 
					 
					
						
						
							
							BaseTools: Workspace - refactor RegEx to minimize multiple compiling  
						
						... 
						
						
						
						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> 
						
						
					 
					
						2018-04-26 14:35:36 +08:00 
						 
				 
			
				
					
						
							
							
								Carsey, Jaben 
							
						 
					 
					
						
						
						
						
							
						
						
							1eb72acddd 
							
						 
					 
					
						
						
							
							BaseTools: Share RegEx between files  
						
						... 
						
						
						
						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> 
						
						
					 
					
						2018-04-26 14:27:59 +08:00 
						 
				 
			
				
					
						
							
							
								Carsey, Jaben 
							
						 
					 
					
						
						
						
						
							
						
						
							2a9aac46a6 
							
						 
					 
					
						
						
							
							BaseTools: GenPatchPcdTable - refactor RegEx to minimize multiple compiling  
						
						... 
						
						
						
						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> 
						
						
					 
					
						2018-04-26 14:27:59 +08:00 
						 
				 
			
				
					
						
							
							
								Carsey, Jaben 
							
						 
					 
					
						
						
						
						
							
						
						
							6686d9a125 
							
						 
					 
					
						
						
							
							BaseTools: Misc - refactor RegEx to minimize multiple compiling  
						
						... 
						
						
						
						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> 
						
						
					 
					
						2018-04-26 14:27:58 +08:00 
						 
				 
			
				
					
						
							
							
								Carsey, Jaben 
							
						 
					 
					
						
						
						
						
							
						
						
							656d2539be 
							
						 
					 
					
						
						
							
							BaseTools: replace 'UINT8','UINT16','UINT32','UINT64','VOID*' with shared constants.  
						
						... 
						
						
						
						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> 
						
						
					 
					
						2018-04-23 11:11:20 +08:00 
						 
				 
			
				
					
						
							
							
								Carsey, Jaben 
							
						 
					 
					
						
						
						
						
							
						
						
							b491aa95ab 
							
						 
					 
					
						
						
							
							BaseTools: remove un-needed use of .keys() on dictionaries  
						
						... 
						
						
						
						sometimes just delete it.
sometimes the loop needed .values() instead
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> 
						
						
					 
					
						2018-04-23 11:11:19 +08:00 
						 
				 
			
				
					
						
							
							
								Carsey, Jaben 
							
						 
					 
					
						
						
						
						
							
						
						
							4e50eba1b6 
							
						 
					 
					
						
						
							
							BaseTools: remove PlatformClass as it's not used  
						
						... 
						
						
						
						v2 - update makefile too.
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> 
						
						
					 
					
						2018-04-23 11:08:44 +08:00 
						 
				 
			
				
					
						
							
							
								Carsey, Jaben 
							
						 
					 
					
						
						
						
						
							
						
						
							66d04638e5 
							
						 
					 
					
						
						
							
							BaseTools: remove EdkIIWorkspace as its not used.  
						
						... 
						
						
						
						v2 - update makefile too.
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> 
						
						
					 
					
						2018-04-23 11:08:24 +08:00 
						 
				 
			
				
					
						
							
							
								Yunhua Feng 
							
						 
					 
					
						
						
						
						
							
						
						
							3378782691 
							
						 
					 
					
						
						
							
							BaseTools: Support DSC component !include PCD items  
						
						... 
						
						
						
						DSC format:
[Components]
  TestPkg/TestDriver.inf {
    <PcdsFixedAtBuild>
      !include TestPkg/Test.txt
  }
Test.txt content:
PcdToken.PcdCName | 0x123
Per spec we should support this usage.
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Yunhua Feng <yunhuax.feng@intel.com>
Reviewed-by: Yonghong Zhu <yonghong.zhu@intel.com> 
						
						
					 
					
						2018-04-19 15:47:36 +08:00 
						 
				 
			
				
					
						
							
							
								Carsey, Jaben 
							
						 
					 
					
						
						
						
						
							
						
						
							25598f8bdb 
							
						 
					 
					
						
						
							
							BaseTools: merge towards minimum PCD MAX <something> methods  
						
						... 
						
						
						
						we have 5 different max val or max byte for PCDs.
refactor and remove 2 methods.
we need 3, as one computes for VOID* PCDs.
Cc: Bob Feng <bob.feng@intel.com>
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> 
						
						
					 
					
						2018-04-18 22:15:36 +08:00 
						 
				 
			
				
					
						
							
							
								Carsey, Jaben 
							
						 
					 
					
						
						
						
						
							
						
						
							9eb87141ec 
							
						 
					 
					
						
						
							
							BaseTools: refactor and remove un-needed use of .keys() on dictionaries  
						
						... 
						
						
						
						sometimes just delete it.
sometimes the loop needed .values() instead
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> 
						
						
					 
					
						2018-04-18 22:15:36 +08:00 
						 
				 
			
				
					
						
							
							
								Carsey, Jaben 
							
						 
					 
					
						
						
						
						
							
						
						
							55c84777ee 
							
						 
					 
					
						
						
							
							BaseTools: use predefined constants instead of local strings  
						
						... 
						
						
						
						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> 
						
						
					 
					
						2018-04-18 22:15:35 +08:00 
						 
				 
			
				
					
						
							
							
								Carsey, Jaben 
							
						 
					 
					
						
						
						
						
							
						
						
							5a693b89a1 
							
						 
					 
					
						
						
							
							BaseTools: refactor and remove more keys() usage  
						
						... 
						
						
						
						this is no longer required to make dictionary objects iterable.
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> 
						
						
					 
					
						2018-04-18 22:15:35 +08:00 
						 
				 
			
				
					
						
							
							
								Carsey, Jaben 
							
						 
					 
					
						
						
						
						
							
						
						
							d498274fcd 
							
						 
					 
					
						
						
							
							BaseTools: Eot - Remove FvImage file  
						
						... 
						
						
						
						move the single used class from FvImage to Eot
delete the FvImage file
remove FvImage from makefile
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> 
						
						
					 
					
						2018-04-18 22:15:34 +08:00 
						 
				 
			
				
					
						
							
							
								Carsey, Jaben 
							
						 
					 
					
						
						
						
						
							
						
						
							5f446994a2 
							
						 
					 
					
						
						
							
							BaseTools: Remove un-needed list comprehension  
						
						... 
						
						
						
						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> 
						
						
					 
					
						2018-04-18 22:15:34 +08:00 
						 
				 
			
				
					
						
							
							
								Carsey, Jaben 
							
						 
					 
					
						
						
						
						
							
						
						
							b96c7cf8ca 
							
						 
					 
					
						
						
							
							BaseTools: remove duplicate function name  
						
						... 
						
						
						
						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> 
						
						
					 
					
						2018-04-18 22:15:33 +08:00 
						 
				 
			
				
					
						
							
							
								Carsey, Jaben 
							
						 
					 
					
						
						
						
						
							
						
						
							9e508f3acc 
							
						 
					 
					
						
						
							
							BaseTools: change DscBuildData functions without need for self to staticmethod  
						
						... 
						
						
						
						prepend functiosn with @staticmethod
change calls to use class name, not self
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> 
						
						
					 
					
						2018-04-17 21:17:55 +08:00 
						 
				 
			
				
					
						
							
							
								Carsey, Jaben 
							
						 
					 
					
						
						
						
						
							
						
						
							e1a76b4702 
							
						 
					 
					
						
						
							
							BaseTools: remove an unused file  
						
						... 
						
						
						
						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> 
						
						
					 
					
						2018-04-17 21:02:06 +08:00 
						 
				 
			
				
					
						
							
							
								Carsey, Jaben 
							
						 
					 
					
						
						
						
						
							
						
						
							9fb9cbfb65 
							
						 
					 
					
						
						
							
							BaseTools: delete an unused file  
						
						... 
						
						
						
						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> 
						
						
					 
					
						2018-04-17 20:54:15 +08:00 
						 
				 
			
				
					
						
							
							
								Carsey, Jaben 
							
						 
					 
					
						
						
						
						
							
						
						
							80bfbd3e83 
							
						 
					 
					
						
						
							
							BaseTools: FfsInfStatement - remove unused function  
						
						... 
						
						
						
						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> 
						
						
					 
					
						2018-04-17 20:49:55 +08:00 
						 
				 
			
				
					
						
							
							
								Carsey, Jaben 
							
						 
					 
					
						
						
						
						
							
						
						
							227dbb1190 
							
						 
					 
					
						
						
							
							BaseTools: FdfParser - refactor functions to make static  
						
						... 
						
						
						
						make functions that doesn't use self into @staticmethod
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> 
						
						
					 
					
						2018-04-17 20:49:54 +08:00 
						 
				 
			
				
					
						
							
							
								Carsey, Jaben 
							
						 
					 
					
						
						
						
						
							
						
						
							cdc9b0c296 
							
						 
					 
					
						
						
							
							BaseTools: FdfParser refactor to remove a dictionary  
						
						... 
						
						
						
						__GetInfStatement() does not use the dict parameter, so remove it
from the API and from all callers.
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> 
						
						
					 
					
						2018-04-17 20:49:54 +08:00 
						 
				 
			
				
					
						
							
							
								Carsey, Jaben 
							
						 
					 
					
						
						
						
						
							
						
						
							c3dc9d29d1 
							
						 
					 
					
						
						
							
							BaseTools: refactor DepexSection.GenSection  
						
						... 
						
						
						
						change default parameter value to None since the parameter isn't used.
remove temporary dictionary and just do the reaplce call.
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> 
						
						
					 
					
						2018-04-17 20:49:53 +08:00 
						 
				 
			
				
					
						
							
							
								Carsey, Jaben 
							
						 
					 
					
						
						
						
						
							
						
						
							888d7f196d 
							
						 
					 
					
						
						
							
							BaseTools: use dictionary.get() when we have value if not found  
						
						... 
						
						
						
						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> 
						
						
					 
					
						2018-04-17 20:49:53 +08:00 
						 
				 
			
				
					
						
							
							
								Carsey, Jaben 
							
						 
					 
					
						
						
						
						
							
						
						
							fe1abb4beb 
							
						 
					 
					
						
						
							
							BaseTools: move RegEx compile out of loops  
						
						... 
						
						
						
						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> 
						
						
					 
					
						2018-04-17 20:49:53 +08:00 
						 
				 
			
				
					
						
							
							
								Carsey, Jaben 
							
						 
					 
					
						
						
						
						
							
						
						
							a6c910e358 
							
						 
					 
					
						
						
							
							BaseTools: Remove unused functions from DscBuildData  
						
						... 
						
						
						
						3 functions were never called:
_dumpPcdInfo
__STRING2OCTList
__UNICODE2OCTList
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> 
						
						
					 
					
						2018-04-17 20:49:52 +08:00 
						 
				 
			
				
					
						
							
							
								Carsey, Jaben 
							
						 
					 
					
						
						
						
						
							
						
						
							820a98a463 
							
						 
					 
					
						
						
							
							BaseTools: remove unused local variable.  
						
						... 
						
						
						
						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> 
						
						
					 
					
						2018-04-17 20:49:52 +08:00 
						 
				 
			
				
					
						
							
							
								Carsey, Jaben 
							
						 
					 
					
						
						
						
						
							
						
						
							128d435f5e 
							
						 
					 
					
						
						
							
							BaseTools: fix None comparisons  
						
						... 
						
						
						
						when comparing a list/string against None and empty, just compare the object.
when comparing against None, dont use !=, ==, <>
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> 
						
						
					 
					
						2018-04-17 20:49:32 +08:00 
						 
				 
			
				
					
						
							
							
								Carsey, Jaben 
							
						 
					 
					
						
						
						
						
							
						
						
							72fbe88d72 
							
						 
					 
					
						
						
							
							BaseTools: reduce list usage when not needed  
						
						... 
						
						
						
						remove not needed lists.  some were just counted and others
should be sets.
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> 
						
						
					 
					
						2018-04-17 20:48:53 +08:00 
						 
				 
			
				
					
						
							
							
								Carsey, Jaben 
							
						 
					 
					
						
						
						
						
							
						
						
							1563349a96 
							
						 
					 
					
						
						
							
							BaseTools: use set instead of list for a variable to be used with in  
						
						... 
						
						
						
						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> 
						
						
					 
					
						2018-04-17 20:48:53 +08:00 
						 
				 
			
				
					
						
							
							
								Carsey, Jaben 
							
						 
					 
					
						
						
						
						
							
						
						
							066c71544e 
							
						 
					 
					
						
						
							
							BaseTools: skip updating temporary variable.  
						
						... 
						
						
						
						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> 
						
						
					 
					
						2018-04-17 20:48:52 +08:00 
						 
				 
			
				
					
						
							
							
								Carsey, Jaben 
							
						 
					 
					
						
						
						
						
							
						
						
							1530ceda05 
							
						 
					 
					
						
						
							
							BaseTools: no need to save the data  
						
						... 
						
						
						
						It's never accessed.
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> 
						
						
					 
					
						2018-04-17 20:48:52 +08:00 
						 
				 
			
				
					
						
							
							
								Carsey, Jaben 
							
						 
					 
					
						
						
						
						
							
						
						
							4d3b9389e7 
							
						 
					 
					
						
						
							
							BaseTools: use existing shared variable  
						
						... 
						
						
						
						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> 
						
						
					 
					
						2018-04-17 20:48:51 +08:00 
						 
				 
			
				
					
						
							
							
								Feng, YunhuaX 
							
						 
					 
					
						
						
						
						
							
						
						
							e7df35b2bc 
							
						 
					 
					
						
						
							
							BaseTools: Fix one or more multiply defined symbols found issue  
						
						... 
						
						
						
						self.Guids update with package Guids will generate multiply defined
GUID symbols in AutoGen file
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: Yonghong Zhu <yonghong.zhu@intel.com> 
						
						
					 
					
						2018-04-15 08:39:31 +08:00 
						 
				 
			
				
					
						
							
							
								Feng, YunhuaX 
							
						 
					 
					
						
						
						
						
							
						
						
							a146c532c7 
							
						 
					 
					
						
						
							
							BaseTools: fix --genfds-multi-thread generate makefile issue  
						
						... 
						
						
						
						1. when inf file is binary module, not generate makefile,
  so need generate ffs with previous method.
2. generate Ui section maybe override and the string is not
  $(MODULE_NAME)
  like as:
  INF  RuleOverride = UI MdeModulePkg/Application/UiApp/UiApp.inf
3. Trim generate incorrect Offset.raw when some vfr not generate .lst
   file in Debug directory, Trim get the VFR name with the .c files
   replacement.
4. fix some depex file not generate issue
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: Yonghong Zhu <yonghong.zhu@intel.com> 
						
						
					 
					
						2018-04-11 14:47:55 +08:00 
						 
				 
			
				
					
						
							
							
								Yonghong Zhu 
							
						 
					 
					
						
						
						
						
							
						
						
							8b0e67821b 
							
						 
					 
					
						
						
							
							BaseTools: Fix the build error caused by eca980c0c899  
						
						... 
						
						
						
						Roll back the fixed at build pcd collection to include the pcd in
Module and Library.
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Yonghong Zhu <yonghong.zhu@intel.com>
Tested-by: Laszlo Ersek <lersek@redhat.com>
Reviewed-by: Liming Gao <liming.gao@intel.com> 
						
						
					 
					
						2018-04-11 09:55:41 +08:00 
						 
				 
			
				
					
						
							
							
								Carsey, Jaben 
							
						 
					 
					
						
						
						
						
							
						
						
							13d909f89a 
							
						 
					 
					
						
						
							
							BaseTools: Remove unneeded files  
						
						... 
						
						
						
						These files are not used by any tool:
BaseTools/Source/Python/Common/DecClassObject.py
BaseTools/Source/Python/Common/DscClassObject.py
BaseTools/Source/Python/Common/FdfClassObject.py
BaseTools/Source/Python/Common/InfClassObject.py
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> 
						
						
					 
					
						2018-04-10 14:28:45 +08:00 
						 
				 
			
				
					
						
							
							
								Feng, YunhuaX 
							
						 
					 
					
						
						
						
						
							
						
						
							e96995a134 
							
						 
					 
					
						
						
							
							BaseTools: Remove EdkIIWorkspaceBuild.py from source code  
						
						... 
						
						
						
						Remove EdkIIWorkspaceBuild.py from source code
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: Yonghong Zhu <yonghong.zhu@intel.com> 
						
						
					 
					
						2018-04-10 14:28:17 +08:00 
						 
				 
			
				
					
						
							
							
								Carsey, Jaben 
							
						 
					 
					
						
						
						
						
							
						
						
							f6562949e4 
							
						 
					 
					
						
						
							
							BaseTools: copy a dictionary from InfClassObject to BuildReport  
						
						... 
						
						
						
						InfClassObject will be deleted.
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> 
						
						
					 
					
						2018-04-10 14:21:54 +08:00 
						 
				 
			
				
					
						
							
							
								Yonghong Zhu 
							
						 
					 
					
						
						
						
						
							
						
						
							f9fa014ee0 
							
						 
					 
					
						
						
							
							BaseTools: Fix two cases that use GUID CName as PCD Value  
						
						... 
						
						
						
						1. use CName format in components section:
  [Components]
   TestPkg/TestDriver.inf {
     <PcdsFixedAtBuild>
      PcdToken.PcdName |{GUID(TestGuid)}|VOID*|16
  }
2. Use Guid CName format in INF and the Guid is defined in the DEC
file but not write in driver's [Guids] section.
    PcdToken.PcdName  | {GUID(TestGuid)}
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Yonghong Zhu <yonghong.zhu@intel.com>
Reviewed-by: Liming Gao <liming.gao@intel.com> 
						
						
					 
					
						2018-04-10 13:59:06 +08:00 
						 
				 
			
				
					
						
							
							
								Yonghong Zhu 
							
						 
					 
					
						
						
						
						
							
						
						
							c33081c912 
							
						 
					 
					
						
						
							
							BaseTools: Fix the bug for VOID* pcd max size from component section  
						
						... 
						
						
						
						When the Pcd defined in components section, its value's size is larger
than the value's size in [pcd] section, it cause build error, because
original code use the size get in [pcd] section as max size.
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Yonghong Zhu <yonghong.zhu@intel.com>
Reviewed-by: Liming Gao <liming.gao@intel.com> 
						
						
					 
					
						2018-04-10 13:58:14 +08:00