Review Request 127003: KF5 port : Port away from kdebug to use qDebug
Laurent Montel
montel at kde.org
Mon Feb 15 06:42:35 UTC 2016
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/127003/#review92375
-----------------------------------------------------------
it will better to use qCDebug(CATEGORY...) <<
you can generate category debug with cmake method:
see ecm_qt_declare_logging_category(ktnef_SRCS HEADER ktnef_debug.h IDENTIFIER KTNEFAPPS_LOG CATEGORY_NAME log_ktnefapps)
=> you can use this category log KTNEFAPPS_LOG (it's an example you need to adapt to kdf)
=> so after that we can remove debug with external application.
- Laurent Montel
On fév. 8, 2016, 12:11 après-midi, Yoann Laissus wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/127003/
> -----------------------------------------------------------
>
> (Updated fév. 8, 2016, 12:11 après-midi)
>
>
> Review request for KDE Utils.
>
>
> Repository: kdf
>
>
> Description
> -------
>
> KF5 port : Replace all kDebug by their qDebug equivalent.
> It also get rid of a legacy test class which still use Qt3Support.
>
> It's the first step before the real port : see #126139
>
>
> Diffs
> -----
>
> disklist.cpp 12e12353748bbc46db013b90b5292c0cfa8f6c2f
> disks.cpp 83f9729021fd24e2a19d333f5ef1503d7760e684
> kconftest.cpp 05d09d372a09f72be8134c77f956919d24182c9c
> kdf.cpp c1f8aa64526623a60b37c65319c6e933d66d3e35
> kdfwidget.cpp 9312365631e6e766e5700c57ec17b1064073ed67
> kwikdisk.cpp 713ebfc056c82a4101b736f46b4803e75db7e7e0
> mntconfig.cpp 33cd92a9b3325dca4d4b2038b904f9ee8d16e71f
>
> Diff: https://git.reviewboard.kde.org/r/127003/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> Yoann Laissus
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-utils-devel/attachments/20160215/49a63fc9/attachment.html>
More information about the Kde-utils-devel
mailing list