Review Request 117275: Deprecate the catalog name stuff from KAboutData

Alex Merry alex.merry at kde.org
Tue Apr 1 01:38:00 UTC 2014


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

Review request for KDE Frameworks, David Faure, Kevin Ottens, and Michael Pyne.


Repository: kcoreaddons


Description
-------

This is another thing on the "if only we'd spotted it before beta1" list.  Unfortunately, removing the second argument from the constructor is completely ambiguous for calls that do not have at least enough arguments to specify the license.


Deprecate the catalog name stuff from KAboutData

This is pretty useless - the translation catalog has to be set before
KAboutData is constructed in order to translate its arguments.


Diffs
-----

  src/lib/kaboutdata.h cff1e3f67e33657fdd265a82166ef2a04cbcc3d1 
  src/lib/kaboutdata.cpp ce64a13aaa89bb4bc077f05e5f8e175d6a441ead 

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


Testing
-------

Builds, tests pass.


Thanks,

Alex Merry

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


More information about the Kde-frameworks-devel mailing list