Quote from my message on the forum :
After updating mousepad to 0.5.2.
I'm unable to use "select all", either by the shortcut (ctrl+A) or clicking in the menu "select all".
Now the strange thing begins.. On my desktop (which uses GCC 10.2.0 and bdver2 march), there is no issue. Works as expected.
It only happens on my laptop, which uses GCC 9.3.0 and core2 march..
Otherwise no difference between my systems, same profile (hardened), same packages and both are fully updated..
I tried to compile it with debug enabled and ran it with gdb but since it doesn't crash, it doesn't show much info.
So legit bug ? Or something else ?
edit : Made two gif to show the issue :
An user can confirm the issue on his system as well, quote :
I can verify this glitch. [...]
My primary desktop is 9.3.0 and my system compiled native (Intel(R) Core(TM) i7-2600 CPU @ 3.40GHz).
Created attachment 688143 [details]
emerge --info on core2/GCC9 system
I have checked this on a core2 system, I have checked my aarch64 machines an a i686 machine. All work as expected, which would suggest a permissions issue with ~/.config/Mousepad
(In reply to Jory A. Pratt from comment #2)
> I have checked this on a core2 system, I have checked my aarch64 machines an
> a i686 machine. All work as expected, which would suggest a permissions
> issue with ~/.config/Mousepad
Permissions were fine but I deleted ~/.config/Mousepad to start fresh anyway -> Issue persists.
I tried also to create a new user with a fresh xfce profile, same issue.
Then I put mousepad-0.5.1 in my localrepo and compiled it and it works fine again. So it seems mousepad-0.5.2, at least on my system and figueroa's system on the forum.
(In reply to David Duchesne from comment #3)
> (In reply to Jory A. Pratt from comment #2)
> > I have checked this on a core2 system, I have checked my aarch64 machines an
> > a i686 machine. All work as expected, which would suggest a permissions
> > issue with ~/.config/Mousepad
> Permissions were fine but I deleted ~/.config/Mousepad to start fresh anyway
> -> Issue persists.
> I tried also to create a new user with a fresh xfce profile, same issue.
> Then I put mousepad-0.5.1 in my localrepo and compiled it and it works fine
> again. So it seems mousepad-0.5.2, at least on my system and figueroa's
> system on the forum.
*So it seems mousepad-0.5.2 has an issue
I confirm the existence of this bug. Newly installed Mousepad-0.5.2 no select-all.
Downgraded to 0.5.1 via local repo, select-all works as normal.
I'm on a stable, up-to-date, amd64 system.
After some compile test, I found that the following commit is causing the issue on my amd64 / core2 system.
The previous commit doesn't cause any issue (select all works)
I reproduced this and narrowed it down to having glib-2.66.7 compiled with gcc-9.3.0 while gcc-10.2.0 produces a working library. The aforementioned commit switches to signalling glib for this bit of buffer management instead of doing it in-house. The symptom is such that if you first select any amount of text and then hit Ctrl+A, it deselects everything, as if glib is using null or only one position for both the beginning and end of the new selection, instead of using the beginning and end of the text.
P.S. Normally, CFLAGS="-march=amdfam10 -O2 -pipe" so I tried the others:
-O, -O1, -O2, -O3, -Ofast, -Og -> fail
-Os, -O0 -> pass
Thank you for your detailed research!
Let's see what glib maintainers think of it. CC-ing toolchain in case they have any idea what could go wrong here.
I think the code behaviour in that mousepad commit is just undefined behaviour.
select-all signal takes a boolean parameter, that says if to select all or unselect all. If you don't pass it, it's probably undefined behaviour and it does whatever happens to be in some next bytes on signal emission data or something.
They need to do
g_signal_emit_by_name (window->active->textview, "select-all", TRUE);
g_signal_emit_by_name (window->active->textview, "select-all");
Thanks for the reply. I'll send a patch upstream.
Let's wait if upstream replies, and then I'll backport it.
The bug has been closed via the following commit(s):
Author: Michał Górny <firstname.lastname@example.org>
AuthorDate: 2021-03-01 13:12:55 +0000
Commit: Michał Górny <email@example.com>
CommitDate: 2021-03-01 13:15:09 +0000
app-editors/mousepad: Backport select-all fix
Signed-off-by: Michał Górny <firstname.lastname@example.org>
.../mousepad/files/mousepad-0.5.2-select-all.patch | 28 ++++++++++++++++++++++
...sepad-0.5.2.ebuild => mousepad-0.5.2-r1.ebuild} | 4 ++++
...sepad-0.5.3.ebuild => mousepad-0.5.3-r1.ebuild} | 4 ++++
3 files changed, 36 insertions(+)
oh, that is funny. Obviously I never expected 'select-all' handler to have its own power to select none. TIL:RTFD. Thanks!