D14820: Select current format on the configure backend dialog

Oliver Sander noreply at phabricator.kde.org
Tue Aug 14 12:15:22 BST 2018


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
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/okular-devel/attachments/20180814/4b506b99/attachment.html>


More information about the Okular-devel mailing list