From cf51f73e02cab85f508bcbef362ff75a7ae18d7f Mon Sep 17 00:00:00 2001 From: =?utf8?q?Bj=C3=B6rn=20Jacke?= Date: Sun, 25 Aug 2019 23:06:19 +0200 Subject: [PATCH] schema.py: avoid inefficient string concatenations Signed-off-by: Bjoern Jacke Reviewed-by: Andrew Bartlett --- python/samba/schema.py | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/python/samba/schema.py b/python/samba/schema.py index aedee82bb8f..caea7e358ae 100644 --- a/python/samba/schema.py +++ b/python/samba/schema.py @@ -111,9 +111,7 @@ class Schema(object): setup_path('ad-schema/%s' % Schema.base_schemas[base_schema][1])) if files is not None: - for file in files: - data = get_string(open(file, 'rb').read()) - self.schema_data += data + self.schema_data = "".join(get_string(open(file, 'rb').read()) for file in files) self.schema_data = substitute_var(self.schema_data, {"SCHEMADN": schemadn}) @@ -135,8 +133,7 @@ class Schema(object): self.prefixmap_data = open(setup_path("prefixMap.txt"), 'rb').read() if additional_prefixmap is not None: - for map in additional_prefixmap: - self.prefixmap_data += "%s\n" % map + self.prefixmap_data += "".join("%s\n" % map for map in additional_prefixmap) self.prefixmap_data = b64encode(self.prefixmap_data).decode('utf8') -- 2.47.3