Re: [edk2] [PATCH v1 1/1] BaseTools: AutoGen - change class variable to funciton variable

2018-07-24 Thread Zhu, Yonghong
Reviewed-by: Yonghong Zhu Best Regards, Zhu Yonghong -Original Message- From: Carsey, Jaben Sent: Friday, July 20, 2018 1:58 AM To: edk2-devel@lists.01.org Cc: Gao, Liming ; Zhu, Yonghong Subject: [PATCH v1 1/1] BaseTools: AutoGen - change class variable to funciton variable This

[edk2] [PATCH v1 1/1] BaseTools: AutoGen - change class variable to funciton variable

2018-07-19 Thread Jaben Carsey
This variable is only used in one function, make it local there. Also when iterating on the variable, use dict.items() to get value instead of re-looking up the value multiple times. Cc: Liming Gao Cc: Yonghong Zhu Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Jaben