BaseTools: dont make iterator into list if not needed

functions (like join) can use the iterator just as easily.

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

diff --git a/BaseTools/Source/Python/Ecc/MetaFileWorkspace/MetaDataTable.py b/BaseTools/Source/Python/Ecc/MetaFileWorkspace/MetaDataTable.py
index 6b980150f5..760f88cc72 100644
--- a/BaseTools/Source/Python/Ecc/MetaFileWorkspace/MetaDataTable.py
+++ b/BaseTools/Source/Python/Ecc/MetaFileWorkspace/MetaDataTable.py
@@ -1,7 +1,7 @@
 ## @file

 # This file is used to create/update/query/erase table for files

 #

-# Copyright (c) 2008 - 2014, Intel Corporation. All rights reserved.<BR>

+# Copyright (c) 2008 - 2018, Intel Corporation. All rights reserved.<BR>

 # This program and the accompanying materials

 # are licensed and made available under the terms and conditions of the BSD License

 # which accompanies this distribution.  The full text of the license may be found at

@@ -73,7 +73,7 @@ class Table(object):
         self.ID = self.ID + self._ID_STEP_

         if self.ID >= (self.IdBase + self._ID_MAX_):

             self.ID = self.IdBase + self._ID_STEP_

-        Values = ", ".join([str(Arg) for Arg in Args])

+        Values = ", ".join(str(Arg) for Arg in Args)

         SqlCommand = "insert into %s values(%s, %s)" % (self.Table, self.ID, Values)

         EdkLogger.debug(EdkLogger.DEBUG_5, SqlCommand)

         self.Cur.execute(SqlCommand)