Carsey, Jaben 
							
						 
					 
					
						
						
						
						
							
						
						
							eb2c0b2c80 
							
						 
					 
					
						
						
							
							BaseTools: remove uncalled 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> 
						
						
					 
					
						2018-04-10 10:00:00 +08:00 
						 
				 
			
				
					
						
							
							
								Carsey, Jaben 
							
						 
					 
					
						
						
						
						
							
						
						
							ce3082a6e7 
							
						 
					 
					
						
						
							
							BaseTools: remove unused file  
						
						... 
						
						
						
						ToolsDefClassObject didnt need Dictionary, it needed an import from there.
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 09:58:08 +08:00 
						 
				 
			
				
					
						
							
							
								Yonghong Zhu 
							
						 
					 
					
						
						
						
						
							
						
						
							64797018df 
							
						 
					 
					
						
						
							
							BaseTools: Pcds in [Components] are not display correct in the report  
						
						... 
						
						
						
						The Pcd used in [Components] section, the PCD value is displayed
incorrect in the build report because the PCD default value was not
override.
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-09 14:05:45 +08:00 
						 
				 
			
				
					
						
							
							
								Yonghong Zhu 
							
						 
					 
					
						
						
						
						
							
						
						
							b91b8ee4c9 
							
						 
					 
					
						
						
							
							BaseTools: Pcd not used info should not in Module PCD section  
						
						... 
						
						
						
						Pcds in Conditional Directives and Pcds not used are Platform Level
info, it should not display in Module PCD Section.
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-09 14:05:01 +08:00 
						 
				 
			
				
					
						
							
							
								Carsey, Jaben 
							
						 
					 
					
						
						
						
						
							
						
						
							175a4b5db3 
							
						 
					 
					
						
						
							
							BaseTools: dont make temporary dict  
						
						... 
						
						
						
						just make the key list directly
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-09 14:02:57 +08:00 
						 
				 
			
				
					
						
							
							
								Feng, YunhuaX 
							
						 
					 
					
						
						
						
						
							
						
						
							95cc496216 
							
						 
					 
					
						
						
							
							BaseTools: Remove FdfParserLite.py from source since it is not used  
						
						... 
						
						
						
						Remove FdfParserLite.py from source code since it is not used.
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-09 11:10:51 +08:00 
						 
				 
			
				
					
						
							
							
								Carsey, Jaben 
							
						 
					 
					
						
						
						
						
							
						
						
							f8c0a004a5 
							
						 
					 
					
						
						
							
							BaseTools: small cleanup  
						
						... 
						
						
						
						just deleting else: then pass as they have no effect.
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-08 16:33:04 +08:00 
						 
				 
			
				
					
						
							
							
								Carsey, Jaben 
							
						 
					 
					
						
						
						
						
							
						
						
							ff5635e9ac 
							
						 
					 
					
						
						
							
							BaseTools: Autogen - change from list to set  
						
						... 
						
						
						
						by changing from list to set(), we can skip all the preprocessing
to prevent duplication and we dont need to convert to a set() later
on for each 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-08 16:33:03 +08:00 
						 
				 
			
				
					
						
							
							
								Carsey, Jaben 
							
						 
					 
					
						
						
						
						
							
						
						
							e6c2468a9e 
							
						 
					 
					
						
						
							
							BaseTools: move RegEx to root of file and share it  
						
						... 
						
						
						
						make it easy to import and use by others
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-08 16:33:03 +08:00 
						 
				 
			
				
					
						
							
							
								Carsey, Jaben 
							
						 
					 
					
						
						
						
						
							
						
						
							bfc8f5667a 
							
						 
					 
					
						
						
							
							BaseTools: remove redundant check  
						
						... 
						
						
						
						The RegEx matches begining and end of the string, dont then check length.
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-08 15:34:12 +08:00 
						 
				 
			
				
					
						
							
							
								Carsey, Jaben 
							
						 
					 
					
						
						
						
						
							
						
						
							1f26f5fdb9 
							
						 
					 
					
						
						
							
							BaseTools: AutoGen - use the new shared RegEx  
						
						... 
						
						
						
						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-08 15:34:11 +08:00 
						 
				 
			
				
					
						
							
							
								Carsey, Jaben 
							
						 
					 
					
						
						
						
						
							
						
						
							7da06eeede 
							
						 
					 
					
						
						
							
							BaseTools: Add new RegEx pattern to GlobalData  
						
						... 
						
						
						
						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-08 15:34:11 +08:00 
						 
				 
			
				
					
						
							
							
								Carsey, Jaben 
							
						 
					 
					
						
						
						
						
							
						
						
							ffe720c53e 
							
						 
					 
					
						
						
							
							BaseTools: GenFds - move RegEx compile  
						
						... 
						
						
						
						compile each RegEx once not in loops/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> 
						
						
					 
					
						2018-04-08 15:34:10 +08:00 
						 
				 
			
				
					
						
							
							
								Carsey, Jaben 
							
						 
					 
					
						
						
						
						
							
						
						
							c8802c3dcc 
							
						 
					 
					
						
						
							
							BaseTools: Autogen - move RegEx compile  
						
						... 
						
						
						
						compile each RegEx once not in loops/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> 
						
						
					 
					
						2018-04-08 15:34:10 +08:00 
						 
				 
			
				
					
						
							
							
								Carsey, Jaben 
							
						 
					 
					
						
						
						
						
							
						
						
							ab0ff0b0a6 
							
						 
					 
					
						
						
							
							BaseTools: Remove unused code from Misc  
						
						... 
						
						
						
						remove the functions and classes
remove any imports of these
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-08 14:50:18 +08:00 
						 
				 
			
				
					
						
							
							
								Carsey, Jaben 
							
						 
					 
					
						
						
						
						
							
						
						
							a0767bae29 
							
						 
					 
					
						
						
							
							BaseTools: Workspace - use built in OrderedDict instead of custom version.  
						
						... 
						
						
						
						We dont use any feature added by custom dictionary class.
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-08 14:50:18 +08:00 
						 
				 
			
				
					
						
							
							
								Carsey, Jaben 
							
						 
					 
					
						
						
						
						
							
						
						
							1f1c671284 
							
						 
					 
					
						
						
							
							BaseTools: use combined version of OrderedDict  
						
						... 
						
						
						
						since we need order and a default entry, use collections dicts to
auto generate.
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-08 14:50:17 +08:00 
						 
				 
			
				
					
						
							
							
								Carsey, Jaben 
							
						 
					 
					
						
						
						
						
							
						
						
							6e6d767edf 
							
						 
					 
					
						
						
							
							BaseTools: use built in OrderedDict instead of custom version.  
						
						... 
						
						
						
						We dont use any feature added by custom dictionary class.
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-08 14:50:17 +08:00 
						 
				 
			
				
					
						
							
							
								Carsey, Jaben 
							
						 
					 
					
						
						
						
						
							
						
						
							0d8ff45567 
							
						 
					 
					
						
						
							
							BaseTools: Eot remove unused code  
						
						... 
						
						
						
						2 functions and a dictionary that are not used.
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-08 14:50:16 +08:00 
						 
				 
			
				
					
						
							
							
								Carsey, Jaben 
							
						 
					 
					
						
						
						
						
							
						
						
							9006a2c6a3 
							
						 
					 
					
						
						
							
							BaseTools - AutoGen - replace custom dictionary class with python standard one  
						
						... 
						
						
						
						We have a custom ordered dictionary class.  works fine with python OrderedDict version.
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-08 14:50:16 +08:00 
						 
				 
			
				
					
						
							
							
								Carsey, Jaben 
							
						 
					 
					
						
						
						
						
							
						
						
							386eb9d793 
							
						 
					 
					
						
						
							
							BaseTools: remove unused import statement  
						
						... 
						
						
						
						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-08 14:50:16 +08:00 
						 
				 
			
				
					
						
							
							
								Carsey, Jaben 
							
						 
					 
					
						
						
						
						
							
						
						
							140b2b11a0 
							
						 
					 
					
						
						
							
							BaseTools: Eot tool never populates this dictionary  
						
						... 
						
						
						
						we initialize this dict and then check it's contents, but never add items.
we can remove it without any effect.
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-08 14:50:15 +08:00 
						 
				 
			
				
					
						
							
							
								Carsey, Jaben 
							
						 
					 
					
						
						
						
						
							
						
						
							5af2a627d1 
							
						 
					 
					
						
						
							
							BaseTools: use built in dict instead of custom version.  
						
						... 
						
						
						
						We dont use any feature added by custom dictionary class.
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-08 14:50:15 +08:00 
						 
				 
			
				
					
						
							
							
								Carsey, Jaben 
							
						 
					 
					
						
						
						
						
							
						
						
							5060928b12 
							
						 
					 
					
						
						
							
							BaseTools: Use local variable for list of constants.  
						
						... 
						
						
						
						instead of listing in multiple places, use a single list.
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-08 14:50:14 +08:00 
						 
				 
			
				
					
						
							
							
								Carsey, Jaben 
							
						 
					 
					
						
						
						
						
							
						
						
							c4172f8005 
							
						 
					 
					
						
						
							
							BaseTools: remove uncalled functions  
						
						... 
						
						
						
						this same function in 2 classes is never called.
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-03 17:39:34 +08:00 
						 
				 
			
				
					
						
							
							
								Carsey, Jaben 
							
						 
					 
					
						
						
						
						
							
						
						
							5bcf1d5671 
							
						 
					 
					
						
						
							
							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> 
						
						
					 
					
						2018-04-03 17:38:42 +08:00 
						 
				 
			
				
					
						
							
							
								Carsey, Jaben 
							
						 
					 
					
						
						
						
						
							
						
						
							231fdbb210 
							
						 
					 
					
						
						
							
							BaseTools: remove uncalled function  
						
						... 
						
						
						
						no one calls __IsWhiteSpace() (none of the 4 copies)
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-03 17:38:10 +08:00 
						 
				 
			
				
					
						
							
							
								Carsey, Jaben 
							
						 
					 
					
						
						
						
						
							
						
						
							cfbe3c3500 
							
						 
					 
					
						
						
							
							BaseTools: change hex parsing to use built in  
						
						... 
						
						
						
						use <char> in string.hexdigits instead of custom functions.
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-03 17:37:33 +08:00 
						 
				 
			
				
					
						
							
							
								Carsey, Jaben 
							
						 
					 
					
						
						
						
						
							
						
						
							147a656b34 
							
						 
					 
					
						
						
							
							BaseTools: use new RegEx from FdfParserLite  
						
						... 
						
						
						
						FdfParser has identical one.  import and share.
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-03 17:36:15 +08:00 
						 
				 
			
				
					
						
							
							
								Carsey, Jaben 
							
						 
					 
					
						
						
						
						
							
						
						
							4f9e71e1f7 
							
						 
					 
					
						
						
							
							BaseTools: use single RegExp for token matching  
						
						... 
						
						
						
						same pattern was compiled 3 places in the file.  just compile once and share.
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-03 17:35:31 +08:00 
						 
				 
			
				
					
						
							
							
								Carsey, Jaben 
							
						 
					 
					
						
						
						
						
							
						
						
							0b560b980c 
							
						 
					 
					
						
						
							
							BaseTools: cleanup class heirarchy  
						
						... 
						
						
						
						remove totally empty classes from class heirarchy
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-03-30 09:17:30 +08:00 
						 
				 
			
				
					
						
							
							
								Carsey, Jaben 
							
						 
					 
					
						
						
						
						
							
						
						
							5fbb0f9908 
							
						 
					 
					
						
						
							
							BaseTools: remove loop and variables.  
						
						... 
						
						
						
						this loop does nothing. none of Key, Item, nor DevicePathList
are ever used.
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-03-30 09:17:09 +08:00 
						 
				 
			
				
					
						
							
							
								Carsey, Jaben 
							
						 
					 
					
						
						
						
						
							
						
						
							0a014fba41 
							
						 
					 
					
						
						
							
							BaseTools: use in to compare single chars  
						
						... 
						
						
						
						instead if 3 Startswith for single chars, just use in with a list of chars
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-03-30 09:16:27 +08:00 
						 
				 
			
				
					
						
							
							
								Carsey, Jaben 
							
						 
					 
					
						
						
						
						
							
						
						
							0944818a19 
							
						 
					 
					
						
						
							
							BaseTools: no need to do int() API work for it  
						
						... 
						
						
						
						int() with base=0 will already auto determine base from preceeding 0x/0X
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-03-30 09:16:01 +08:00 
						 
				 
			
				
					
						
							
							
								Carsey, Jaben 
							
						 
					 
					
						
						
						
						
							
						
						
							4231a8193e 
							
						 
					 
					
						
						
							
							BaseTools: Remove equality operator with None  
						
						... 
						
						
						
						replace "== None" with "is None" and "!= None" with "is not None"
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-03-30 08:25:13 +08:00 
						 
				 
			
				
					
						
							
							
								Bin Wang 
							
						 
					 
					
						
						
						
						
							
						
						
							b1956b5d42 
							
						 
					 
					
						
						
							
							BaseTools: compare GUID value should not case-sensitive  
						
						... 
						
						
						
						build report error when the same Guid value in FDF file use lowercase,
in tools_def.txt file use uppercase.
The guid value's compare should not case-sensitive.
Cc: Liming Gao <liming.gao@intel.com>
Cc: Yonghong Zhu <yonghong.zhu@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Bin Wang <binx.a.wang@intel.com>
Reviewed-by: Yonghong Zhu <yonghong.zhu@intel.com> 
						
						
					 
					
						2018-03-29 16:34:54 +08:00 
						 
				 
			
				
					
						
							
							
								Carsey, Jaben 
							
						 
					 
					
						
						
						
						
							
						
						
							cc0321f22a 
							
						 
					 
					
						
						
							
							BaseTools: refactor repeated RegExp when no special searching is needed.  
						
						... 
						
						
						
						use str.replace and try/except.
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-03-29 16:33:55 +08:00 
						 
				 
			
				
					
						
							
							
								Carsey, Jaben 
							
						 
					 
					
						
						
						
						
							
						
						
							e52aed0d85 
							
						 
					 
					
						
						
							
							BaseTools: dont use enumerate when un-needed  
						
						... 
						
						
						
						Since we only use the item from the list and not the numeric value,
dont bother with enumerate()
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-03-29 16:31:24 +08:00 
						 
				 
			
				
					
						
							
							
								Carsey, Jaben 
							
						 
					 
					
						
						
						
						
							
						
						
							38504ad3e3 
							
						 
					 
					
						
						
							
							BaseTools: move regular expression compile out of function call.  
						
						... 
						
						
						
						move to the root of the file and dont recompile.
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-03-29 16:28:43 +08:00 
						 
				 
			
				
					
						
							
							
								Carsey, Jaben 
							
						 
					 
					
						
						
						
						
							
						
						
							663b9e061e 
							
						 
					 
					
						
						
							
							BaseTools: expression can use single in instead of 3 API calls.  
						
						... 
						
						
						
						change 3 StartsWith() calls to a single 'in' operation.
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-03-29 16:28:11 +08:00 
						 
				 
			
				
					
						
							
							
								Carsey, Jaben 
							
						 
					 
					
						
						
						
						
							
						
						
							56326323e6 
							
						 
					 
					
						
						
							
							BaseTools: remove local hex number regular expression  
						
						... 
						
						
						
						Change to using the new shared hex number regular expression
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-03-29 16:26:38 +08:00 
						 
				 
			
				
					
						
							
							
								Carsey, Jaben 
							
						 
					 
					
						
						
						
						
							
						
						
							018f7b827f 
							
						 
					 
					
						
						
							
							BaseTools: Use precompiled RegExp  
						
						... 
						
						
						
						avoid recompiling the regular expression for each use in a while loop
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-03-29 16:20:22 +08:00 
						 
				 
			
				
					
						
							
							
								Carsey, Jaben 
							
						 
					 
					
						
						
						
						
							
						
						
							b1a9e404d4 
							
						 
					 
					
						
						
							
							BaseTools: use new shared GUID regular expressions  
						
						... 
						
						
						
						remove local variables that are GUID matching and replace with shared
expression.
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-03-29 16:18:29 +08:00 
						 
				 
			
				
					
						
							
							
								Carsey, Jaben 
							
						 
					 
					
						
						
						
						
							
						
						
							0760ed06a1 
							
						 
					 
					
						
						
							
							BaseTools: GlobalData Add a regular expression for a hex number  
						
						... 
						
						
						
						add a shared precompiled regular expression
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-03-29 10:17:46 +08:00 
						 
				 
			
				
					
						
							
							
								Carsey, Jaben 
							
						 
					 
					
						
						
						
						
							
						
						
							9e7790a352 
							
						 
					 
					
						
						
							
							BaseTools: Add new RegExp for future use  
						
						... 
						
						
						
						Add a precompiled RegExp for 4 hex chars.
v2: fixed incorrect numbers of {}
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-03-29 08:55:09 +08:00 
						 
				 
			
				
					
						
							
							
								Carsey, Jaben 
							
						 
					 
					
						
						
						
						
							
						
						
							256e2d885b 
							
						 
					 
					
						
						
							
							BaseTools: Regular Expressions refactor out the hex char for later reuse  
						
						... 
						
						
						
						move hex character info from GUID expressions into seperate variable to
facilitate reuse.
I had a type with insufficient {} in the first version.
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-03-29 08:54:35 +08:00 
						 
				 
			
				
					
						
							
							
								Carsey, Jaben 
							
						 
					 
					
						
						
						
						
							
						
						
							cd67d66484 
							
						 
					 
					
						
						
							
							BaseTools: add GUID pattern to global data  
						
						... 
						
						
						
						add a shared global regular expression for GUID matching
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-03-29 08:36:06 +08:00 
						 
				 
			
				
					
						
							
							
								Carsey, Jaben 
							
						 
					 
					
						
						
						
						
							
						
						
							0f17c9fef1 
							
						 
					 
					
						
						
							
							BaseTools: GlobalData share same MACRO name definition  
						
						... 
						
						
						
						use the same MACRO name definition across shared regular expression patterns.
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-03-29 08:35:17 +08:00 
						 
				 
			
				
					
						
							
							
								Carsey, Jaben 
							
						 
					 
					
						
						
						
						
							
						
						
							4d603daa3a 
							
						 
					 
					
						
						
							
							BaseTools: FdfParser and FdfParserLite share reg exp  
						
						... 
						
						
						
						FdfParser can share regular expression from FdfParserLite.
reduce overlap and reduce recompile of the same expression.
v2: fix missed replacement of Pattern with shared variable
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-03-29 08:34:09 +08:00 
						 
				 
			
				
					
						
							
							
								Liming Gao 
							
						 
					 
					
						
						
						
						
							
						
						
							d1b777440b 
							
						 
					 
					
						
						
							
							BaseTools: Update Rsa2048Sha256Sign to use openssl dgst option  
						
						... 
						
						
						
						Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Liming Gao <liming.gao@intel.com>
Cc: Yonghong Zhu <yonghong.zhu@intel.com>
Cc: Qin Long <qin.long@intel.com>
Reviewed-by: Qin Long <qin.long@intel.com> 
						
						
					 
					
						2018-03-28 09:58:43 +08:00