fixed list comprehension