D7929: [WIP] Add new Column View option to KDirOperator

2017-09-21 Thread Nathaniel Graham
ngraham added a comment. I'll just leave this here: ;-) https://bugs.kde.org/show_bug.cgi?id=290747 REPOSITORY R241 KIO REVISION DETAIL https://phabricator.kde.org/D7929 To: fvogt, #frameworks Cc: ngraham, elvisangelaccio, lbeltrame, #frameworks

D7929: [WIP] Add new Column View option to KDirOperator

2017-09-21 Thread Fabian Vogt
fvogt added a comment. In https://phabricator.kde.org/D7929#147742, @elvisangelaccio wrote: > I'm not sure if it's a good idea to fork QColumnView in KIO. Is it not possible to contribute the changes upstream? It's not that easy - QColumnView is practically abandoned upstream (wit

D7929: [WIP] Add new Column View option to KDirOperator

2017-09-21 Thread Elvis Angelaccio
elvisangelaccio added a comment. I'm not sure if it's a good idea to fork QColumnView in KIO. Is it not possible to contribute the changes upstream? REPOSITORY R241 KIO REVISION DETAIL https://phabricator.kde.org/D7929 To: fvogt, #frameworks Cc: elvisangelaccio, lbeltrame, #frameworks

D7929: [WIP] Add new Column View option to KDirOperator

2017-09-21 Thread Luca Beltrame
lbeltrame added a comment. A screenshot before/after (or just after, since it is an addition) would be helpful in this case, since it is a visual change. REPOSITORY R241 KIO REVISION DETAIL https://phabricator.kde.org/D7929 To: fvogt, #frameworks Cc: lbeltrame, #frameworks

D7855: Fix tags KIO-slave protocol description

2017-09-21 Thread Elvis Angelaccio
elvisangelaccio accepted this revision as: elvisangelaccio. This revision is now accepted and ready to land. REPOSITORY R293 Baloo REVISION DETAIL https://phabricator.kde.org/D7855 To: nicolasfella, #frameworks, elvisangelaccio Cc: #frameworks

D7929: [WIP] Add new Column View option to KDirOperator

2017-09-21 Thread Fabian Vogt
fvogt created this revision. fvogt added a reviewer: Frameworks. Restricted Application added a project: Frameworks. REVISION SUMMARY It uses an imported and improved version of QColumnView for display. KFileItemDelegate got amended with an arrow if used inside KColumnView, to replace the ba

D7926: Remove pf.path() from container before the reference got screwed up by it.remove()

2017-09-21 Thread Kai Uwe Broulik
broulik accepted this revision. This revision is now accepted and ready to land. REPOSITORY R293 Baloo BRANCH fix_crash_pendingfilequeue REVISION DETAIL https://phabricator.kde.org/D7926 To: chehrlic, broulik Cc: #frameworks

D7926: Remove pf.path() from container before the reference got screwed up by it.remove()

2017-09-21 Thread Christian Ehrlicher
chehrlic created this revision. Restricted Application added a project: Frameworks. Restricted Application added a subscriber: Frameworks. REVISION SUMMARY With https://phabricator.kde.org/R293:a75db7aedfcd356b027774e0dc156c334d4cb869 I introduced a crash when removing a watched directory. T

D6709: [RFC] Add support for sharing file descriptor between KIO slave and KAuth helper

2017-09-21 Thread Chinmoy Ranjan Pradhan
chinmoyr updated this revision to Diff 19758. chinmoyr added a comment. Added license header CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D6709?vs=19757&id=19758 BRANCH sharefd REVISION DETAIL https://phabricator.kde.org/D6709 AFFECTED FILES src/ioslaves/file/CMakeLists.txt

D7915: Highlighting indexer: check existence of referenced context names

2017-09-21 Thread Volker Krause
vkrause accepted this revision. vkrause added a comment. This revision is now accepted and ready to land. good idea :) REPOSITORY R216 Syntax Highlighting REVISION DETAIL https://phabricator.kde.org/D7915 To: dhaumann, #framework_syntax_highlighting, cullmann, vkrause Cc: vkrause, #frame

D6709: [RFC] Add support for sharing file descriptor between KIO slave and KAuth helper

2017-09-21 Thread Chinmoy Ranjan Pradhan
chinmoyr updated this revision to Diff 19757. chinmoyr marked an inline comment as done. chinmoyr added a comment. - separated FdReceiver and FdSender - KMSgHdr -> FDMessageHeader and KSockaddrUn -> SocketAddress and moved them to private header - removed `stopListening()` method. Qt can h

D7855: Fix tags KIO-slave protocol description

2017-09-21 Thread Nicolas Fella
nicolasfella added a comment. This is a workaround for issues which will be more visible when https://phabricator.kde.org/D7700 gets merged REPOSITORY R293 Baloo REVISION DETAIL https://phabricator.kde.org/D7855 To: nicolasfella, #frameworks Cc: #frameworks

D7915: Highlighting indexer: check existence of referenced context names

2017-09-21 Thread Dominik Haumann
dhaumann added a comment. Maybe worth to note: This patch only detects usage of non-existing contexts. However, this patch does NOT catch unused contexts (which might be interesting as well). REPOSITORY R216 Syntax Highlighting REVISION DETAIL https://phabricator.kde.org/D7915 To: dha

D7915: Highlighting indexer: check existence of referenced context names

2017-09-21 Thread Dominik Haumann
dhaumann created this revision. dhaumann added reviewers: Framework: Syntax Highlighting, cullmann. Restricted Application added a project: Frameworks. Restricted Application added a subscriber: Frameworks. REVISION SUMMARY This patch extends the highlighting indexer by collecting a list of all

KDE CI: Frameworks kwayland kf5-qt5 FreeBSDQt5.7 - Build # 46 - Still Unstable!

2017-09-21 Thread no-reply
BUILD UNSTABLE Build URL https://build.kde.org/job/Frameworks%20kwayland%20kf5-qt5%20FreeBSDQt5.7/46/ Project: Frameworks kwayland kf5-qt5 FreeBSDQt5.7 Date of build: Thu, 21 Sep 2017 13:11:57 + Build duration: 10 min and counting JUnit Tests Nam

D7898: Only send OutputConfig sendApplied / sendFailed to the right resource

2017-09-21 Thread David Edmundson
This revision was automatically updated to reflect the committed changes. Closed by commit R127:2b3f9509ac12: Only send OutputConfig sendApplied / sendFailed to the right resource (authored by davidedmundson). REPOSITORY R127 KWayland CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D78

KDE CI: Frameworks kirigami kf5-qt5 XenialQt5.7 - Build # 83 - Still Unstable!

2017-09-21 Thread no-reply
BUILD UNSTABLE Build URL https://build.kde.org/job/Frameworks%20kirigami%20kf5-qt5%20XenialQt5.7/83/ Project: Frameworks kirigami kf5-qt5 XenialQt5.7 Date of build: Thu, 21 Sep 2017 13:04:33 + Build duration: 4 min 33 sec and counting JUnit Tests

KDE CI: Frameworks kirigami kf5-qt5 FreeBSDQt5.7 - Build # 79 - Still Unstable!

2017-09-21 Thread no-reply
BUILD UNSTABLE Build URL https://build.kde.org/job/Frameworks%20kirigami%20kf5-qt5%20FreeBSDQt5.7/79/ Project: Frameworks kirigami kf5-qt5 FreeBSDQt5.7 Date of build: Thu, 21 Sep 2017 13:04:33 + Build duration: 1 min 5 sec and counting JUnit Tests

D7898: Only send OutputConfig sendApplied / sendFailed to the right resource

2017-09-21 Thread Sebastian Kügler
sebas added a comment. FWIW, I tested it, it fixes kwin crashing when adding a setApplied() call on the OutputConfiguration, and it also means the applied signal arrives. REPOSITORY R127 KWayland BRANCH master REVISION DETAIL https://phabricator.kde.org/D7898 To: davidedmundson, #pla

KDE CI: Frameworks plasma-framework kf5-qt5 FreeBSDQt5.7 - Build # 116 - Still Unstable!

2017-09-21 Thread no-reply
BUILD UNSTABLE Build URL https://build.kde.org/job/Frameworks%20plasma-framework%20kf5-qt5%20FreeBSDQt5.7/116/ Project: Frameworks plasma-framework kf5-qt5 FreeBSDQt5.7 Date of build: Thu, 21 Sep 2017 12:34:05 + Build duration: 10 min and counting JUni

KDE CI: Frameworks plasma-framework kf5-qt5 XenialQt5.7 - Build # 112 - Still Unstable!

2017-09-21 Thread no-reply
BUILD UNSTABLE Build URL https://build.kde.org/job/Frameworks%20plasma-framework%20kf5-qt5%20XenialQt5.7/112/ Project: Frameworks plasma-framework kf5-qt5 XenialQt5.7 Date of build: Thu, 21 Sep 2017 12:34:05 + Build duration: 5 min 37 sec and counting

D7898: Only send OutputConfig sendApplied / sendFailed to the right resource

2017-09-21 Thread Sebastian Kügler
sebas accepted this revision. sebas added a comment. This revision is now accepted and ready to land. Yes, the initial idea was to send applied() to all connected resources. That is not s_allResources, so your patch is correct. The resources that are actually bound are not currently tracked.

KDE CI: Frameworks kwayland kf5-qt5 FreeBSDQt5.7 - Build # 45 - Still Unstable!

2017-09-21 Thread no-reply
BUILD UNSTABLE Build URL https://build.kde.org/job/Frameworks%20kwayland%20kf5-qt5%20FreeBSDQt5.7/45/ Project: Frameworks kwayland kf5-qt5 FreeBSDQt5.7 Date of build: Thu, 21 Sep 2017 11:52:51 + Build duration: 6 min 42 sec and counting JUnit Tests

D7909: Add syntax support for Crystal Programming Language

2017-09-21 Thread Faustino Aguilar
faguilar edited the test plan for this revision. faguilar added a reviewer: Framework: Syntax Highlighting. REPOSITORY R216 Syntax Highlighting REVISION DETAIL https://phabricator.kde.org/D7909 To: faguilar, dhaumann, vkrause, #framework_syntax_highlighting Cc: #frameworks, faguilar, cullman

D7909: Add syntax support for Crystal Programming Language

2017-09-21 Thread Faustino Aguilar
faguilar updated this revision to Diff 19732. faguilar added a comment. Add Crystal input for autotests REPOSITORY R216 Syntax Highlighting CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D7909?vs=19730&id=19732 REVISION DETAIL https://phabricator.kde.org/D7909 AFFECTED FILES

D7909: Add syntax support for Crystal Programming Language

2017-09-21 Thread Faustino Aguilar
faguilar added a comment. I'm user of Manjaro KDE and I love coding using Kate. Currently I'm focused in Crystal learning using Ruby syntax, however Ruby syntax doesn't support all Crystal features. REPOSITORY R216 Syntax Highlighting REVISION DETAIL https://phabricator.kde.org/D7909 T

D7909: Add syntax support for Crystal Programming Language

2017-09-21 Thread Faustino Aguilar
faguilar created this revision. faguilar added reviewers: dhaumann, vkrause. faguilar added a project: Framework: Syntax Highlighting. Restricted Application added a project: Frameworks. Restricted Application added a subscriber: Frameworks. REVISION SUMMARY This add syntax support the Crystal P

D7879: [KConfigGroup] reserve() more and add some C++11

2017-09-21 Thread Kai Uwe Broulik
This revision was automatically updated to reflect the committed changes. Closed by commit R237:fb15e56857ca: [KConfigGroup] reserve() more and add some C++11 (authored by broulik). CHANGED PRIOR TO COMMIT https://phabricator.kde.org/D7879?vs=19671&id=19727#toc REPOSITORY R237 KConfig CHANG

D7879: [KConfigGroup] reserve() more and add some C++11

2017-09-21 Thread Milian Wolff
mwolff accepted this revision. mwolff added a comment. ah ok, and since we don't have a proper byte array view yet, there's nothing you can do (which is sad here!) the other comments from me where suggestions for further improvements, they don't need to hold up this patch REPOSITORY R