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