Code review comment for lp:~m2j/openlp/bug-1094342

Revision history for this message
Meinert Jordan (m2j) wrote :

> Works on windows and one linux. I tested the sorting only with py2. On py3 I
> only tested if it starts.
>
> I haven't reviewed the code nor did performance tests yet.

Hi, I've looked to the Song class, and realized, that it makes sence to buffer the whole key generation for song titles. This should improve the overall performance (compared to trunk). So just wait for my next code commit.

« Back to merge proposal