Subject: Re: bug#76538: 31.0.50; 31.0.50; 31.0.50; feature/igc: using magit-section-cycle-global (S-TAB) and magit-section-toggle (TAB) in some random ways blocks GNU Emacs.

From: Eli Zaretskii <[email-masked]>
Date: Mon, 03 Mar 2025 21:17:01 +0200

> Date: Mon, 03 Mar 2025 17:07:02 +0000
> From: Pip Cet <[email-masked]>
> Cc: [email-masked], [email-masked], [email-masked], [email-masked], [email-masked], [email-masked]
> 
> "Eli Zaretskii" <[email-masked]> writes:
> 
> > As for the call with 'false' as 2nd arg: First, when it->bidi_p is
> > true, we don't use it->len for moving to the next display element.
> 
> True.  Moving the eassert inside the if might be the minimal change
> there.

No, I think setting it->len correctly would be a better, cleaner
change.  Not that I understand how come you hit this assertion while
we never do on master...