- User Since
- Feb 10 2018, 7:40 AM (49 w, 3 d)
Nov 1 2018
Oct 27 2018
Also, summary is incorrect:
This is just the fist part, more will follow.
Oct 19 2018
Oct 16 2018
Thanks and welcome aboard Miroslav!
Other than minor things noted inline, looks good! Thanks Alex.
Looks much better, thanks!
Sep 22 2018
Thanks Davide! This looks less cluttered.
More importantly, whenever we make a change after this is merged, we will see the warning and fix the code. We put an effort before v2.7 release to make Krusader warning-free and we should keep it this way. So now the -Wsuggest-override is on the list of warning directives that we track. Thanks Davide (and Toni ;) )!
Sep 15 2018
Why not to update the flags for both debug and release (CMAKE_CXX_FLAGS var)? What's your reasoning? Does it hurt the release build?
Sep 13 2018
Indeed, all warnings of this type are fixed. Thanks Davide!
Sep 12 2018
and reproducible adding -Wsuggest-override or -Winconsistent-missing-override (not sure, haven't tested with clang) to CMAKE_CXX_FLAGS
Should we add this into cmake config file to ensure that these flags are always on?
Aug 23 2018
Aug 16 2018
Aug 12 2018
Aug 5 2018
Apply to stable branch. This will go after changelog / news update commits.
- updated documentation ChangeLog
Jul 31 2018
Jul 29 2018
Thanks Toni! Works nicely.
Please also add FIXED: tag. See https://phabricator.kde.org/w/krusader/#commit-patch-guidelines
Please add fallback logic for the old setting. We can deprecate the fallback code with 2.9+ release.
Jul 24 2018
Jul 18 2018
Thanks! Please use git cherry-pick -x when updating 2.7 branch in the future.
Jul 17 2018
For this kind of change please go through reviews in the future because any change becomes live automatically.
Tested, works as expected. Thanks a lot Toni!
Jul 15 2018
I have trouble applying the patch to current master:
$ arc patch D13499 INFO Base commit is not in local repository; trying to fetch. Created and checked out branch arcpatch-D13499. Checking patch krusader/Panel/panelfunc.cpp... error: while searching for:
Jul 8 2018
Reviewed and tested - works nicely!
Please push this.
Toni, this sounds like a great change! Thanks for working on this!
Jul 6 2018
Thanks Toni and Yuri.
It won't work. It will produce a.txt, b.txt, c.txt according to the 3rd screenshot because #name stands for the full file name in the proposed syntax.
"How do I insert a hash symbol into the file name?" - is also a problem.
Jun 28 2018
Toni, thanks for checking and testing.
Jun 22 2018
Thanks for the review and useful tips, Toni!
This thing bugs me for a while, and now I finally have a chance to fix it. I think it's a regression from previous versions because I remember my frustration when it was introduced. According to the related bug, it should be before v2.4.0-beta1, but I don't have any really old versions installed. Toni, you mentioned an old VM with an old Krusader on it — can you check, please?