bug#75199: 30.0.93; Emacs crashes when completing with corfu

[Eli Zaretskii]

> From: Daniel Mendler <mail@daniel-mendler.de>
> Cc: Eli Zaretskii <eliz@gnu.org>,  luangruo@yahoo.com,  75199@debbugs.gnu.org
> Date: Mon, 30 Dec 2024 19:00:45 +0100
> 
> Zhengyi Fu <i@fuzy.me> writes:
> 
> > On 2024-12-30 23:24, Eli Zaretskii wrote:
> >
> >> Earlier you wrote:
> >> 
> >>> The error is probably caused by the width value of 0 passed to
> >>> XCreatePixmapFromBitmapData().
> >>> It seems gui_intersect_rectangles() returns true even if the resulted
> >>> rectangle is empty.  Is that correct behavior?
> >> Can you tell how this happens?  Is corfu doing something special with
> >> the fringes?
> >
> > I noticed the following calls in corfu--popup-show before the error:
> >
> > * define-fringe-bitmap(corfu--bar [-241] 1 8 (top periodic))
> > * define-fringe-bitmap(corfu--nil [])
> 
> I've adjusted the second call to the following, also for compatibility
> with older Emacs versions.
> 
> (define-fringe-bitmap 'corfu--nil [0] 1 1)

Doesn't

  (define-fringe-bitmap 'corfu--nil [0])

work as well?  If it doesn't, I guess the doc string is inaccurate?

> For the future, maybe `define-fringe-bitmap' should either check its
> argument carefully and disallow empty bitmaps, or empty bitmaps should
> be explicitly allowed?

Adding such argument-checking is fine by me, patches welcome.

Thanks.