On Fri, Nov 08, 2019 at 09:29:29PM +0100, Oswald Buddenhagen wrote:
I've pushed a commit up to the branch 'kevin/sigwinch-fixes'. Would you mind giving that a try and seeing if it resolves the issue?no luck. and i'm additionally getting a blank screen when invoking the mailbox browser, until i ctrl-l (that was broken before, too, just differently). as before, things normalize once i resize the window.
You mean as a result of the branch commit? That seems highly unlikely. The patch simply removes the race condition of resetting the flag after the redraw. All the other flag handlers in Mutt already do it that way.
Mutt installs its signal handlers twice - once before initscr(), and once after. The second installation was originally only for slang, but was changed to always run 18 years ago.that unification was probably wrong - slang installs an own handler, while ncurses won't overwrite an already installed one (if it's built to even install an own handler at all, but that's the default).
That's what the code indicates. So for ncurses Mutt installs its handler first, while for slang it installs afterward. The "Solaris 8 hack" was installing the handlers afterwards for ncurses too, but installing the same handlers again shouldn't lead to this odd behavior. Where is the race?
but that double init cries HACK!! in the first place, and i wouldn't be surprised if that was the source of the race.
By all means, comment it out and see: main.c:591. However I'm starting to suspect something funky with your terminal or terminfo...
-- Kevin J. McCarthy GPG Fingerprint: 8975 A9B3 3AA3 7910 385C 5308 ADEF 7684 8031 6BDA
signature.asc
Description: PGP signature
