KDE CI: Frameworks » purpose » kf5-qt5 SUSEQt5.10 - Build # 52 - Unstable!

2019-06-04 Thread CI System
BUILD UNSTABLE Build URL https://build.kde.org/job/Frameworks/job/purpose/job/kf5-qt5%20SUSEQt5.10/52/ Project: kf5-qt5 SUSEQt5.10 Date of build: Tue, 04 Jun 2019 11:30:23 + Build duration: 12 min and counting BUILD ARTIFACTS

D21574: Update KBruch icon

2019-06-04 Thread Carl Schwan
ognarb created this revision. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. ognarb requested review of this revision. REVISION SUMMARY Inspired from application-edu and cantor icon TEST PLAN Screenshot comming REPOSITORY R266 Breeze Icons BRANCH

D21574: Update KBruch icon

2019-06-04 Thread Carl Schwan
ognarb edited the test plan for this revision. ognarb added a reviewer: VDG. REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D21574 To: ognarb, #vdg Cc: kde-frameworks-devel, LeGast00n, michaelh, ngraham, bruns

D19771: Use placeholder instead of label

2019-06-04 Thread Carl Schwan
ognarb abandoned this revision. ognarb added a comment. I don't have time and the motivation for this revision anymore. REPOSITORY R39 KTextEditor REVISION DETAIL https://phabricator.kde.org/D19771 To: ognarb, #ktexteditor, #vdg Cc: dhaumann, cullmann, cfeck, loh.tar, ngraham,

D21204: Ensure no trailing slash in mountpoint read from fstab file.

2019-06-04 Thread Méven Car
meven added a comment. I have tested the new patch. It is a bit a workaround but does not edit any udi. What do you think ? REPOSITORY R245 Solid REVISION DETAIL https://phabricator.kde.org/D21204 To: meven, bruns, #frameworks Cc: ngraham, bruns, apol, kde-frameworks-devel,

D21574: Update KBruch icon

2019-06-04 Thread Carl Schwan
ognarb edited the test plan for this revision. REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D21574 To: ognarb, #vdg Cc: kde-frameworks-devel, LeGast00n, michaelh, ngraham, bruns

D21565: phabricator: Fix fallthrough in switch

2019-06-04 Thread Alexander Potashev
This revision was automatically updated to reflect the committed changes. Closed by commit R495:a520d9ea1800: phabricator: Fix fallthrough in switch (authored by aspotashev). REPOSITORY R495 Purpose Library CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D21565?vs=59080=59122

D21574: Update KBruch icon

2019-06-04 Thread Nathaniel Graham
ngraham added a comment. Nice! A line of dark pixels on the bottom would help it fit the style even better too. See https://hig.kde.org/style/icon.html#shadows REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D21574 To: ognarb, #vdg Cc: ngraham,

D17595: Upstream Dolphin's file rename dialog

2019-06-04 Thread Nathaniel Graham
ngraham added a comment. Thanks so much! \o/ REPOSITORY R241 KIO REVISION DETAIL https://phabricator.kde.org/D17595 To: ngraham, #frameworks, #dolphin, broulik Cc: ltoscano, bruns, meven, dhaumann, pino, kde-frameworks-devel, LeGast00n, michaelh, ngraham

D21577: [UnindexedFileIndexer] Skip filetime checks for new files

2019-06-04 Thread Stefan Brüns
bruns created this revision. bruns added reviewers: Baloo, ngraham, astippich, poboiko. Herald added projects: Frameworks, Baloo. Herald added a subscriber: kde-frameworks-devel. bruns requested review of this revision. REVISION SUMMARY In case a file is not in the index, either because it is

D21574: Update KBruch icon

2019-06-04 Thread Noah Davis
ndavis added a comment. In D21574#474173 , @ognarb wrote: > Sorry it arc who also uploaded the png that I created for the review, I will fix it. Ah, fair enough. Easy mistake to make. REPOSITORY R266 Breeze Icons REVISION DETAIL

D21574: Update KBruch icon

2019-06-04 Thread Carl Schwan
ognarb updated this revision to Diff 59131. ognarb added a comment. Add shadow REPOSITORY R266 Breeze Icons CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D21574?vs=59123=59131 BRANCH kbruch REVISION DETAIL https://phabricator.kde.org/D21574 AFFECTED FILES

D21579: [FilteredDirIterator] Avoid RegExp overhead for exact matches

2019-06-04 Thread Stefan Brüns
bruns created this revision. bruns added reviewers: Baloo, ngraham, astippich, poboiko, broulik. Herald added projects: Frameworks, Baloo. Herald added a subscriber: kde-frameworks-devel. bruns requested review of this revision. REVISION SUMMARY Quite a few of the filter strings are exact

D21581: Detect valgrind, avoid database removal when using valgrind

2019-06-04 Thread Nathaniel Graham
ngraham accepted this revision. ngraham added a comment. Hah, wow REPOSITORY R293 Baloo BRANCH master REVISION DETAIL https://phabricator.kde.org/D21581 To: bruns, #baloo, ngraham, astippich, poboiko, broulik, mgallien Cc: mgallien, kde-frameworks-devel, LeGast00n, domson,

D21581: Detect valgrind, avoid database removal when using valgrind

2019-06-04 Thread Matthieu Gallien
mgallien accepted this revision. mgallien added a comment. This revision is now accepted and ready to land. Very nice. Thank you REPOSITORY R293 Baloo BRANCH master REVISION DETAIL https://phabricator.kde.org/D21581 To: bruns, #baloo, ngraham, astippich, poboiko, broulik, mgallien

D21581: Detect valgrind, avoid database removal when using valgrind

2019-06-04 Thread Stefan Brüns
This revision was automatically updated to reflect the committed changes. Closed by commit R293:446aeb07d922: Detect valgrind, avoid database removal when using valgrind (authored by bruns). REPOSITORY R293 Baloo CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D21581?vs=59137=59139

D21576: [UnindexedFileIndexer] Do not try to add nonexistant file to index

2019-06-04 Thread Stefan Brüns
bruns created this revision. bruns added reviewers: Baloo, ngraham, astippich, poboiko. Herald added projects: Frameworks, Baloo. Herald added a subscriber: kde-frameworks-devel. bruns requested review of this revision. REVISION SUMMARY A file may be deleted after it has been picked up by the

D21574: Update KBruch icon

2019-06-04 Thread Carl Schwan
ognarb added a comment. In D21574#474171 , @ndavis wrote: > Wait, is this a PNG? This needs to be just an SVG. We don't ship PNGs in breeze-icons. Sorry it arc who also uploaded the png that I created for the review, I will fix it.

D21581: Detect valgrind, avoid database removal when using valgrind

2019-06-04 Thread Stefan Brüns
bruns created this revision. bruns added reviewers: Baloo, ngraham, astippich, poboiko, broulik. Herald added projects: Frameworks, Baloo. Herald added a subscriber: kde-frameworks-devel. bruns requested review of this revision. REVISION SUMMARY The mmap size supported by valgrind is set at

D21581: Detect valgrind, avoid database removal when using valgrind

2019-06-04 Thread Stefan Brüns
bruns edited the summary of this revision. REPOSITORY R293 Baloo REVISION DETAIL https://phabricator.kde.org/D21581 To: bruns, #baloo, ngraham, astippich, poboiko, broulik Cc: kde-frameworks-devel, LeGast00n, domson, ashaposhnikov, michaelh, astippich, spoorun, ngraham, bruns, abrahams

D21576: [UnindexedFileIndexer] Do not try to add nonexistant file to index

2019-06-04 Thread Igor Poboiko
poboiko accepted this revision. This revision is now accepted and ready to land. REPOSITORY R293 Baloo BRANCH submit_unindexed REVISION DETAIL https://phabricator.kde.org/D21576 To: bruns, #baloo, ngraham, astippich, poboiko Cc: kde-frameworks-devel, LeGast00n, domson, ashaposhnikov,

D12647: Move the inline preview button into the menu

2019-06-04 Thread Nathaniel Graham
ngraham added a comment. Oops, sorry I missed your comments! Yes, removing it from the task description makes sense. REPOSITORY R241 KIO REVISION DETAIL https://phabricator.kde.org/D12647 To: ngraham, #frameworks Cc: meven, kde-frameworks-devel, anemeth, LeGast00n, michaelh, ngraham,

D21574: Update KBruch icon

2019-06-04 Thread Noah Davis
ndavis added a comment. Wait, is this a PNG? This needs to be an SVG. REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D21574 To: ognarb, #vdg, ndavis Cc: ndavis, ngraham, kde-frameworks-devel, LeGast00n, michaelh, bruns

D21574: Update KBruch icon

2019-06-04 Thread Noah Davis
ndavis requested changes to this revision. ndavis added a comment. This revision now requires changes to proceed. There needs to be a copy in breeze dark as well. Nice job though! REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D21574 To: ognarb, #vdg, ndavis

D21578: [UnindexedFileIterator] Delay mimetype determination until it is needed

2019-06-04 Thread Stefan Brüns
bruns created this revision. bruns added reviewers: Baloo, ngraham, astippich, poboiko. Herald added projects: Frameworks, Baloo. Herald added a subscriber: kde-frameworks-devel. bruns requested review of this revision. REVISION SUMMARY The mimetype is only required if all the other checks

D21574: Update KBruch icon

2019-06-04 Thread Noah Davis
ndavis added a comment. Once you're done with the SVG, make sure you clean it: https://community.kde.org/Guidelines_and_HOWTOs/Icon_Workflow_Tips#SVG_optimization REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D21574 To: ognarb, #vdg, ndavis Cc: ndavis,

KDE CI: Frameworks » baloo » kf5-qt5 WindowsMSVCQt5.11 - Build # 174 - Failure!

2019-06-04 Thread CI System
BUILD FAILURE Build URL https://build.kde.org/job/Frameworks/job/baloo/job/kf5-qt5%20WindowsMSVCQt5.11/174/ Project: kf5-qt5 WindowsMSVCQt5.11 Date of build: Tue, 04 Jun 2019 16:32:48 + Build duration: 4 min 15 sec and counting CONSOLE OUTPUT

D21581: Detect valgrind, avoid database removal when using valgrind

2019-06-04 Thread Stefan Brüns
bruns updated this revision to Diff 59137. bruns added a comment. rebase REPOSITORY R293 Baloo CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D21581?vs=59136=59137 BRANCH master REVISION DETAIL https://phabricator.kde.org/D21581 AFFECTED FILES src/engine/database.cpp

D21576: [UnindexedFileIndexer] Do not try to add nonexistant file to index

2019-06-04 Thread Nathaniel Graham
ngraham accepted this revision. ngraham added a comment. Buh! REPOSITORY R293 Baloo BRANCH submit_unindexed REVISION DETAIL https://phabricator.kde.org/D21576 To: bruns, #baloo, ngraham, astippich, poboiko Cc: kde-frameworks-devel, LeGast00n, domson, ashaposhnikov, michaelh,

D21577: [UnindexedFileIndexer] Skip filetime checks for new files

2019-06-04 Thread Stefan Brüns
bruns marked an inline comment as done. bruns added inline comments. INLINE COMMENTS > poboiko wrote in unindexedfileiterator.cpp:109 > Why do we need it here? leftover REPOSITORY R293 Baloo REVISION DETAIL https://phabricator.kde.org/D21577 To: bruns, #baloo, ngraham, astippich, poboiko

D21415: Fix the alignment of the label of the toolbutton

2019-06-04 Thread Dimitris Kardarakos
This revision was automatically updated to reflect the committed changes. Closed by commit R242:cd8b6b71cebc: Fix the alignment of the label of the toolbutton (authored by dkardarakos). REPOSITORY R242 Plasma Framework (Library) CHANGES SINCE LAST UPDATE

D21576: [UnindexedFileIndexer] Do not try to add nonexistant file to index

2019-06-04 Thread Stefan Brüns
This revision was automatically updated to reflect the committed changes. Closed by commit R293:7ab4cc5611a3: [UnindexedFileIndexer] Do not try to add nonexistant file to index (authored by bruns). REPOSITORY R293 Baloo CHANGES SINCE LAST UPDATE

D21583: [FilteredDirIterator] Combine all suffixes into one large RegExp

2019-06-04 Thread Stefan Brüns
bruns updated this revision to Diff 59148. bruns added a comment. replace one QLatin1String with QStringLiteral REPOSITORY R293 Baloo CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D21583?vs=59146=59148 BRANCH submit_unindexed REVISION DETAIL

D15739: [Places panel] Don't show Root by default

2019-06-04 Thread Méven Car
meven added a comment. gentle ping @ngraham How ironic ;) Unless you would rather let someone else take care of this. REPOSITORY R241 KIO REVISION DETAIL https://phabricator.kde.org/D15739 To: ngraham, #dolphin, #vdg, tcanabrava Cc: meven, elvisangelaccio, Codezela, davidc,

D21587: Lua: update for Lua54 and end of function as Keyword rather than Control

2019-06-04 Thread jonathan poelen
jpoelen edited the summary of this revision. REPOSITORY R216 Syntax Highlighting REVISION DETAIL https://phabricator.kde.org/D21587 To: jpoelen, #framework_syntax_highlighting, dhaumann, cullmann Cc: kwrite-devel, kde-frameworks-devel, LeGast00n, gennad, bmortimer, domson, michaelh,

D21578: [UnindexedFileIterator] Delay mimetype determination until it is needed

2019-06-04 Thread Nathaniel Graham
ngraham accepted this revision. ngraham added a comment. Makes perfect sense. REPOSITORY R293 Baloo BRANCH submit_unindexed REVISION DETAIL https://phabricator.kde.org/D21578 To: bruns, #baloo, ngraham, astippich, poboiko Cc: kde-frameworks-devel, LeGast00n, domson, ashaposhnikov,

KDE CI: Frameworks » bluez-qt » kf5-qt5 FreeBSDQt5.12 - Build # 20 - Unstable!

2019-06-04 Thread CI System
BUILD UNSTABLE Build URL https://build.kde.org/job/Frameworks/job/bluez-qt/job/kf5-qt5%20FreeBSDQt5.12/20/ Project: kf5-qt5 FreeBSDQt5.12 Date of build: Tue, 04 Jun 2019 17:18:44 + Build duration: 17 min and counting JUnit Tests Name:

D21579: [FilteredDirIterator] Avoid RegExp overhead for exact matches

2019-06-04 Thread Nathaniel Graham
ngraham accepted this revision. ngraham added a comment. This revision is now accepted and ready to land. Brilliant! REPOSITORY R293 Baloo BRANCH submit_unindexed REVISION DETAIL https://phabricator.kde.org/D21579 To: bruns, #baloo, ngraham, astippich, poboiko, broulik Cc:

KDE CI: Frameworks » plasma-framework » kf5-qt5 SUSEQt5.10 - Build # 120 - Still Unstable!

2019-06-04 Thread CI System
BUILD UNSTABLE Build URL https://build.kde.org/job/Frameworks/job/plasma-framework/job/kf5-qt5%20SUSEQt5.10/120/ Project: kf5-qt5 SUSEQt5.10 Date of build: Tue, 04 Jun 2019 17:47:09 + Build duration: 5 min 43 sec and counting BUILD ARTIFACTS

D21578: [UnindexedFileIterator] Delay mimetype determination until it is needed

2019-06-04 Thread Stefan Brüns
This revision was automatically updated to reflect the committed changes. Closed by commit R293:cdcad139a529: [UnindexedFileIterator] Delay mimetype determination until it is needed (authored by bruns). CHANGED PRIOR TO COMMIT https://phabricator.kde.org/D21578?vs=59129=59144#toc REPOSITORY

D21579: [FilteredDirIterator] Avoid RegExp overhead for exact matches

2019-06-04 Thread Stefan Brüns
This revision was automatically updated to reflect the committed changes. Closed by commit R293:be59ea539ba8: [FilteredDirIterator] Avoid RegExp overhead for exact matches (authored by bruns). REPOSITORY R293 Baloo CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D21579?vs=59132=59145

D21584: Add LE Advertising and GATT APIs

2019-06-04 Thread Manuel Weichselbaumer
mweichselbaumer added a reviewer: drosca. REPOSITORY R269 BluezQt REVISION DETAIL https://phabricator.kde.org/D21584 To: mweichselbaumer, drosca Cc: kde-frameworks-devel, LeGast00n, michaelh, ngraham, bruns

D21586: Add syntax definition for ninja build

2019-06-04 Thread jonathan poelen
jpoelen created this revision. jpoelen added reviewers: Framework: Syntax Highlighting, dhaumann, cullmann. jpoelen added projects: Framework: Syntax Highlighting, Kate, Frameworks. jpoelen requested review of this revision. REPOSITORY R216 Syntax Highlighting BRANCH ninja (branched from

D21579: [FilteredDirIterator] Avoid RegExp overhead for exact matches

2019-06-04 Thread Igor Poboiko
poboiko added inline comments. INLINE COMMENTS > regexpcache.cpp:60 > +} > f.replace(QLatin1Char('.'), QStringLiteral("\\.")); > f.replace(QLatin1Char('?'), QLatin1Char('.')); A quick note for future. There is `QRegularExpression::wildcardToRegularExpression()`, which

D21581: Detect valgrind, avoid database removal when using valgrind

2019-06-04 Thread Ben Cooksley
bcooksley added a comment. This changes breaks the build on Windows/MSVC because 'valgrind.h' doesn't support MSVC. Can you please correct this / disable this check on Windows? REPOSITORY R293 Baloo REVISION DETAIL https://phabricator.kde.org/D21581 To: bruns, #baloo, ngraham,

KDE CI: Frameworks » plasma-framework » kf5-qt5 FreeBSDQt5.12 - Build # 123 - Still Unstable!

2019-06-04 Thread CI System
BUILD UNSTABLE Build URL https://build.kde.org/job/Frameworks/job/plasma-framework/job/kf5-qt5%20FreeBSDQt5.12/123/ Project: kf5-qt5 FreeBSDQt5.12 Date of build: Tue, 04 Jun 2019 17:47:09 + Build duration: 1 hr 12 min and counting JUnit Tests

KDE CI: Frameworks » kio » kf5-qt5 FreeBSDQt5.12 - Build # 127 - Still Unstable!

2019-06-04 Thread CI System
BUILD UNSTABLE Build URL https://build.kde.org/job/Frameworks/job/kio/job/kf5-qt5%20FreeBSDQt5.12/127/ Project: kf5-qt5 FreeBSDQt5.12 Date of build: Tue, 04 Jun 2019 17:52:45 + Build duration: 1 hr 15 min and counting JUnit Tests Name:

D21585: C++: update for C++20

2019-06-04 Thread jonathan poelen
jpoelen updated this revision to Diff 59156. jpoelen added a comment. oops, missing version increase REPOSITORY R216 Syntax Highlighting CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D21585?vs=59152=59156 BRANCH cpp2a (branched from master) REVISION DETAIL

D21577: [UnindexedFileIndexer] Skip filetime checks for new files

2019-06-04 Thread Stefan Brüns
bruns updated this revision to Diff 59147. bruns added a comment. remove leftover double timeinfo fetch rebase REPOSITORY R293 Baloo CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D21577?vs=59128=59147 BRANCH master REVISION DETAIL https://phabricator.kde.org/D21577

KDE CI: Frameworks » baloo » kf5-qt5 WindowsMSVCQt5.11 - Build # 175 - Still Failing!

2019-06-04 Thread CI System
BUILD FAILURE Build URL https://build.kde.org/job/Frameworks/job/baloo/job/kf5-qt5%20WindowsMSVCQt5.11/175/ Project: kf5-qt5 WindowsMSVCQt5.11 Date of build: Tue, 04 Jun 2019 17:58:39 + Build duration: 10 min and counting CONSOLE OUTPUT

KDE CI: Frameworks » bluez-qt » kf5-qt5 SUSEQt5.10 - Build # 20 - Unstable!

2019-06-04 Thread CI System
BUILD UNSTABLE Build URL https://build.kde.org/job/Frameworks/job/bluez-qt/job/kf5-qt5%20SUSEQt5.10/20/ Project: kf5-qt5 SUSEQt5.10 Date of build: Tue, 04 Jun 2019 17:18:44 + Build duration: 3 min 35 sec and counting BUILD ARTIFACTS

D21577: [UnindexedFileIndexer] Skip filetime checks for new files

2019-06-04 Thread Igor Poboiko
poboiko added a comment. I didn't know `QFileInfo` fetches information on demand (and caches it). That is the reason for this change, right? I think it would be nice to elaborate on that in summary, or maybe as a brief comment in the code. INLINE COMMENTS > unindexedfileiterator.cpp:109

D21577: [UnindexedFileIndexer] Skip filetime checks for new files

2019-06-04 Thread Stefan Brüns
bruns added a comment. In D21577#474212 , @poboiko wrote: > I didn't know `QFileInfo` fetches information on demand (and caches it). That is the reason for this change, right? > I think it would be nice to elaborate on that in summary, or

D21583: [FilteredDirIterator] Combine all suffixes into one large RegExp

2019-06-04 Thread Nathaniel Graham
ngraham added a comment. Would it be make more sense to just have a single file type regex in the config file rather than parsing the existing data and re-processing it in the code? REPOSITORY R293 Baloo REVISION DETAIL https://phabricator.kde.org/D21583 To: bruns, #baloo, ngraham,

D21508: Jam: various improvements and fixes

2019-06-04 Thread jonathan poelen
jpoelen updated this revision to Diff 59150. jpoelen added a comment. - fix module - remove hard coded color REPOSITORY R216 Syntax Highlighting CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D21508?vs=58930=59150 BRANCH fix_jam (branched from master) REVISION DETAIL

D21584: Add LE Advertising and GATT APIs

2019-06-04 Thread Manuel Weichselbaumer
mweichselbaumer created this revision. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. mweichselbaumer requested review of this revision. REPOSITORY R269 BluezQt BRANCH ble_gatt REVISION DETAIL https://phabricator.kde.org/D21584 AFFECTED FILES

D21585: C++: update for C++20

2019-06-04 Thread jonathan poelen
jpoelen created this revision. jpoelen added reviewers: Framework: Syntax Highlighting, dhaumann, cullmann. jpoelen added projects: Framework: Syntax Highlighting, Kate, Frameworks. jpoelen requested review of this revision. REVISION SUMMARY - new attributes: assert, ensures and expects - new

D21578: [UnindexedFileIterator] Delay mimetype determination until it is needed

2019-06-04 Thread Igor Poboiko
poboiko accepted this revision. This revision is now accepted and ready to land. REPOSITORY R293 Baloo BRANCH submit_unindexed REVISION DETAIL https://phabricator.kde.org/D21578 To: bruns, #baloo, ngraham, astippich, poboiko Cc: kde-frameworks-devel, LeGast00n, domson, ashaposhnikov,

KDE CI: Frameworks » bluez-qt » kf5-qt5 SUSEQt5.12 - Build # 20 - Unstable!

2019-06-04 Thread CI System
BUILD UNSTABLE Build URL https://build.kde.org/job/Frameworks/job/bluez-qt/job/kf5-qt5%20SUSEQt5.12/20/ Project: kf5-qt5 SUSEQt5.12 Date of build: Tue, 04 Jun 2019 17:18:44 + Build duration: 19 min and counting BUILD ARTIFACTS

KDE CI: Frameworks » plasma-framework » kf5-qt5 SUSEQt5.12 - Build # 113 - Still Unstable!

2019-06-04 Thread CI System
BUILD UNSTABLE Build URL https://build.kde.org/job/Frameworks/job/plasma-framework/job/kf5-qt5%20SUSEQt5.12/113/ Project: kf5-qt5 SUSEQt5.12 Date of build: Tue, 04 Jun 2019 17:47:09 + Build duration: 13 min and counting BUILD ARTIFACTS

KDE CI: Frameworks » kio » kf5-qt5 SUSEQt5.12 - Build # 117 - Fixed!

2019-06-04 Thread CI System
BUILD SUCCESS Build URL https://build.kde.org/job/Frameworks/job/kio/job/kf5-qt5%20SUSEQt5.12/117/ Project: kf5-qt5 SUSEQt5.12 Date of build: Tue, 04 Jun 2019 17:52:44 + Build duration: 8 min 46 sec and counting BUILD ARTIFACTS

D21583: [FilteredDirIterator] Combine all suffixes into one large RegExp

2019-06-04 Thread Stefan Brüns
bruns created this revision. bruns added reviewers: Baloo, ngraham, poboiko. Herald added projects: Frameworks, Baloo. Herald added a subscriber: kde-frameworks-devel. bruns requested review of this revision. REVISION SUMMARY Most of the about 50 default filters are suffix matches, which can be

D21587: Lua: update for Lua54 and end of function as Keyword rather than Control

2019-06-04 Thread jonathan poelen
jpoelen created this revision. jpoelen added reviewers: Framework: Syntax Highlighting, dhaumann, cullmann. jpoelen added projects: Framework: Syntax Highlighting, Kate, Frameworks. jpoelen requested review of this revision. REVISION SUMMARY function foo() if x then end <--- Control

D21577: [UnindexedFileIndexer] Skip filetime checks for new files

2019-06-04 Thread Stefan Brüns
bruns marked an inline comment as done. REPOSITORY R293 Baloo REVISION DETAIL https://phabricator.kde.org/D21577 To: bruns, #baloo, ngraham, astippich, poboiko Cc: kde-frameworks-devel, LeGast00n, domson, ashaposhnikov, michaelh, astippich, spoorun, ngraham, bruns, abrahams

KDE CI: Frameworks » baloo » kf5-qt5 WindowsMSVCQt5.11 - Build # 176 - Still Failing!

2019-06-04 Thread CI System
BUILD FAILURE Build URL https://build.kde.org/job/Frameworks/job/baloo/job/kf5-qt5%20WindowsMSVCQt5.11/176/ Project: kf5-qt5 WindowsMSVCQt5.11 Date of build: Tue, 04 Jun 2019 18:10:03 + Build duration: 21 min and counting CONSOLE OUTPUT

D21587: Lua: update for Lua54 and end of function as Keyword rather than Control

2019-06-04 Thread jonathan poelen
This revision was automatically updated to reflect the committed changes. Closed by commit R216:602bbc7b7397: Lua: update for Lua54 and end of function as Keyword rather than Control (authored by jpoelen). REPOSITORY R216 Syntax Highlighting CHANGES SINCE LAST UPDATE

D21508: Jam: various improvements and fixes

2019-06-04 Thread jonathan poelen
This revision was automatically updated to reflect the committed changes. Closed by commit R216:6864c25f1d54: Jam: various improvements and fixes (authored by jpoelen). REPOSITORY R216 Syntax Highlighting CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D21508?vs=59150=59164 REVISION

D21585: C++: update for C++20

2019-06-04 Thread Dominik Haumann
dhaumann accepted this revision. dhaumann added a comment. This revision is now accepted and ready to land. Thanks for the update! REPOSITORY R216 Syntax Highlighting BRANCH cpp2a (branched from master) REVISION DETAIL https://phabricator.kde.org/D21585 To: jpoelen,

D21587: Lua: update for Lua54 and end of function as Keyword rather than Control

2019-06-04 Thread Dominik Haumann
dhaumann accepted this revision. dhaumann added a comment. This revision is now accepted and ready to land. Lgtm REPOSITORY R216 Syntax Highlighting BRANCH lua54 (branched from master) REVISION DETAIL https://phabricator.kde.org/D21587 To: jpoelen, #framework_syntax_highlighting,

D21508: Jam: various improvements and fixes

2019-06-04 Thread Dominik Haumann
dhaumann accepted this revision. REPOSITORY R216 Syntax Highlighting BRANCH fix_jam (branched from master) REVISION DETAIL https://phabricator.kde.org/D21508 To: jpoelen, #framework_syntax_highlighting, dhaumann, cullmann Cc: kde-frameworks-devel, kwrite-devel, LeGast00n, domson,

D21583: [FilteredDirIterator] Combine all suffixes into one large RegExp

2019-06-04 Thread Stefan Brüns
bruns added a comment. In D21583#474266 , @ngraham wrote: > Would it be make more sense to just have a single file type regex in the config file rather than parsing the existing data and re-processing it in the code? The config is quite

D21585: C++: update for C++20

2019-06-04 Thread jonathan poelen
This revision was automatically updated to reflect the committed changes. Closed by commit R216:9ba02971123d: C++: update for C++20 (authored by jpoelen). REPOSITORY R216 Syntax Highlighting CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D21585?vs=59156=59162 REVISION DETAIL

D21583: [FilteredDirIterator] Combine all suffixes into one large RegExp

2019-06-04 Thread Nathaniel Graham
ngraham accepted this revision. ngraham added a comment. This revision is now accepted and ready to land. All right, sounds good for now then. We can refactor the config file later. REPOSITORY R293 Baloo BRANCH submit_unindexed REVISION DETAIL https://phabricator.kde.org/D21583 To:

D17595: Upstream Dolphin's file rename dialog

2019-06-04 Thread Méven Car
meven added a comment. In D17595#473826 , @ngraham wrote: > Would you be interested in doing that and taking over this patch? I'm afraid I don't think I have the time right now for the kind of deep dive that it requires. If not, that's okay. :)

D21572: [UnindexedFileIndexer] Loop optimizations (avoid detach, invariants)

2019-06-04 Thread Stefan Brüns
bruns created this revision. bruns added reviewers: Baloo, ngraham, astippich, poboiko. Herald added projects: Frameworks, Baloo. Herald added a subscriber: kde-frameworks-devel. bruns requested review of this revision. REVISION SUMMARY Avoid detach of QStringList in for loop. Move invariant

D21572: [UnindexedFileIndexer] Loop optimizations (avoid detach, invariants)

2019-06-04 Thread Stefan Brüns
This revision was automatically updated to reflect the committed changes. Closed by commit R293:4fcc08d86881: [UnindexedFileIndexer] Loop optimizations (avoid detach, invariants) (authored by bruns). REPOSITORY R293 Baloo CHANGES SINCE LAST UPDATE

D21572: [UnindexedFileIndexer] Loop optimizations (avoid detach, invariants)

2019-06-04 Thread Kai Uwe Broulik
broulik accepted this revision. This revision is now accepted and ready to land. REPOSITORY R293 Baloo BRANCH master REVISION DETAIL https://phabricator.kde.org/D21572 To: bruns, #baloo, ngraham, astippich, poboiko, broulik Cc: kde-frameworks-devel, LeGast00n, domson, ashaposhnikov,

KDE CI: Frameworks » baloo » kf5-qt5 FreeBSDQt5.12 - Build # 86 - Fixed!

2019-06-04 Thread CI System
BUILD SUCCESS Build URL https://build.kde.org/job/Frameworks/job/baloo/job/kf5-qt5%20FreeBSDQt5.12/86/ Project: kf5-qt5 FreeBSDQt5.12 Date of build: Tue, 04 Jun 2019 08:58:26 + Build duration: 5 min 8 sec and counting JUnit Tests Name:

D21459: Always save view settings when switching from one view mode to another

2019-06-04 Thread Méven Car
meven edited the summary of this revision. REPOSITORY R241 KIO BRANCH arcpatch-D21459 REVISION DETAIL https://phabricator.kde.org/D21459 To: meven, #frameworks, ngraham Cc: kde-frameworks-devel, LeGast00n, michaelh, ngraham, bruns