Re: Review Request 123867: Fix build on OSX clang and replace another abs with std::abs

2015-05-21 Thread Bhushan Shah
On May 21, 2015, 1:31 p.m., Milian Wolff wrote: src/klocalizedstring.cpp, line 23 https://git.reviewboard.kde.org/r/123867/diff/1/?file=370328#file370328line23 shouldn't this be #include cmath Bhushan Shah wrote: ah yeah. meh; didn't fix it.. - Bhushan

Re: Review Request 123867: Fix build on OSX clang and replace another abs with std::abs

2015-05-21 Thread Milian Wolff
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123867/#review80680 --- Ship it! Ship It! src/klocalizedstring.cpp (line 23)

Re: Review Request 123867: Fix build on OSX clang and replace another abs with std::abs

2015-05-21 Thread Bhushan Shah
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123867/ --- (Updated May 21, 2015, 8:19 a.m.) Status -- This change has been

Re: Review Request 123867: Fix build on OSX clang and replace another abs with std::abs

2015-05-21 Thread Bhushan Shah
On May 21, 2015, 1:31 p.m., Milian Wolff wrote: src/klocalizedstring.cpp, line 23 https://git.reviewboard.kde.org/r/123867/diff/1/?file=370328#file370328line23 shouldn't this be #include cmath ah yeah. - Bhushan

Re: Review Request 123857: Fix crash after a user has launched kbuildsycoca as root.

2015-05-21 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123857/ --- (Updated May 21, 2015, 11:36 a.m.) Review request for KDE Frameworks.

Re: Review Request 123735: version of QmlObject with a static engine

2015-05-21 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123735/ --- (Updated May 21, 2015, 10:52 a.m.) Review request for KDE Frameworks and

KI18n programmer's guide mentions non-existent kcfgc parameter TranslationDomain

2015-05-21 Thread Alexander Potashev
Hi, The current ki18n/docs/programmers-guide.md says that a TranslationDomain field in .kcfgc file can be used to set the translation domain. But in fact, KConfig's kconfig_compiler_kf5 does not read that field. Is there a plan to revive support for TranslationDomain in kconfig_compiler_kf5, or

Review Request 123872: Add TranslationDomain attribute to kconfig_compiler

2015-05-21 Thread Chusslove Illich
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123872/ --- Review request for KDE Frameworks, Alexander Potashev and Matthew Dawson.

Re: Review Request 123872: Add TranslationDomain attribute to kconfig_compiler

2015-05-21 Thread Matthew Dawson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123872/#review80698 --- LGTM in general, but could you please add/update a unit test

Re: Review Request 123735: version of QmlObject with a static engine

2015-05-21 Thread Vishesh Handa
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123735/#review80695 --- Ship it! Awesome. I love the test!

Re: Review Request 123735: version of QmlObject with a static engine

2015-05-21 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123735/ --- (Updated May 21, 2015, 4:02 p.m.) Review request for KDE Frameworks and