Review Request 112689: Drop KI18n dependencies in KCompletion
Aleix Pol Gonzalez
aleixpol at kde.org
Fri Sep 13 11:01:55 UTC 2013
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/112689/
-----------------------------------------------------------
(Updated Sept. 13, 2013, 11:01 a.m.)
Review request for KDE Frameworks.
Changes
-------
Properly use tr() as a replacement for i18nc()
Description
-------
It's not really adding much value here, so I'd say we can drop it.
I'm not sure that tr() is as well the correct replacement for i18nc(), can anybody confirm?
Diffs (updated)
-----
staging/kcompletion/src/CMakeLists.txt dd9d99c
staging/kcompletion/src/kcombobox.cpp 8a73a1b
staging/kcompletion/src/khistorycombobox.cpp 2e53bd9
staging/kcompletion/src/klineedit.cpp 76ef377
staging/knotifications/CMakeLists.txt 2e42ab3
Diff: http://git.reviewboard.kde.org/r/112689/diff/
Testing
-------
builds, tests pass.
Thanks,
Aleix Pol Gonzalez
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20130913/10b7ba72/attachment.html>
More information about the Kde-frameworks-devel
mailing list