Cutlery/calibre
1
0
Fork 0
mirror of https://github.com/kovidgoyal/calibre.git synced 2025-07-09 03:04:10 -04:00
Code Issues Packages Projects Releases Wiki Activity

Improve perf of list_ template formatter functions

Browse Source
This commit is contained in:
Kovid Goyal 2011-10-16 06:36:03 +05:30
commit 14782270ff
1 changed files with 9 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

19
src/calibre/utils/formatter_functions.py
View File

@ -909,12 +909,10 @@ class BuiltinListUnion(BuiltinFormatterFunction):
l2 = [l.strip() for l in list2.split(separator) if l.strip()] l2 = [l.strip() for l in list2.split(separator) if l.strip()]
lcl1 = set([icu_lower(l) for l in l1]) lcl1 = set([icu_lower(l) for l in l1])
res = [] res = set(l1)
for i in l1:
res.append(i)
for i in l2: for i in l2:
if icu_lower(i) not in lcl1: if icu_lower(i) not in lcl1:
res.append(i) res.add(i)
if separator == ',': if separator == ',':
return ', '.join(res) return ', '.join(res)
return separator.join(res) return separator.join(res)
@ -930,7 +928,7 @@ class BuiltinListDifference(BuiltinFormatterFunction):
def evaluate(self, formatter, kwargs, mi, locals, list1, list2, separator): def evaluate(self, formatter, kwargs, mi, locals, list1, list2, separator):
l1 = [l.strip() for l in list1.split(separator) if l.strip()] l1 = [l.strip() for l in list1.split(separator) if l.strip()]
l2 = [icu_lower(l.strip()) for l in list2.split(separator) if l.strip()] l2 = set([icu_lower(l.strip()) for l in list2.split(separator) if l.strip()])
res = [] res = []
for i in l1: for i in l1:
@ -951,12 +949,12 @@ class BuiltinListIntersection(BuiltinFormatterFunction):
def evaluate(self, formatter, kwargs, mi, locals, list1, list2, separator): def evaluate(self, formatter, kwargs, mi, locals, list1, list2, separator):
l1 = [l.strip() for l in list1.split(separator) if l.strip()] l1 = [l.strip() for l in list1.split(separator) if l.strip()]
l2 = [icu_lower(l.strip()) for l in list2.split(separator) if l.strip()] l2 = set([icu_lower(l.strip()) for l in list2.split(separator) if l.strip()])
res = [] res = set()
for i in l1: for i in l1:
if icu_lower(i) in l2: if icu_lower(i) in l2:
res.append(i) res.add(i)
if separator == ',': if separator == ',':
return ', '.join(res) return ', '.join(res)
return separator.join(res) return separator.join(res)
@ -1007,12 +1005,13 @@ class BuiltinListRe(BuiltinFormatterFunction):
def evaluate(self, formatter, kwargs, mi, locals, src_list, separator, search_re, opt_replace): def evaluate(self, formatter, kwargs, mi, locals, src_list, separator, search_re, opt_replace):
l = [l.strip() for l in src_list.split(separator) if l.strip()] l = [l.strip() for l in src_list.split(separator) if l.strip()]
res = [] res = set()
for item in l: for item in l:
if re.search(search_re, item, flags=re.I) is not None: if re.search(search_re, item, flags=re.I) is not None:
if opt_replace: if opt_replace:
item = re.sub(search_re, opt_replace, item) item = re.sub(search_re, opt_replace, item)
res.append(item) for i in [l.strip() for l in item.split(',') if l.strip()]:
res.add(i)
if separator == ',': if separator == ',':
return ', '.join(res) return ', '.join(res)
return separator.join(res) return separator.join(res)