Review Request 123268: Fix extraction of some messages

Lasse Liehu lasse.liehu at gmail.com
Mon Apr 6 14:36:26 UTC 2015


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

(Updated April 6, 2015, 2:36 p.m.)


Status
------

This change has been marked as submitted.


Review request for KDE Frameworks and Christoph Feck.


Changes
-------

Submitted with commit 5fcad6720b2815ca85b9a67f086900883e95734c by Lasse Liehu to branch master.


Repository: kwidgetsaddons


Description
-------

Wrapping QCoreApplication::translate calls into a new function broke extraction.

Also ported remaining I18N_NOOP2 uses to Qt's i18n system.

Added semicolons to the end of QT_TRANSLATE_NOOP3 uses because without them only the first //: comment would be extracted.


Diffs
-----

  src/fonthelpers.cpp 1a1a2f4 
  src/kdatetimeedit.cpp 63686ad 
  src/kmessagebox.cpp 7c2a577 
  src/kmimetypechooser.cpp 51c4044 

Diff: https://git.reviewboard.kde.org/r/123268/diff/


Testing
-------

Compiled and ran unit tests. Also ran lrelease locally and checked that messages and their //: comments were extracted properly.


Thanks,

Lasse Liehu

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20150406/ad1a82b8/attachment.html>


More information about the Kde-frameworks-devel mailing list