D18420: Optimize and reduce code in GuiObserver

Christoph Roick noreply at phabricator.kde.org
Sun Jan 20 21:35:16 GMT 2019


croick added a comment.


  Would you agree on such a change?
  `case "Worksheet"_hash:` is still human-readable (compare to `if (className == "Worksheet")`.
  The very unlikely case of a hash collision would appear at compile time (if new dock widgets are introduced).
  On the other hand the selection of the right dock is now more efficient than before.

REPOSITORY
  R262 LabPlot

REVISION DETAIL
  https://phabricator.kde.org/D18420

To: croick, #labplot
Cc: kde-edu, narvaez, apol
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-edu/attachments/20190120/24bccb3b/attachment.html>


More information about the kde-edu mailing list