amdw / jianjin

A vocabulary manager web application for Chinese
6 stars 1 forks source link

Remove unnecessary list conversions introduced during move to Python 3 #56

Closed amdw closed 9 years ago

amdw commented 9 years ago

During the move to Python 3, a few list() casts were introduced, e.g. around calls to zip() and dict.items(). This was due to conservative decisions in the implementation of the 2to3 converter, as the return type of these functions changed.

However, these casts are likely unnecessary in many cases as using these as iterators or dynamic views is likely fine. Need to check and remove these casts as appropriate. This will help both performance and readability.