Review Request 121096: convert qDebug and qWarning to kDebug and kWarning

René J.V. Bertin rjvbertin at gmail.com
Mon Nov 13 16:40:37 GMT 2017


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/121096/
-----------------------------------------------------------

(Updated nov. 13, 2017, 4:40 après-midi)


Status
------

This change has been discarded.


Review request for KDEPIM.


Repository: kdepim-runtime


Description
-------

This patch does as said in the summary. Rationale is to avoid log pollution, storing potentially sensitive data in unexpected locations and waste of CPU cycles.
First impressions are that disabling debug output (which is now under kdebugdialog control) does indeed make KMail considerably "snappier" (at least when combined with akonadi built with -DQT_NO_DEBUG_OUTPUT).

This also makes kdepim-runtime adhere more closely to KDE debugging output guidelines.


Diffs
-----

  agents/newmailnotifier/newmailnotifiershowmessagejob.cpp cbbcbed 
  qml/kde/kdeintegrationplugin.cpp c1360b4 
  qml/kde/tests/qml_moves/dynamictreemodel.cpp 5537bb7 
  qml/kde/tests/qml_moves/mainwindow.cpp 1324575 
  qml/kde/tests/qmlbreadcrumbnavigation/checkableitemproxymodel.cpp f61485a 
  qml/kde/tests/qmlbreadcrumbnavigation/kmodelindexproxymapper.cpp c464938 
  qml/kde/tests/qmlbreadcrumbnavigation/kproxyitemselectionmodel.cpp 6d176fb 
  qml/kde/tests/qmlbreadcrumbnavigation/qmllistselectionmodel.cpp dbc794e 
  resources/imap/tests/testsubscriptiondialog.cpp 9653b76 
  resources/kolabproxy/kolabhandler.cpp 0574464 
  resources/kolabproxy/upgradejob.cpp 74f80fe 
  resources/maildir/libmaildir/maildir.cpp 9ad4a66 
  resources/mixedmaildir/kmindexreader/tests/testidxreader.cpp d51cc9d 
  resources/mixedmaildir/tests/itemfetchtest.cpp 5372ce5 
  resources/openxchange/openxchangeresource.cpp 377b908 
  resources/openxchange/oxa/incidenceutils.cpp 1f0592b 
  resources/pop3/tests/fakeserver/fakeserver.cpp 2201045 
  resources/pop3/tests/pop3test.cpp f6b0f2a 


Diff: https://git.reviewboard.kde.org/r/121096/diff/2/


Testing
-------

Ubuntu 14.04 with kdelibs 4.14.2 .


Thanks,

René J.V. Bertin

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-pim/attachments/20171113/25e5cb6a/attachment.html>


More information about the kde-pim mailing list