sander added a comment.
Looking good! Concerning the preparatory commit 9ba8dd2cd7838c626af79a4edcd8b8437205cc02 <https://phabricator.kde.org/R223:9ba8dd2cd7838c626af79a4edcd8b8437205cc02> , which copies the list of loaded generators from a QHash to a QMap: Why not use a QMap for them right away? Certainly the number of generators is low enough such that the efficiency differences between a QHash and a QMap become negligible? REPOSITORY R223 Okular REVISION DETAIL https://phabricator.kde.org/D14820 To: aacid Cc: sander, okular-devel, ngraham, aacid
