D9330: Fix cmake warning (with cmake >= 3.9.0)

2017-12-14 Thread Laurent Montel
mlaurent abandoned this revision. mlaurent added a comment. commited in kcoreaddons REPOSITORY R241 KIO REVISION DETAIL https://phabricator.kde.org/D9330 To: mlaurent, dfaure Cc: kossebau, flherne, #frameworks

Re: KHTML Commit 77c21d19c314d79da1b60e319aa40eff57fef375

2017-12-14 Thread laurent Montel
7fef375 > > Can someone please fix this? > > At the moment it's breaking the build of various components which > depend on KHTML and as a consequence has left the CI system in an > inconsistent state. At this time both Applications and Plasma have > been negatively impac

D9332: Allow filter places based on alternative application name

2017-12-15 Thread Laurent Montel
mlaurent accepted this revision. This revision is now accepted and ready to land. REPOSITORY R241 KIO REVISION DETAIL https://phabricator.kde.org/D9332 To: renatoo, mwolff, mlaurent Cc: mlaurent, aacid, ngraham, #frameworks

Re: KHTML Commit 77c21d19c314d79da1b60e319aa40eff57fef375

2017-12-15 Thread laurent Montel
Le vendredi 15 décembre 2017, 18:47:40 CET Ben Cooksley a écrit : > On Fri, Dec 15, 2017 at 7:41 PM, laurent Montel wrote: > > Le vendredi 15 décembre 2017, 03:38:32 CET Ben Cooksley a écrit : > >> Hi Laurent, all, > > > > Hi, > > Indeed fixed now. >

D9332: Allow filter places based on alternative application name

2017-12-19 Thread Laurent Montel
mlaurent added a comment. Not commited ? REPOSITORY R241 KIO REVISION DETAIL https://phabricator.kde.org/D9332 To: renatoo, mwolff, mlaurent Cc: mlaurent, aacid, ngraham, #frameworks

D9420: Compile commenting remove_defintion(QT_NO_CAST_FROM_ASCII defination)

2017-12-20 Thread Laurent Montel
mlaurent added a comment. Could you paste you compile error here please. INLINE COMMENTS > CMakeLists.txt:1 > -remove_definitions(-DQT_NO_CAST_FROM_ASCII) > +#remove_definitions(-DQT_NO_CAST_FROM_ASCII) > Better to remove it. REPOSITORY R244 KCoreAddons REVISION DETAIL https://phabri

D9420: Compile commenting remove_defintion(QT_NO_CAST_FROM_ASCII defination)

2017-12-20 Thread Laurent Montel
mlaurent requested changes to this revision. This revision now requires changes to proceed. REPOSITORY R244 KCoreAddons REVISION DETAIL https://phabricator.kde.org/D9420 To: velurimithun, mlaurent Cc: #frameworks

D9420: Compile commenting remove_defintion(QT_NO_CAST_FROM_ASCII defination)

2017-12-20 Thread Laurent Montel
mlaurent added a comment. KAboutData aboutData2(QString::fromLatin1(AppName2), QString::fromLatin1(ProgramName2), QString::fromLatin1(Version2)); use QString::fromLatin1 is will work REPOSITORY R244 KCoreAddons REVISION DETAIL https://phabricator.kde.org/D9420 To: velurimithun, mla

D9420: Compile commenting remove_defintion(QT_NO_CAST_FROM_ASCII defination)

2017-12-20 Thread Laurent Montel
mlaurent added a comment. Arguiment from method use QString and not QLatin1String() QString::fromLatin1 returns a QString REPOSITORY R244 KCoreAddons REVISION DETAIL https://phabricator.kde.org/D9420 To: velurimithun, mlaurent Cc: #frameworks

D9420: Compile commenting remove_defintion(QT_NO_CAST_FROM_ASCII )

2017-12-20 Thread Laurent Montel
mlaurent requested changes to this revision. mlaurent added inline comments. This revision now requires changes to proceed. INLINE COMMENTS > kaboutdatatest.cpp:48 > > -static const char AppName[] ="app"; > -static const char ProgramName[] ="ProgramName"; > -static const cha

D9420: Compile commenting remove_defintion(QT_NO_CAST_FROM_ASCII )

2017-12-20 Thread Laurent Montel
mlaurent added a comment. Missing all patch... please see "https://community.kde.org/Infrastructure/Phabricator"; "Step 2: Updating your diff. After you upload the code the reviewer will take a look and give you some comments. If you get a thumbs up, you can skip this step. But ofte

D9420: Compile commenting remove_defintion(QT_NO_CAST_FROM_ASCII )

2017-12-20 Thread Laurent Montel
mlaurent added inline comments. INLINE COMMENTS > desktoptojsontest.cpp:84 > QByteArray input = > -// include an insignificant group > -"[Some Group]\n" > -"Foo=Bar\n" > -"\n" > -"[Desktop Entry]\n" > -// only data insid

D9420: Compile commenting remove_defintion(QT_NO_CAST_FROM_ASCII )

2017-12-20 Thread Laurent Montel
mlaurent added inline comments. INLINE COMMENTS > velurimithun wrote in kprocesstest.cpp:93 > These strings are going as arguments for QStringList() method that why I used > QString::FromLatin1() ? QStringLiteral works fine with QStringList REPOSITORY R244 KCoreAddons REVISION DETAIL http

D9446: WIP: Allow to autogenerate and install categories file

2017-12-20 Thread Laurent Montel
mlaurent created this revision. Restricted Application added projects: Frameworks, Build System. Restricted Application added subscribers: Build System, Frameworks. TEST PLAN Generate kmail categories file REPOSITORY R240 Extra CMake Modules BRANCH autogenerate_categories_file REVISION DE

D9446: WIP: Allow to autogenerate and install categories file

2017-12-20 Thread Laurent Montel
mlaurent added a comment. IT's a WIP as I use file(APPEND...) because I want to generate several categories in one file. But it doesn't work as I don't have idea how to reset file when cmake is started... Do you have an idea ? I will implement autotest and co if we have a solution

D9446: WIP: Allow to autogenerate and install categories file

2017-12-21 Thread Laurent Montel
mlaurent added reviewers: kfunk, lbeltrame, cgiboudeaux, dfaure. REPOSITORY R240 Extra CMake Modules REVISION DETAIL https://phabricator.kde.org/D9446 To: mlaurent, kfunk, lbeltrame, cgiboudeaux, dfaure Cc: #frameworks, #build_system

D9420: Compile commenting remove_defintion(QT_NO_CAST_FROM_ASCII )

2017-12-21 Thread Laurent Montel
mlaurent added a comment. You still missing all patch... Please use arc diff so we are sure that all patch is updated. REPOSITORY R244 KCoreAddons REVISION DETAIL https://phabricator.kde.org/D9420 To: velurimithun, mlaurent Cc: #frameworks

D9491: Sonnet: fix wrong language for suggestions in mixed-language texts

2017-12-24 Thread Laurent Montel
mlaurent accepted this revision. mlaurent added a comment. This revision is now accepted and ready to land. Good thanks ! REPOSITORY R246 Sonnet BRANCH master REVISION DETAIL https://phabricator.kde.org/D9491 To: dfaure, sandsmark, mlaurent Cc: #kde_pim, #frameworks

D9420: Compile commenting remove_defintion(QT_NO_CAST_FROM_ASCII )

2017-12-27 Thread Laurent Montel
mlaurent added a comment. no news ? REPOSITORY R244 KCoreAddons REVISION DETAIL https://phabricator.kde.org/D9420 To: velurimithun, mlaurent Cc: #frameworks

D9420: Compile commenting remove_defintion(QT_NO_CAST_FROM_ASCII )

2017-12-30 Thread Laurent Montel
mlaurent added inline comments. INLINE COMMENTS > kprocesstest.cpp:47 > p.setProcessChannelMode(QProcess::MergedChannels); > -p.start(gargv[0], QStringList() << QString::number(how) << > QStringLiteral("--nocrashhandler")); > +p.start(QString::fromLatin1(gargv[0]), QStringList() <<

D9571: clang-tidy modernize-use-override

2017-12-31 Thread Laurent Montel
mlaurent accepted this revision. This revision is now accepted and ready to land. REPOSITORY R39 KTextEditor BRANCH UseOverride (branched from master) REVISION DETAIL https://phabricator.kde.org/D9571 To: dhaumann, kfunk, mlaurent Cc: #frameworks, kevinapavew, ngraham, demsking, cullmann,

D9420: Compile commenting remove_defintion(QT_NO_CAST_FROM_ASCII )

2018-01-09 Thread Laurent Montel
mlaurent added a comment. indeed code will be broken if we don't use QString::fromUtf8. Could you fix it please ? REPOSITORY R244 KCoreAddons REVISION DETAIL https://phabricator.kde.org/D9420 To: velurimithun, mlaurent Cc: dfaure, #frameworks

D9420: Compile commenting remove_defintion(QT_NO_CAST_FROM_ASCII )

2018-01-10 Thread Laurent Montel
mlaurent requested changes to this revision. mlaurent added a comment. This revision now requires changes to proceed. When I try to compile your patch I have this compile error: Do you are sure that you rebuild all ? /compile/kde5/framework/frameworks/kcoreaddons/autotests/kaboutdatatest

D9420: Compile commenting remove_defintion(QT_NO_CAST_FROM_ASCII )

2018-01-11 Thread Laurent Montel
mlaurent added a comment. Ok I understand why you don't have this problem you need to rebase patch with last kcoreaddons version. So code was added. Please rebase, make sure that it builds and upload patch. Regards. REPOSITORY R244 KCoreAddons REVISION DETAIL https://phabricator.kd

D9884: KMultiTabBar: Fix regression in conversion to new style connect()

2018-01-15 Thread Laurent Montel
mlaurent accepted this revision. mlaurent added a comment. This revision is now accepted and ready to land. indeed good catch REPOSITORY R236 KWidgetsAddons REVISION DETAIL https://phabricator.kde.org/D9884 To: sars, #frameworks, mlaurent

D8908: KDateComboBox: fix dateChanged() not emitted after typing a date

2018-01-15 Thread Laurent Montel
mlaurent accepted this revision. mlaurent added a comment. This revision is now accepted and ready to land. Otherwise all is good. +2 with the QVERIFY(...) INLINE COMMENTS > kdatecomboboxtest.cpp:182 > +combo->activateWindow(); > +QTest::qWaitForWindowActive(combo.data()); > + QVERIF

D9420: Compile commenting remove_defintion(QT_NO_CAST_FROM_ASCII )

2018-01-15 Thread Laurent Montel
mlaurent added a comment. Ping ? REPOSITORY R244 KCoreAddons REVISION DETAIL https://phabricator.kde.org/D9420 To: velurimithun, mlaurent Cc: dfaure, #frameworks

D8908: KDateComboBox: fix dateChanged() not emitted after typing a date

2018-01-16 Thread Laurent Montel
mlaurent accepted this revision. REPOSITORY R236 KWidgetsAddons BRANCH master REVISION DETAIL https://phabricator.kde.org/D8908 To: dfaure, cfeck, dvratil, mlaurent Cc: mlaurent, anthonyfieroni, #frameworks, #kde_pim

D9420: Compile commenting remove_defintion(QT_NO_CAST_FROM_ASCII )

2018-01-20 Thread Laurent Montel
mlaurent added a comment. Ping bis ?:) REPOSITORY R244 KCoreAddons REVISION DETAIL https://phabricator.kde.org/D9420 To: velurimithun, mlaurent Cc: dfaure, #frameworks

D9420: Compile commenting remove_defintion(QT_NO_CAST_FROM_ASCII )

2018-01-21 Thread Laurent Montel
mlaurent requested changes to this revision. mlaurent added a comment. This revision now requires changes to proceed. You still forgot to launch autotest... see: "laurent@linux-5nvn:/compile/kde5/framework/frameworks/kcoreaddons/build> ./bin/kaboutdatatest - Start testing of KAboutDat

D9420: Compile commenting remove_defintion(QT_NO_CAST_FROM_ASCII )

2018-01-22 Thread Laurent Montel
mlaurent accepted this revision. mlaurent added a comment. This revision is now accepted and ready to land. Thanks REPOSITORY R244 KCoreAddons BRANCH complieWithoutRemoveDef REVISION DETAIL https://phabricator.kde.org/D9420 To: velurimithun, mlaurent Cc: dfaure, #frameworks

D9420: Compile commenting remove_defintion(QT_NO_CAST_FROM_ASCII )

2018-01-23 Thread Laurent Montel
mlaurent added a comment. I made a fix. I wait a rebuild on windows to see if it's ok REPOSITORY R244 KCoreAddons REVISION DETAIL https://phabricator.kde.org/D9420 To: velurimithun, mlaurent Cc: bcooksley, dfaure, #frameworks

D10135: Put built-in holiday definitions into qrc

2018-01-27 Thread Laurent Montel
mlaurent accepted this revision. mlaurent added inline comments. This revision is now accepted and ready to land. INLINE COMMENTS > CMakeLists.txt:8 > set(CMAKE_MODULE_PATH ${ECM_MODULE_PATH}) > +set(CMAKE_AUTORCC ON) > Just curious it's not default in ECM ? I never used it. REPOSITORY R17

D10333: Introduce K_PLUGIN_CLASS_WITH_JSON

2018-02-07 Thread Laurent Montel
mlaurent requested changes to this revision. mlaurent added inline comments. This revision now requires changes to proceed. INLINE COMMENTS > kpluginfactory.h:211 > + */ > +#define K_PLUGIN_CLASS_WITH_JSON(classname, jsonFile) > K_PLUGIN_FACTORY_WITH_JSON(classname ## Factory, json, > registerP

D9446: WIP: Allow to autogenerate and install categories file

2018-02-08 Thread Laurent Montel
mlaurent abandoned this revision. REPOSITORY R240 Extra CMake Modules REVISION DETAIL https://phabricator.kde.org/D9446 To: mlaurent, kfunk, lbeltrame, cgiboudeaux, dfaure, fvogt Cc: #frameworks, #build_system, michaelh, ngraham

D10744: Use correct directory to lookup trigrams in windows build dir

2018-03-03 Thread Laurent Montel
mlaurent accepted this revision. mlaurent added a comment. This revision is now accepted and ready to land. Seems good for me REPOSITORY R246 Sonnet BRANCH master REVISION DETAIL https://phabricator.kde.org/D10744 To: vonreth, mlaurent, dfaure, leinir Cc: #frameworks, michaelh

D11285: Introduce sanitizer class

2018-03-15 Thread Laurent Montel
mlaurent requested changes to this revision. mlaurent added inline comments. This revision now requires changes to proceed. INLINE COMMENTS > databasesanitizer.cpp:44 > +struct FileInfo { > +quint32 deviceId; > +quint32 inode; initialize value by default please > databasesa

D11285: Introduce sanitizer class

2018-03-15 Thread Laurent Montel
mlaurent added a comment. Is it possible to create an autotest for this class ? REPOSITORY R293 Baloo REVISION DETAIL https://phabricator.kde.org/D11285 To: michaelh, #baloo, #frameworks, ngraham, mlaurent Cc: mlaurent, ngraham, smithjd, ashaposhnikov, michaelh, spoorun, nicolasfella, a

D11285: Introduce sanitizer class

2018-03-16 Thread Laurent Montel
mlaurent requested changes to this revision. mlaurent added inline comments. This revision now requires changes to proceed. INLINE COMMENTS > databasesanitizer.cpp:46 > +public: > +FileInfo(const quint32 d = 0, const quint32 i = 0, > +const QString& u = QString(), con

D11285: Introduce sanitizer class

2018-03-16 Thread Laurent Montel
mlaurent accepted this revision. mlaurent added a comment. This revision is now accepted and ready to land. +2 for me but I don't know baloo code :) REPOSITORY R293 Baloo BRANCH sanitize-class (branched from master) REVISION DETAIL https://phabricator.kde.org/D11285 To: michaelh, #bal

D11287: Introduce baloodb CLI tool

2018-03-19 Thread Laurent Montel
mlaurent requested changes to this revision. mlaurent added inline comments. This revision now requires changes to proceed. INLINE COMMENTS > main.cpp:103 > +QStringLiteral("devices"), > +i18n("List devices "), > +QStringList{}, extra space at the end of i18n > main.cpp

D11287: Introduce baloodb CLI tool

2018-03-20 Thread Laurent Montel
mlaurent requested changes to this revision. mlaurent added inline comments. This revision now requires changes to proceed. INLINE COMMENTS > CMakeLists.txt:16 > + > +install(TARGETS baloodb DESTINATION ${BIN_INSTALL_DIR}) install(TARGETS baloodb ${KDE_INSTALL_TARGETS_DEFAULT_ARGS}) > main.cpp:

D11287: Introduce baloodb CLI tool

2018-03-20 Thread Laurent Montel
mlaurent added inline comments. INLINE COMMENTS > michaelh wrote in CMakeLists.txt:16 > @mlaurent Is it worth the effort to change this for the other cli tools > accordingly? It's the default variable. But don't chnage it in this patch. Perhaps in the future as you are the baloo maintainer :)

D11287: Introduce baloodb CLI tool

2018-03-20 Thread Laurent Montel
mlaurent accepted this revision. This revision is now accepted and ready to land. REPOSITORY R293 Baloo BRANCH sanitize-cmd (branched from master) REVISION DETAIL https://phabricator.kde.org/D11287 To: michaelh, #baloo, #frameworks, mlaurent Cc: mlaurent, ngraham, ashaposhnikov, michaelh,

D15136: Fixed a typeo

2018-08-28 Thread Laurent Montel
mlaurent accepted this revision. mlaurent added a comment. This revision is now accepted and ready to land. Thanks can you commit it ? Do you have commit access ? REPOSITORY R293 Baloo BRANCH bug397843 (branched from master) REVISION DETAIL https://phabricator.kde.org/D15136 To: ssl

D15136: Fixed a typeo

2018-08-28 Thread Laurent Montel
mlaurent added a comment. I will commit for you. what is your email address ? REPOSITORY R293 Baloo BRANCH bug397843 (branched from master) REVISION DETAIL https://phabricator.kde.org/D15136 To: ssloan, mlaurent Cc: mlaurent, kde-frameworks-devel, #baloo, ashaposhnikov, michaelh, asti

D15136: Fixed a typeo

2018-08-29 Thread Laurent Montel
mlaurent closed this revision. REPOSITORY R293 Baloo REVISION DETAIL https://phabricator.kde.org/D15136 To: ssloan, mlaurent Cc: mlaurent, kde-frameworks-devel, #baloo, ashaposhnikov, michaelh, astippich, spoorun, ngraham, bruns, abrahams

D15136: Fixed a typeo

2018-08-29 Thread Laurent Montel
mlaurent added a comment. done. REPOSITORY R293 Baloo BRANCH bug397843 (branched from master) REVISION DETAIL https://phabricator.kde.org/D15136 To: ssloan, mlaurent Cc: mlaurent, kde-frameworks-devel, #baloo, ashaposhnikov, michaelh, astippich, spoorun, ngraham, bruns, abrahams

D6513: Add support for Attica tags support

2018-09-05 Thread Laurent Montel
mlaurent accepted this revision. REPOSITORY R304 KNewStuff REVISION DETAIL https://phabricator.kde.org/D6513 To: leinir, #knewstuff, apol, #kde_store, whiting, ahiemstra, mlaurent, dfaure, cfeck Cc: dfaure, cfeck, mlaurent, ngraham, ahiemstra, kde-frameworks-devel, #knewstuff, michaelh, Zr

D15566: Add QT_NO_NARROWING_CONVERSIONS_IN_CONNECT as default compile flags

2018-09-16 Thread Laurent Montel
mlaurent created this revision. Herald added projects: Frameworks, Build System. Herald added subscribers: kde-buildsystem, kde-frameworks-devel. mlaurent requested review of this revision. REVISION SUMMARY Use QT_NO_NARROWING_CONVERSIONS_IN_CONNECT as default flags REPOSITORY R240 Extra CMak

D15566: Add QT_NO_NARROWING_CONVERSIONS_IN_CONNECT as default compile flags

2018-09-16 Thread Laurent Montel
mlaurent added a reviewer: dfaure. REPOSITORY R240 Extra CMake Modules REVISION DETAIL https://phabricator.kde.org/D15566 To: mlaurent, dfaure Cc: kde-frameworks-devel, kde-buildsystem, michaelh, ngraham, bruns

D15566: Add QT_NO_NARROWING_CONVERSIONS_IN_CONNECT as default compile flags

2018-09-17 Thread Laurent Montel
mlaurent added a comment. I fixed 2 repos. Otherwise all compiles fine (ok I don't test on windows/mac and perhaps some module is not compiles on my computer) REPOSITORY R240 Extra CMake Modules REVISION DETAIL https://phabricator.kde.org/D15566 To: mlaurent, dfaure Cc: kde-frameworks

D15566: Add QT_NO_NARROWING_CONVERSIONS_IN_CONNECT as default compile flags

2018-09-22 Thread Laurent Montel
mlaurent added a comment. Ping? REPOSITORY R240 Extra CMake Modules REVISION DETAIL https://phabricator.kde.org/D15566 To: mlaurent, dfaure Cc: kde-frameworks-devel, kde-buildsystem, michaelh, ngraham, bruns

D15566: Add QT_NO_NARROWING_CONVERSIONS_IN_CONNECT as default compile flags

2018-09-22 Thread Laurent Montel
This revision was automatically updated to reflect the committed changes. Closed by commit R240:9b528012703a: Add QT_NO_NARROWING_CONVERSIONS_IN_CONNECT as default compile flags (authored by mlaurent). REPOSITORY R240 Extra CMake Modules CHANGES SINCE LAST UPDATE https://phabricator.kde.org/

D15778: Log wallet open failure errors

2018-09-26 Thread Laurent Montel
mlaurent requested changes to this revision. mlaurent added inline comments. This revision now requires changes to proceed. INLINE COMMENTS > kwalletd.cpp:685 > +const auto errorStr = > KWallet::Backend::openRCToString(rc); > +qWarn

D15778: Log wallet open failure errors

2018-09-28 Thread Laurent Montel
mlaurent accepted this revision. This revision is now accepted and ready to land. REVISION DETAIL https://phabricator.kde.org/D15778 To: arjun, mlaurent Cc: mlaurent, kde-frameworks-devel, michaelh, ngraham, bruns

D15277: [RFC] kio_mtp: Move MTP device handling from kioslave to kiod-module

2018-09-29 Thread Laurent Montel
mlaurent requested changes to this revision. mlaurent added inline comments. This revision now requires changes to proceed. INLINE COMMENTS > mtpstorage.cpp:176 > +case LIBMTP_FILETYPE_UNDEF_AUDIO: > +return QStringLiteral(""); > +case LIBMTP_FILETYPE_WMV: QString() directly no ?

D16001: ktextedit: lazy load the QTextToSpeech object

2018-10-07 Thread Laurent Montel
mlaurent accepted this revision. This revision is now accepted and ready to land. REPOSITORY R310 KTextWidgets BRANCH lazy-qtexttospeech (branched from master) REVISION DETAIL https://phabricator.kde.org/D16001 To: pino, mlaurent Cc: kde-frameworks-devel, michaelh, ngraham, bruns

D15277: [mtp] Move MTP device handling from kioslave to kiod-module

2018-10-07 Thread Laurent Montel
mlaurent accepted this revision. This revision is now accepted and ready to land. REPOSITORY R320 KIO Extras BRANCH mtp_next (branched from master) REVISION DETAIL https://phabricator.kde.org/D15277 To: akrutzler, elvisangelaccio, ltoscano, hetzenecker, dfaure, mlaurent Cc: mlaurent, kde-

D16306: [Codecs] Remove unneeded const_cast

2018-10-18 Thread Laurent Montel
mlaurent added a reviewer: dfaure. REPOSITORY R293 Baloo REVISION DETAIL https://phabricator.kde.org/D16306 To: bruns, #baloo, #frameworks, poboiko, ngraham, dfaure Cc: kde-frameworks-devel, ashaposhnikov, michaelh, astippich, spoorun, ngraham, bruns, abrahams

D16350: Allow KHelpCenter to open the right pages of KDE help when KHelpClient is invoked with an anchor

2018-10-23 Thread Laurent Montel
mlaurent accepted this revision. mlaurent added a comment. This revision is now accepted and ready to land. seems ok for me REPOSITORY R265 KConfigWidgets REVISION DETAIL https://phabricator.kde.org/D16350 To: yurchor, #documentation, #frameworks, mlaurent Cc: mlaurent, kde-frameworks-de

D14631: Adds a new RenameDialog to KIO with more options for batch renaming

2018-10-24 Thread Laurent Montel
mlaurent requested changes to this revision. mlaurent added inline comments. This revision now requires changes to proceed. INLINE COMMENTS > CMakeLists.txt:65 > + rename/batchrenamedialog.cpp > +rename/batchrenamedialogmodel_p.cpp > +rename/batchrenametypes_p.cpp fix indent >

D14631: Adds a new RenameDialog to KIO with more options for batch renaming

2018-10-25 Thread Laurent Montel
mlaurent requested changes to this revision. mlaurent added inline comments. This revision now requires changes to proceed. INLINE COMMENTS > batchrenamedialog.cpp:40 > +#include > +#include > +#include remove QtWidget > batchrenamedialog.cpp:41 > +#include > +#include > + remove QtGui pr

D14631: Adds a new RenameDialog to KIO with more options for batch renaming

2018-11-03 Thread Laurent Montel
mlaurent added inline comments. INLINE COMMENTS > emateli wrote in batchrenamedialog.h:69 > You mean to initialize them to nullptr on the header file or using a > constructor initialization list? Also see what problem? in header file. REPOSITORY R241 KIO REVISION DETAIL https://phabricato

D14631: Adds a new RenameDialog to KIO with more options for batch renaming

2018-11-07 Thread Laurent Montel
mlaurent added a comment. For me seems good +1 REPOSITORY R241 KIO REVISION DETAIL https://phabricator.kde.org/D14631 To: emateli, #frameworks, dfaure, mlaurent Cc: mlaurent, asensi, rkflx, dfaure, aacid, ngraham, kde-frameworks-devel, michaelh, bruns

D14631: Adds a new RenameDialog to KIO with more options for batch renaming

2018-11-08 Thread Laurent Montel
mlaurent accepted this revision. REPOSITORY R241 KIO REVISION DETAIL https://phabricator.kde.org/D14631 To: emateli, #frameworks, dfaure, mlaurent Cc: mlaurent, asensi, rkflx, dfaure, aacid, ngraham, kde-frameworks-devel, michaelh, bruns

D16966: Fix typo in i18n message (cmdline option description)

2018-11-18 Thread Laurent Montel
mlaurent accepted this revision. This revision is now accepted and ready to land. REPOSITORY R290 KPackage BRANCH opt-i18n REVISION DETAIL https://phabricator.kde.org/D16966 To: aspotashev, mlaurent Cc: kde-frameworks-devel, michaelh, ngraham, bruns

D17065: Fix mem leak found with asan

2018-11-20 Thread Laurent Montel
mlaurent created this revision. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. mlaurent requested review of this revision. REVISION SUMMARY Fix mem when I used dolphin #0 0x7fa8cd24d900 in operator new(unsigned long) (/usr/lib64/libasan.so.5+0xed

D17065: Fix mem leak found with asan

2018-11-20 Thread Laurent Montel
mlaurent added a reviewer: dfaure. REPOSITORY R241 KIO REVISION DETAIL https://phabricator.kde.org/D17065 To: mlaurent, dfaure Cc: kde-frameworks-devel, michaelh, ngraham, bruns

D17065: Fix mem leak found with asan

2018-11-21 Thread Laurent Montel
mlaurent added a comment. Indeed a very good training "debugging and profiling" I fixed some potential other mem leak in kinfocenter/ark/libksysguard yesterday. Very useful indeed. REPOSITORY R241 KIO BRANCH fix_mem_leak (branched from master) REVISION DETAIL https://phabricato

D17065: Fix mem leak found with asan

2018-11-21 Thread Laurent Montel
This revision was automatically updated to reflect the committed changes. Closed by commit R241:8fb6431784e2: Fix mem leak found with asan (authored by mlaurent). REPOSITORY R241 KIO CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D17065?vs=45928&id=45935 REVISION DETAIL https://pha

D16966: Fix typo in i18n message (cmdline option description)

2018-12-17 Thread Laurent Montel
mlaurent added a comment. no commited yet ?:) REPOSITORY R290 KPackage BRANCH opt-i18n REVISION DETAIL https://phabricator.kde.org/D16966 To: aspotashev, mlaurent Cc: kde-frameworks-devel, michaelh, ngraham, bruns

D18424: Instantiate QApplication before KCrash/KCatalog

2019-01-20 Thread Laurent Montel
mlaurent accepted this revision. This revision is now accepted and ready to land. REPOSITORY R293 Baloo BRANCH master REVISION DETAIL https://phabricator.kde.org/D18424 To: sdepiets, #baloo, bruns, poboiko, mlaurent Cc: kde-frameworks-devel, #baloo, ashaposhnikov, michaelh, astippich, spo

Re: KDE CI: Frameworks » kcodecs » kf5-qt5 AndroidQt5.12 - Build # 15 - Failure!

2019-06-11 Thread laurent Montel
5.777Z] * Qt5Core > [2019-06-12T05:25:25.777Z] > [2019-06-12T05:25:25.777Z] -- The following features have been disabled: > [2019-06-12T05:25:25.777Z] > [2019-06-12T05:25:25.777Z] * QCH, API documentation in QCH format (for e.g. > Qt Assistant, Qt Creator & KDevelop) [2019-06-12T

D20489: [KIO] Make it compile without foreach (Step 1)

2019-07-06 Thread Laurent Montel
mlaurent added a comment. I will look at it REPOSITORY R241 KIO REVISION DETAIL https://phabricator.kde.org/D20489 To: mlaurent, dfaure Cc: kossebau, cfeck, aacid, cgiboudeaux, kde-frameworks-devel, LeGast00n, michaelh, ngraham, bruns

D22488: invoke QIcon::setFallbackThemeName a bit later

2019-07-16 Thread Laurent Montel
mlaurent added a comment. will it fix this bug "https://bugs.kde.org/show_bug.cgi?id=407600"; ? REPOSITORY R302 KIconThemes REVISION DETAIL https://phabricator.kde.org/D22488 To: mart, #frameworks, #plasma Cc: mlaurent, broulik, kde-frameworks-devel, LeGast00n, sbergeron, michaelh, ngra

D22791: Add option to build Qt Designer plugin (BUILD_DESIGNERPLUGIN, default ON

2019-07-31 Thread Laurent Montel
mlaurent accepted this revision. This revision is now accepted and ready to land. REPOSITORY R310 KTextWidgets BRANCH createdesignerplugin REVISION DETAIL https://phabricator.kde.org/D22791 To: kossebau, #frameworks, mlaurent Cc: kde-frameworks-devel, LeGast00n, sbergeron, michaelh, ngrah

D23815: [KConfig] port away from deprecated methods in Qt 5.14

2019-09-10 Thread Laurent Montel
mlaurent added a comment. Why you don't add add_definitions(-DQT_DISABLE_DEPRECATED_BEFORE=0x06) in cmake toplevel ? or 5.14 value ? so we can be sure that it builds without deprecated method until it. (I prefere QT_DISABLE_DEPRECATED_BEFORE=0x06 so we can fix it each time tha

D23815: [KConfig] port away from deprecated methods in Qt 5.14

2019-09-10 Thread Laurent Montel
mlaurent added a comment. "This kind of thing should only be added to the developer's personal build flags, or to CI." on CI without this flags in module is just not a good idea. After that I use in pim* and it's not a problem. If a people wants to use new qt (as David :) ) he will fix

D23815: [KConfig] port away from deprecated methods in Qt 5.14

2019-09-10 Thread Laurent Montel
mlaurent added a comment. "If a user of the last PIM release (as in, a person or a distro) wants to use Qt 5.14, they'll get compilation errors, unnecessarily." and he will fix it :) not a bad idea :) "if (EXISTS "${CMAKE_SOURCE_DIR}/.git")" I like this idea. REPOSITORY R237 KConfig

D23816: [KItemViews] Port to non-deprecated QWheelEvent API

2019-09-10 Thread Laurent Montel
mlaurent accepted this revision. mlaurent added a comment. This revision is now accepted and ready to land. It seems ok for me. +2 REPOSITORY R276 KItemViews BRANCH master REVISION DETAIL https://phabricator.kde.org/D23816 To: dfaure, mlaurent Cc: kde-frameworks-devel, LeGast00n, GB_2

D24468: Use ECMGenerateExportHeader to manage deprecated API better

2019-10-11 Thread Laurent Montel
mlaurent accepted this revision. This revision is now accepted and ready to land. REPOSITORY R236 KWidgetsAddons BRANCH useECMGenerateExportHeader REVISION DETAIL https://phabricator.kde.org/D24468 To: kossebau, #frameworks, cfeck, mlaurent Cc: kde-frameworks-devel, LeGast00n, GB_2, micha

D24467: Use ECMGenerateExportHeader to manage deprecated API better

2019-10-11 Thread Laurent Montel
mlaurent accepted this revision. This revision is now accepted and ready to land. REPOSITORY R306 KParts BRANCH deprecatedapi REVISION DETAIL https://phabricator.kde.org/D24467 To: kossebau, #frameworks, dfaure, mlaurent Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns

D24466: Use ECMGenerateExportHeader to manage deprecated API better

2019-10-12 Thread Laurent Montel
mlaurent accepted this revision. This revision is now accepted and ready to land. REPOSITORY R263 KXmlGui BRANCH deprecatedapi REVISION DETAIL https://phabricator.kde.org/D24466 To: kossebau, #frameworks, dfaure, mlaurent Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns

D23802: Use ECMGenerateExportHeader to manage deprecated API better

2019-10-12 Thread Laurent Montel
mlaurent accepted this revision. This revision is now accepted and ready to land. REPOSITORY R309 KService BRANCH useECMGenerateExportHeader REVISION DETAIL https://phabricator.kde.org/D23802 To: kossebau, #frameworks, dfaure, mlaurent Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh,

D24595: Use ECMGenerateExportHeader to manage deprecated API better

2019-10-13 Thread Laurent Montel
mlaurent accepted this revision. This revision is now accepted and ready to land. REPOSITORY R310 KTextWidgets BRANCH useECMGenerateExportHeader REVISION DETAIL https://phabricator.kde.org/D24595 To: kossebau, #frameworks, mlaurent Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngra

D24603: Port away from deprecated KDELibs4Support

2019-10-13 Thread Laurent Montel
mlaurent accepted this revision. This revision is now accepted and ready to land. REPOSITORY R103 KMenu Editor BRANCH master REVISION DETAIL https://phabricator.kde.org/D24603 To: ahmadsamir, mlaurent Cc: plasma-devel, kde-frameworks-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, fbampalo

D24597: [Konversation] Port from deprecated KTcpSocket to QSslSocket

2019-10-13 Thread Laurent Montel
mlaurent added reviewers: dvratil, vkrause. REPOSITORY R7 Konversation REVISION DETAIL https://phabricator.kde.org/D24597 To: ahmadsamir, #konversation, dvratil, vkrause Cc: kde-frameworks-devel, #konversation

D24608: Use ECMGenerateExportHeader to manage deprecated API better

2019-10-13 Thread Laurent Montel
mlaurent accepted this revision. mlaurent added a comment. This revision is now accepted and ready to land. Seems ok for me REPOSITORY R302 KIconThemes BRANCH useECMGenerateExportHeader REVISION DETAIL https://phabricator.kde.org/D24608 To: kossebau, #frameworks, cfeck, mlaurent Cc: m

D24651: Use ECMGenerateExportHeader to manage deprecated API better

2019-10-14 Thread Laurent Montel
mlaurent accepted this revision. This revision is now accepted and ready to land. REPOSITORY R279 ThreadWeaver BRANCH useECMGenerateExportHeader REVISION DETAIL https://phabricator.kde.org/D24651 To: kossebau, #frameworks, mirko, mlaurent Cc: kde-frameworks-devel, LeGast00n, GB_2, michael

D24643: Use ECMGenerateExportHeader to manage deprecated API better

2019-10-14 Thread Laurent Montel
mlaurent accepted this revision. This revision is now accepted and ready to land. REPOSITORY R268 KGlobalAccel BRANCH useECMGenerateExportHeader REVISION DETAIL https://phabricator.kde.org/D24643 To: kossebau, #frameworks, mlaurent Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngra

D24681: Use ECMGenerateExportHeader to manage deprecated API better

2019-10-15 Thread Laurent Montel
mlaurent accepted this revision. mlaurent added a comment. This revision is now accepted and ready to land. Seems ok for me REPOSITORY R304 KNewStuff BRANCH useECMGenerateExportHeader REVISION DETAIL https://phabricator.kde.org/D24681 To: kossebau, #frameworks, leinir, mlaurent Cc: ml

D24497: Use ECMGenerateExportHeader to manage deprecated API better

2019-10-15 Thread Laurent Montel
mlaurent accepted this revision. mlaurent added a comment. This revision is now accepted and ready to land. Seems ok for me REPOSITORY R265 KConfigWidgets BRANCH useECMGenerateExportHeader REVISION DETAIL https://phabricator.kde.org/D24497 To: kossebau, #frameworks, mlaurent Cc: mlaur

D24707: Use ECMGenerateExportHeader to manage deprecated API better

2019-10-16 Thread Laurent Montel
mlaurent accepted this revision. This revision is now accepted and ready to land. REPOSITORY R269 BluezQt BRANCH useECMGenerateExportHeader REVISION DETAIL https://phabricator.kde.org/D24707 To: kossebau, #frameworks, drosca, mlaurent Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh, n

D24710: Use ECMGenerateExportHeader to manage deprecated API better

2019-10-16 Thread Laurent Montel
mlaurent accepted this revision. This revision is now accepted and ready to land. REPOSITORY R301 KEmoticons BRANCH useECMGenerateExportHeader REVISION DETAIL https://phabricator.kde.org/D24710 To: kossebau, #frameworks, mlaurent Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraha

D24495: Use ECMGenerateExportHeader to manage deprecated API better

2019-10-16 Thread Laurent Montel
mlaurent accepted this revision. This revision is now accepted and ready to land. REPOSITORY R284 KCompletion BRANCH useECMGenerateExportHeader REVISION DETAIL https://phabricator.kde.org/D24495 To: kossebau, #frameworks, mlaurent Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngrah

D24611: Use ECMGenerateExportHeader to manage deprecated API better

2019-10-16 Thread Laurent Montel
mlaurent accepted this revision. This revision is now accepted and ready to land. REPOSITORY R286 KFileMetaData BRANCH useECMGenerateExportHeader REVISION DETAIL https://phabricator.kde.org/D24611 To: kossebau, #frameworks, bruns, mlaurent Cc: kde-frameworks-devel, #baloo, hurikhan77, lot

D24600: Use ECMGenerateExportHeader to manage deprecated API better

2019-10-16 Thread Laurent Montel
mlaurent accepted this revision. This revision is now accepted and ready to land. REPOSITORY R275 KItemModels BRANCH useECMGenerateExportHeader REVISION DETAIL https://phabricator.kde.org/D24600 To: kossebau, #frameworks, dfaure, mlaurent Cc: kde-frameworks-devel, LeGast00n, GB_2, michael

D24650: Use ECMGenerateExportHeader to manage deprecated API better

2019-10-16 Thread Laurent Montel
mlaurent accepted this revision. mlaurent added a comment. This revision is now accepted and ready to land. Seems ok for me REPOSITORY R235 Attica BRANCH useECMGenerateExportHeader REVISION DETAIL https://phabricator.kde.org/D24650 To: kossebau, #frameworks, gladhorn, leinir, mlaurent

D24496: Use ECMGenerateExportHeader to manage deprecated API better

2019-10-17 Thread Laurent Montel
mlaurent accepted this revision. mlaurent added a comment. This revision is now accepted and ready to land. seems ok for me REPOSITORY R237 KConfig BRANCH useECMGenerateExportHeader REVISION DETAIL https://phabricator.kde.org/D24496 To: kossebau, #frameworks, mlaurent Cc: mlaurent, kd

<    1   2   3   4   5   6   7   8   9   >