Review Request 115351: Use kDebug instead of qDebug in QSpellEnchantDict

Kevin Funk kfunk at kde.org
Wed Mar 5 11:28:38 GMT 2014


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


Heh, the Git history looks very sad in this part of kdelibs. Noone worked on this for years now.

Personally I'd just get rid off the debug output, seems like noone cares anyway. (I'm also annoyed by the useless output)

- Kevin Funk


On Jan. 28, 2014, 1 p.m., Milian Wolff wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/115351/
> -----------------------------------------------------------
> 
> (Updated Jan. 28, 2014, 1 p.m.)
> 
> 
> Review request for kdelibs.
> 
> 
> Repository: kdelibs
> 
> 
> Description
> -------
> 
> Enabling automatic spell checking in Kate, I see a lot of debug output in the form of:
> 
> Enchant dict for "en_US" 0x43a1480 
> 
> This patch should fix this and only output the lines when debug output is enabled via kdebugdialog.
> 
> 
> Diffs
> -----
> 
>   sonnet/plugins/enchant/enchantdict.cpp c289d26 
> 
> Diff: https://git.reviewboard.kde.org/r/115351/diff/
> 
> 
> Testing
> -------
> 
> None actually, not even tested if it compiles.
> 
> 
> Thanks,
> 
> Milian Wolff
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20140305/93c63aca/attachment.htm>


More information about the kde-core-devel mailing list