brauch (Sven Brauch)
User

Projects

Today

  • Clear sailing ahead.

Tomorrow

  • Clear sailing ahead.

Saturday

  • Clear sailing ahead.

User Details

User Since
Jan 25 2016, 10:25 AM (129 w, 3 d)
Availability
Available

Recent Activity

Sat, Jun 23

brauch committed R32:144ab70c5d40: Open konsole with the active document's project root or it's directory (authored by amhndu).
Open konsole with the active document's project root or it's directory
Sat, Jun 23, 3:01 PM
brauch closed D13654: Open konsole tool with the single project's root directory.
Sat, Jun 23, 3:01 PM · KDevelop
brauch added a comment to D13654: Open konsole tool with the single project's root directory.

Looks reasonable to me. Can you submit the patch yourself or should I do it?

Sat, Jun 23, 2:46 PM · KDevelop
brauch accepted D13654: Open konsole tool with the single project's root directory.

No, this looks good to me. Thanks!

Sat, Jun 23, 2:27 PM · KDevelop
brauch added inline comments to D13654: Open konsole tool with the single project's root directory.
Sat, Jun 23, 11:29 AM · KDevelop
brauch added a comment to D13444: Update to display the name of the session in the "Delete Session" confirmation dialogue box.

Of course, nobody is exactly sitting here thinking "eeh, now I've been waiting for that guy for *weeks* to get his patch in" ;)
If you need advice, feel free to ask.

Sat, Jun 23, 11:26 AM · KDevelop

Thu, Jun 21

brauch added a comment to D13444: Update to display the name of the session in the "Delete Session" confirmation dialogue box.

No, I think you should introduce a way to get the session name before the colon is added. You can introduce a new role for the item model for example, have a look at QStandardItem::setData.

Thu, Jun 21, 6:24 PM · KDevelop
brauch added a comment to D13444: Update to display the name of the session in the "Delete Session" confirmation dialogue box.

Try opening a few projects in the session. The names of these projects should follow the colon afterwards.

Thu, Jun 21, 11:22 AM · KDevelop
brauch added a comment to D13444: Update to display the name of the session in the "Delete Session" confirmation dialogue box.

AFAIK that is followed by the names of the projects open in that session, and that is also where your approach with chopping of the chars will probably go wrong.

Thu, Jun 21, 10:35 AM · KDevelop
brauch added a comment to D13444: Update to display the name of the session in the "Delete Session" confirmation dialogue box.

Can I get an instance when chopping off the last 3 letters won't work? Like, shouldn't the last 3 letters always be ": "
Or will it change if the name of the session's too big or something?

Thu, Jun 21, 8:29 AM · KDevelop
brauch added a comment to D13444: Update to display the name of the session in the "Delete Session" confirmation dialogue box.

KDE only uses github as a read-only mirror. You will get a notification when it is submitted.

Thu, Jun 21, 8:09 AM · KDevelop
brauch added inline comments to D13444: Update to display the name of the session in the "Delete Session" confirmation dialogue box.
Thu, Jun 21, 7:33 AM · KDevelop
brauch accepted D13444: Update to display the name of the session in the "Delete Session" confirmation dialogue box.

Otherwise looks good, I can submit this later.

Thu, Jun 21, 7:23 AM · KDevelop

Wed, Jun 20

brauch committed R32:207d850ed3b9: fix bug where activating current view won't focus it (authored by amhndu).
fix bug where activating current view won't focus it
Wed, Jun 20, 6:03 PM
brauch closed D13592: Fix bug where activating current view won't focus it.
Wed, Jun 20, 6:03 PM · KDevelop

Jun 19 2018

brauch added a comment to D13592: Fix bug where activating current view won't focus it.

Ah, cool. In that case I would need your email address to set you as the author of the change.

Jun 19 2018, 6:34 AM · KDevelop

Jun 18 2018

brauch accepted D13592: Fix bug where activating current view won't focus it.

Looks good to me, thanks! Can you submit it yourself?

Jun 18 2018, 5:50 PM · KDevelop

Jun 14 2018

brauch committed R40:da2e40b9ed9f: Merge branch 'Applications/18.04' (authored by brauch).
Merge branch 'Applications/18.04'
Jun 14 2018, 6:51 AM
brauch added a comment to D13517: Fix QuickOpen with Qt 5.11 (bug 395039).

Yes sorry, I felt too tired yesterday to correctly resolve the version number merge conflict ;) Done now.

Jun 14 2018, 6:50 AM · Kate

Jun 13 2018

brauch committed R40:291b68994d7c: Fix QuickOpen with Qt 5.11 (authored by brauch).
Fix QuickOpen with Qt 5.11
Jun 13 2018, 8:56 PM
brauch closed D13517: Fix QuickOpen with Qt 5.11 (bug 395039).
Jun 13 2018, 8:56 PM · Kate
brauch added a comment to D13517: Fix QuickOpen with Qt 5.11 (bug 395039).

Looks like it was already fixed for Qt 5.11.1, e15fc26e9fdbff141890a3e2e8dc4ef935d022a0 sounds a lot like it:

Jun 13 2018, 3:41 PM · Kate
brauch added a comment to D13517: Fix QuickOpen with Qt 5.11 (bug 395039).

I'll dig around a bit and see if I can find the change in Qt causing this. In the meantime, do you want this in applications/18.04 or master only?

Jun 13 2018, 3:27 PM · Kate
brauch updated the summary of D13517: Fix QuickOpen with Qt 5.11 (bug 395039).
Jun 13 2018, 2:38 PM · Kate
brauch requested review of D13517: Fix QuickOpen with Qt 5.11 (bug 395039).
Jun 13 2018, 2:33 PM · Kate

Jun 7 2018

brauch committed R39:e6f87dd57008: Fix caret width (authored by shubham).
Fix caret width
Jun 7 2018, 7:08 AM

May 17 2018

brauch committed R53:fac5305248c6: Set version to 5.2.3 (authored by brauch).
Set version to 5.2.3
May 17 2018, 6:39 PM
brauch committed R52:66128771f328: Set version to 5.2.3 (authored by brauch).
Set version to 5.2.3
May 17 2018, 6:37 PM
brauch committed R32:b2e5b2bdbec3: Set version to 5.2.3 (authored by brauch).
Set version to 5.2.3
May 17 2018, 5:04 PM

May 9 2018

brauch added a comment to D12768: Allow wrapping selection off top/bottom of autocomplete results.

What happens if the overload selection window is open in addition (like in KDevelop)?

May 9 2018, 7:14 AM · KTextEditor, Frameworks, Kate

May 7 2018

brauch added inline comments to D12662: Add InlineNoteInterface.
May 7 2018, 7:12 AM · Frameworks, Kate

May 3 2018

brauch added a comment to D12662: Add InlineNoteInterface.

Oh, heh, yes it does. It's just the docstring which says otherwise ("is an interface for the View"), and that's what I looked at at that time. That should be changed. ;)

May 3 2018, 10:28 PM · Frameworks, Kate
brauch added a comment to D12662: Add InlineNoteInterface.

I think fixing the selection rendering issue would be nice.

May 3 2018, 6:16 PM · Frameworks, Kate

May 2 2018

brauch added a comment to D12662: Add InlineNoteInterface.

Looks good from the implementation too so far. One thing I do not see is any changes to the cursorToX / xToCursor functions, is there really no change required there?

May 2 2018, 6:36 PM · Frameworks, Kate
brauch added a comment to D12662: Add InlineNoteInterface.

Awesome idea! Do you have a screenshot of how it looks?

May 2 2018, 4:10 PM · Frameworks, Kate

Apr 23 2018

brauch accepted D12298: Group completion requests and only handle the last one.

Didn't do super much C++ since but didn't notice anything. Thank you!

Apr 23 2018, 4:40 PM

Apr 18 2018

brauch added a comment to D12331: Always use the clang builtin headers for the libclang version we use.

Given the amount of problems we had with this in the past, I think this change makes perfect sense. I applied the patch here, I'll see if anything breaks in the next few days ...

Apr 18 2018, 10:04 PM
brauch added inline comments to D12298: Group completion requests and only handle the last one.
Apr 18 2018, 9:31 PM
brauch added a comment to D12298: Group completion requests and only handle the last one.

I think the change makes sense, I applied the patch here, let's see how it goes.

Apr 18 2018, 7:16 AM

Apr 9 2018

brauch added a comment to D12016: [ktexteditor] much faster positionFromCursor.

Oh and, you do not need to inherit QObject to use connect; you can connect to a lambda calling the member function AFAIK or so. Just omit the third argument in connect(). What you lose by doing this is the automatic disconnect of the connection when the receiver object is deleted, so make sure that doesn't happen.

Apr 9 2018, 4:49 PM · Frameworks, Kate
brauch added a comment to D12016: [ktexteditor] much faster positionFromCursor.

Re. binary compatibility: should be fine because this class is not exported (no KTEXTEDITOR_EXPORT macro).

Apr 9 2018, 4:47 PM · Frameworks, Kate

Apr 7 2018

brauch committed R52:b94cfb51e7b0: Set version to 5.2.2 (authored by brauch).
Set version to 5.2.2
Apr 7 2018, 9:57 PM
brauch committed R53:481d9971dcfe: Set version to 5.2.2 (authored by brauch).
Set version to 5.2.2
Apr 7 2018, 9:56 PM
brauch committed R32:7e1bf52dfda3: Set version to 5.2.2 (authored by brauch).
Set version to 5.2.2
Apr 7 2018, 9:56 PM

Apr 4 2018

brauch accepted D11933: Only do fuzzy path matching on the very last path/filter segments.

I remember seeing this odd behaviour as well. I agree, the changed version makes more sense.

Apr 4 2018, 8:34 PM
brauch added a comment to D11921: CMake Server: Reuse existing build directory path when available.

Thank you for fixing this! 5.2 sounds ok to me, but let's not put it into the release we were planning for the next few days (i.e. not in 5.2.2), or should we?

Apr 4 2018, 4:44 PM

Apr 3 2018

brauch committed R32:ac58c101d13a: Fix build on haiku (authored by Zoltán Mizsei <zmizsei@extrowerk.com>).
Fix build on haiku
Apr 3 2018, 8:08 PM

Apr 1 2018

brauch added a comment to D11838: Turn on line numbers by default.

For KDevelop this is fine, I don't think we have any objection.

Apr 1 2018, 6:42 PM · Frameworks, Kate

Mar 30 2018

brauch added a comment to D11811: avoid Asan runtime error: shift exponent -1 is negative.

Change looks good (the previous code definitely looks like nonsense), but what does this mean for existing settings, saved previously?

Mar 30 2018, 5:57 PM · Frameworks, Kate

Mar 29 2018

brauch committed R32:384c75ccac53: Fix build of language plugins on Windows (authored by brauch).
Fix build of language plugins on Windows
Mar 29 2018, 8:36 PM

Mar 19 2018

brauch accepted D11431: Fix bug 386918.

Makes sense for me, I'm not aware of any common global shortcut using Alt+F9. Do you have commit access?

Mar 19 2018, 9:32 PM · KDevelop
brauch accepted D11474: Fixes #392031 - Ensure that codestyle.py always return something on stdout to unlock m_mutex..

Uh, good find. Thanks for the patch. I should somehow have used a mutex locker for that ...

Mar 19 2018, 9:30 PM

Mar 13 2018

brauch accepted D11291: Do not add return type to constructors declaration when editing definition in signature assistant..

Looks good to me, thanks!

Mar 13 2018, 5:13 PM · KDevelop

Mar 5 2018

brauch accepted D10705: UI improvement - add expandable line edit widget.

Looks godd, thank you!

Mar 5 2018, 8:20 AM

Feb 25 2018

brauch added a comment to D10311: Projects plugin: fix git file listing for umlauts such as äöü.

I looked it up, it depends on the file system. NTFS always uses UTF8, but FAT uses some weird 1980's charset. So I think this breaks if you open files from FAT file systems.

Feb 25 2018, 10:28 AM · Kate

Feb 24 2018

brauch accepted D10311: Projects plugin: fix git file listing for umlauts such as äöü.

Looks good to me, also because null termination sounds better than \n (filenames can easily contain \n although they usually don't).

Feb 24 2018, 11:56 PM · Kate

Jan 13 2018

brauch committed R32:71f7bed96dd0: fix crash when stopping process (authored by brauch).
fix crash when stopping process
Jan 13 2018, 9:58 PM
brauch closed D9858: dont crash when cancelling process.
Jan 13 2018, 9:57 PM
brauch updated the summary of D9858: dont crash when cancelling process.
Jan 13 2018, 3:41 PM

Jan 2 2018

brauch added reviewers for D9624: Install parser headers and cmake config files to support client packages: pprkut, kfunk, mtijink.

I'll add some people who have recently been working on kdev-php.

Jan 2 2018, 11:04 PM

Dec 30 2017

brauch accepted D9569: Fix wildcard matching for modelines.

Heh. Fix looks obviously correct to me (good find), and tests are always nice.

Dec 30 2017, 8:31 PM · Frameworks, Kate

Dec 24 2017

brauch committed R39:ee0d4a1009b9: Merge branch 'master' into multicursor (authored by brauch).
Merge branch 'master' into multicursor
Dec 24 2017, 9:44 PM

Dec 23 2017

brauch reopened D7660: Fix a regression caused by changing backspace key behavior.
Dec 23 2017, 10:44 PM · Frameworks
brauch added a comment to D7660: Fix a regression caused by changing backspace key behavior.

After this was submitted master doesn't compile for me, and if I fix the compile in the trivial way the test fails. Can you have another look?

Dec 23 2017, 10:40 PM · Frameworks
brauch accepted D9488: Fix appstream metadata filename and some content, and install it.

All the changes look sensible to me, thanks!

Dec 23 2017, 3:44 PM

Dec 21 2017

brauch accepted D9472: Format comments before setting them on the DUChain.

All three changes look good to me, although the formatting of comments is of course only useful until that clang bug is fixed ... but still, better than nothing and esp. when we already have the implentation in the codebase anyways.

Dec 21 2017, 11:58 PM

Dec 17 2017

brauch added a comment to D9363: Add support for __DIR__ and __TRAIT__..

String change is probably ok if there is enough time before the next release (a few weeks) AFAIU. My two cents about where to merge it would be, put it in 5.2 only if you consider it a bug fix -- i.e. if there are projects which do not parse properly because this feature is missing go for it, otherwise put it in master. There's nothing worse than adding regression bugs in patch releases because of minor features like this.

Dec 17 2017, 1:49 PM
brauch accepted D9363: Add support for __DIR__ and __TRAIT__..

I'm not up-to-date with the PHP standard, but guessing what the language feature does, code-wise this looks fine to me.

Dec 17 2017, 12:30 PM

Dec 14 2017

brauch accepted D9236: Fill lists of default-constructed types directly, not append any by value.

Not much is left of the original patch but this change makes sense to me ;) thanks!

Dec 14 2017, 9:09 AM

Dec 13 2017

brauch requested changes to D9316: Remove KF5::ItemModels from public interface of KDevPlatformUtil.

Yeah, sorry, I'm also against this. Linking an extra lib we depend on anyways is a problem a compuer has to deal with, extra code is a problem humans have to deal with. The former wins against the latter unless there is a very good reason why not.

Dec 13 2017, 10:38 PM
brauch committed R39:5deca9c7bed6: Merge branch 'master' into multicursor (authored by brauch).
Merge branch 'master' into multicursor
Dec 13 2017, 7:57 AM

Dec 12 2017

brauch added a comment to D9297: Fix #387391: Random responses from the C/C++ language support.

Hmm. There's a comment in the line above which states it explicitly uses canonicalFilePath to avoid issues with symlinks. If we resolve symlinks like you suggest, there will be situations like files which are part of a project but for which the project's root directory is not a prefix of the file path. Are we somewhat sure this doesn't break in other places?

Dec 12 2017, 2:53 PM · KDevelop

Dec 10 2017

brauch committed R39:3c86fe77cc01: implement "selection to aligned block" (authored by brauch).
implement "selection to aligned block"
Dec 10 2017, 8:42 PM
brauch committed R39:484013bb10a7: use multi-selection for "find all" (authored by brauch).
use multi-selection for "find all"
Dec 10 2017, 12:48 PM

Dec 7 2017

brauch added a comment to D9236: Fill lists of default-constructed types directly, not append any by value.

Is this actually faster? Why?

Dec 7 2017, 5:44 PM

Dec 4 2017

brauch added a comment to D9160: QList -> QVector where sizeof(T) > sizeof(void*).

Hmm, this will break build of all the plugins, no? Other than that, I'm in favour of this change, thanks for the work!

Dec 4 2017, 7:10 AM

Dec 3 2017

brauch committed R39:40685be38791: implement block select as a transform between block- and continuous selection (authored by brauch).
implement block select as a transform between block- and continuous selection
Dec 3 2017, 10:16 PM

Dec 1 2017

brauch committed R39:2ceedf47557f: fix selection being randomly cleared when drag-scrolling (authored by brauch).
fix selection being randomly cleared when drag-scrolling
Dec 1 2017, 6:56 PM

Nov 30 2017

brauch accepted D9074: Add "Open a Session" action for app jump list in task manager & Co..

Yes, cool feature, go for it!

Nov 30 2017, 7:59 PM
brauch committed R39:1967b78afaa7: rename "Mouse" selection mode to "Character" (authored by brauch).
rename "Mouse" selection mode to "Character"
Nov 30 2017, 1:09 PM
brauch committed R39:688b6ef1f3d1: implement two-anchor word select (authored by brauch).
implement two-anchor word select
Nov 30 2017, 12:32 PM
brauch committed R39:3bb6f6cb5b57: mostly fix word select: spaces handled properly now (authored by brauch).
mostly fix word select: spaces handled properly now
Nov 30 2017, 1:47 AM
brauch committed R39:0eee07f161ca: WIP: mouse select tests (authored by brauch).
WIP: mouse select tests
Nov 30 2017, 1:47 AM

Nov 29 2017

brauch accepted D9035: Properly launch Web Browser as detached process and handle errors if ocurred.

Thanks! I'll submit this as well.

Nov 29 2017, 1:39 PM
brauch added a comment to D9046: Update debug messages.

Ah, you don't have write access to KDE repos? Then I will submit this for you. Thanks!

Nov 29 2017, 12:18 PM
brauch committed R39:1f08c2613585: Merge branch 'master' into multicursor (authored by brauch).
Merge branch 'master' into multicursor
Nov 29 2017, 10:53 AM
brauch accepted D9034: Do not crash if there is still pending connections .

Alright, if you think it makes sense, submit it. It's certainly an improvement over the old situation. Thanks!

Nov 29 2017, 10:43 AM
brauch accepted D9046: Update debug messages.

Thanks, please submit!

Nov 29 2017, 10:31 AM

Nov 28 2017

brauch updated the diff for D9016: remove global statics.

Hmpf, right, QList compares its head to its tail which is certainly not atomic. I *think* for QVector this would be safe (it compares its size to zero), but better not take chances. Sorry that this is such a hassle.

Nov 28 2017, 10:58 PM
brauch accepted D9038: Bump cmake_qt5guiapp template to C++11 code, cmake 3.0 and QScopedPointer.

LGTM

Nov 28 2017, 9:59 PM
brauch updated the diff for D9016: remove global statics.

next try :D

Nov 28 2017, 5:21 PM
brauch added a comment to D9035: Properly launch Web Browser as detached process and handle errors if ocurred.

Yes, anyway I use that name because is consistent with the XDebugJob class (which uses the same mehtod name).

Nov 28 2017, 3:22 PM
brauch added a comment to D9034: Do not crash if there is still pending connections .

Ok, but can we not at least set the state to Ended when the connection breaks, or something like that?

Nov 28 2017, 3:14 PM
brauch added a comment to D9034: Do not crash if there is still pending connections .

Hm yes, ok, so it makes sense why this fixes the crash. I think we can keep this as a safety guard. However I think the actual problem is that the state is not set to EndedState when the debug session actually ends, which also has other undesirable consequences (e.g. KDevelop doesn't switch back to code view, etc.). Do you have an idea why this happens?

Nov 28 2017, 2:13 PM
brauch added a comment to D9035: Properly launch Web Browser as detached process and handle errors if ocurred.

To clarify my comment above, if what I said is correct, I would suggest to call the function "processFailedToStart"or so, and make it a non-slot.

Nov 28 2017, 2:08 PM
brauch requested changes to D9035: Properly launch Web Browser as detached process and handle errors if ocurred.
Nov 28 2017, 2:07 PM
brauch added a comment to D9034: Do not crash if there is still pending connections .

I think I hit this issue yesterday when testing kdev-xdebug, esp. the debugger wouldn't stop properly when stepping over the end of the program. I don't quite understand what issue exactly this change fixes and why, can you write a few lines?

Nov 28 2017, 2:00 PM
brauch accepted D9036: Updata signal/slot connect syntax to Qt5.

Very nice, thanks.

Nov 28 2017, 1:59 PM
brauch committed R67:8ff83dbcb2b7: allow configuring port as well (authored by brauch).
allow configuring port as well
Nov 28 2017, 11:07 AM

Nov 27 2017

brauch committed R67:6b6f89423cb0: make sure path starts with a slash (authored by brauch).
make sure path starts with a slash
Nov 27 2017, 10:53 PM