- User Since
- Dec 1 2015, 3:40 PM (177 w, 2 d)
Tue, Apr 23
Sat, Apr 20
Mind you guys that the internal implementation without a profile is kinda
broken, so this patch makes sense to me.
Wed, Apr 17
None, I’m ok with it
Fri, Apr 12
Ok, I'll update the code for that.
- Remove useless lambda
- Add missing label, Add tooltips
Thu, Apr 11
Only because I’m forbidden to remove configuration options that exist, for
me this is one of those niche cases that shouldn’t exist.
I tougth about showing the paths but my first tougth was “why do the user
cares where this is being stored”?
Wed, Apr 10
This is more future proof as if we change the type of the function we don’t
need to change that return.
We set this variable in two or three different files and in a few different
methods. I remember that one of the first things that I tried to do for
konsole was to simplify the creation of sessions (and this broke a lot of
code and you where not happy) but it seems that we do need to make some
compromise and align that.
I beg your pardon? (How do I test the new session? konsole is working
for me so maybe it's something I missed)
I forgot this for quite a while, I'll update today.
Tue, Apr 9
I'll add a backtrace as soon as I test this again. since I'm succeeding in my konsole code cleanup I'll get this back on track.
too many things to fix here and there. :)
we don't have a clangformat file specified for kde projects, maybe this is
something good to add. I voted for global .editoconfigfiles but I was
- Further simplify code
When we have multiple tabs / splits this is not confusing?
this should fix most of the issues. I have another patch to go related to actions, but this one needs to land first.
- Fix action identifiers
@ngraham can you accept even with the shortcut issue that you are having? I have other reviews that depend on this one. I promise you to fix the shortcut bug <3
- Use reference with range for loops
the adition of the phabricator configuration file is unrelated to a clean, please split in two patches.
the specific warning is https://clang.llvm.org/extra/clang-tidy/checks/readability-else-after-return.html
I believe that he builds in clang and clang 8 added quite a bunch of new
Mon, Apr 8
true, but then the instance is there anyway. what we gain from this?
Hm? This doesn’t touches that.
waiting for @hindenburg ok before landing. :)
- Better variable name
- Whitespace / Todos removal
sorry, my mistake. this is correct.
that looks factually wrong. what are you trying to accomplish here?
Session Save / Restore
Fri, Apr 5
OpenMP is missing, Andre is working on it.
Thu, Apr 4
This code can load sessions, it has a better behavior than the old session load code, but still has bugs. I'll finish this luckly today / tomorrow.
- Sessions partially working
- Code cleanup
I'll remove the debugs / message handlers as soon as everything is working. but it's one of the fastest ways for me to keep the debug messages between runs so I can understand what's wrong.
the session is correctly saved, and the SessionManager correctly loads all the sessions back, the ViewManager however is broken on load. that's what I'm debugging now. so even tougth we don't display any terminal, they are up and running.
- messageHandler + tons of debug
- Sabe json correctly in the configuration
Well, I finally managed to discover something:
when MainWindow::saveProperties is triggered we already destroyed the sessions and tabs, and that's the reason it's not saving anything.
I'm trying to debug the destructor order now to see if I can change something.
Wed, Apr 3
- Implement the Load of Sessions
- Solve crash while quitting konsole via Session End
- Still Broken, do not review unless you can help. :)
Sat, Mar 30
The code for sessions is commented out, in the original review you said
that you would help me see the saving the session code as now we have to
deal with splits.
Fri, Mar 29
- Remove empty line
This is mostly working, but I will not merge this untill we fix the current state of the shortcuts (that I'm preparing in a separate patch).
after the shortcuts are correct, there will be a follow up patch here to use the switcher style *or* the next-prev-tab style (as we have today) selectable by the user, defaulting to the current behavior.
- Simplify handling of code
- Handle list with the shortcut selected
- Mostly working
- Remove mnemonics from text
I’m in a konsole spree this week so expect patches.
Thu, Mar 28
how that the splits are merged, I'm preparing a round of patches that where rotting in my pc.
- Use one liners where appropriate.
- Remove sender() and QVariant() conversions
- Declare + initialize, instead of setting the value later
- Don't create unecessary temporaries
- Extract code from huge function into a smaller one