* bug#50699: 28.0.50; Assertation failed when built with checking and mouse face is set to :box 0 [not found] <874kaf4h94.fsf.ref@yahoo.com> @ 2021-09-20 11:29 ` Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors 2021-09-20 12:23 ` Eli Zaretskii 0 siblings, 1 reply; 5+ messages in thread From: Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors @ 2021-09-20 11:29 UTC (permalink / raw) To: 50699 When I set the box of the mouse face on a section of text to 0, such as with (insert (propertize "abc" foo 'mouse-face '(:box 0))), and move the mouse over that, and I build with enable-checking, Emacs crashes with: xfaces.c:5976: Emacs fatal error: assertion failed: XFIXNUM (box) != 0 Fatal error 6: Aborted In GNU Emacs 28.0.50 (build 57, i386-pc-solaris2.11, X toolkit, Xaw scroll bars) of 2021-09-19 built on beached-whale Repository revision: bc59c98f096f7d01cbccf98d4fdd9c3f0385e896 (please ignore this, I reproduced the bug on latest master) Repository branch: HEAD Windowing system distributor 'The X.Org Foundation', version 11.0.12101002 Configured using: 'configure --with-gnutls=ifavailable --disable-acl' Configured features: GIF JPEG MODULES PDUMPER PNG THREADS TIFF TOOLKIT_SCROLL_BARS X11 XDBE XIM XPM LUCID ZLIB Important settings: value of $LANG: en_GB.UTF-8 locale-coding-system: utf-8-unix Major mode: Shell Minor modes in effect: shell-dirtrack-mode: t tooltip-mode: t global-eldoc-mode: t electric-indent-mode: t mouse-wheel-mode: t tool-bar-mode: t menu-bar-mode: t file-name-shadow-mode: t global-font-lock-mode: t font-lock-mode: t blink-cursor-mode: t auto-composition-mode: t auto-encryption-mode: t auto-compression-mode: t line-number-mode: t indent-tabs-mode: t transient-mark-mode: t Features: (shadow sort mail-extr emacsbug message rmc puny rfc822 mml mml-sec epa derived epg rfc6068 epg-config gnus-util rmail rmail-loaddefs mm-decode mm-bodies mm-encode mail-parse rfc2231 mailabbrev gmm-utils mailheader sendmail rfc2047 rfc2045 ietf-drums mm-util mail-prsvr mail-utils mule-util rect cus-start cus-load pixel-scroll smerge-mode diff whitespace term disp-table ehelp pulse color vc-mtn vc-hg vc-bzr vc-src vc-sccs vc-svn vc-cvs vc-rcs thingatpt etags fileloop generator xref project noutline outline dired-aux dired dired-loaddefs cl-extra help-mode tabify imenu man time-date shell pcomplete compile text-property-search gdb-mi gud comint ansi-color ring misearch multi-isearch vc-git diff-mode easy-mmode vc vc-dispatcher bug-reference cc-mode cc-fonts cc-guess cc-menus cc-cmds cc-styles cc-align cc-engine cc-vars cc-defs edmacro kmacro sly-autoloads info package browse-url url url-proxy url-privacy url-expand url-methods url-history url-cookie url-domsuf url-util mailcap url-handlers url-parse auth-source cl-seq eieio eieio-core cl-macs eieio-loaddefs password-cache json subr-x map url-vars seq byte-opt gv bytecomp byte-compile cconv cl-loaddefs cl-lib iso-transl tooltip eldoc electric uniquify ediff-hook vc-hooks lisp-float-type elisp-mode mwheel term/x-win x-win term/common-win x-dnd tool-bar dnd fontset image regexp-opt fringe tabulated-list replace newcomment text-mode lisp-mode prog-mode register page tab-bar menu-bar rfn-eshadow isearch easymenu timer select scroll-bar mouse jit-lock font-lock syntax font-core term/tty-colors frame minibuffer cl-generic cham georgian utf-8-lang misc-lang vietnamese tibetan thai tai-viet lao korean japanese eucjp-ms cp51932 hebrew greek romanian slovak czech european ethiopic indian cyrillic chinese composite charscript charprop case-table epa-hook jka-cmpr-hook help simple abbrev obarray cl-preloaded nadvice button loaddefs faces cus-face macroexp files window text-properties overlay sha1 md5 base64 format env code-pages mule custom widget hashtable-print-readable backquote threads dynamic-setting x-toolkit x multi-tty make-network-process emacs) Memory information: ((conses 8 319550 36788) (symbols 24 15660 1) (strings 16 56375 3223) (string-bytes 1 2028150) (vectors 8 30879) (vector-slots 4 461927 37450) (floats 8 160 205) (intervals 28 45544 746) (buffers 564 43)) ^ permalink raw reply [flat|nested] 5+ messages in thread
* bug#50699: 28.0.50; Assertation failed when built with checking and mouse face is set to :box 0 2021-09-20 11:29 ` bug#50699: 28.0.50; Assertation failed when built with checking and mouse face is set to :box 0 Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors @ 2021-09-20 12:23 ` Eli Zaretskii 2021-09-20 12:54 ` Eli Zaretskii 2021-09-21 0:35 ` Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors 0 siblings, 2 replies; 5+ messages in thread From: Eli Zaretskii @ 2021-09-20 12:23 UTC (permalink / raw) To: Po Lu; +Cc: 50699 > Date: Mon, 20 Sep 2021 19:29:43 +0800 > From: Po Lu via "Bug reports for GNU Emacs, > the Swiss army knife of text editors" <bug-gnu-emacs@gnu.org> > > > When I set the box of the mouse face on a section of text to 0, such as > with (insert (propertize "abc" foo 'mouse-face '(:box 0))), and move the > mouse over that, and I build with enable-checking, Emacs crashes with: > > xfaces.c:5976: Emacs fatal error: assertion failed: XFIXNUM (box) != 0 > Fatal error 6: Aborted Thanks, should be fixed now. ^ permalink raw reply [flat|nested] 5+ messages in thread
* bug#50699: 28.0.50; Assertation failed when built with checking and mouse face is set to :box 0 2021-09-20 12:23 ` Eli Zaretskii @ 2021-09-20 12:54 ` Eli Zaretskii 2022-08-26 12:30 ` Lars Ingebrigtsen 2021-09-21 0:35 ` Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors 1 sibling, 1 reply; 5+ messages in thread From: Eli Zaretskii @ 2021-09-20 12:54 UTC (permalink / raw) To: luangruo; +Cc: 50699 > Date: Mon, 20 Sep 2021 15:23:08 +0300 > From: Eli Zaretskii <eliz@gnu.org> > Cc: 50699@debbugs.gnu.org > > > When I set the box of the mouse face on a section of text to 0, such as > > with (insert (propertize "abc" foo 'mouse-face '(:box 0))), and move the > > mouse over that, and I build with enable-checking, Emacs crashes with: > > > > xfaces.c:5976: Emacs fatal error: assertion failed: XFIXNUM (box) != 0 > > Fatal error 6: Aborted > > Thanks, should be fixed now. Btw, there's a lot of strange stuff going on with this. First, we accept (:box N) and (:box (N . M)) forms, but they are not documented. The form (:box (:line-width (N . M))) is also undocumented. Second, we disallow zero in (:box 0), but allow it in (:box (0 . 0)) and also in (:box (:line-width 0)) (and handle them as if 1 was specified instead of zero). It looks like this didn't change since Emacs 21, so it isn't bitrot of any kind. Does anyone know why we have these strange inconsistencies? ^ permalink raw reply [flat|nested] 5+ messages in thread
* bug#50699: 28.0.50; Assertation failed when built with checking and mouse face is set to :box 0 2021-09-20 12:54 ` Eli Zaretskii @ 2022-08-26 12:30 ` Lars Ingebrigtsen 0 siblings, 0 replies; 5+ messages in thread From: Lars Ingebrigtsen @ 2022-08-26 12:30 UTC (permalink / raw) To: Eli Zaretskii; +Cc: luangruo, 50699 Eli Zaretskii <eliz@gnu.org> writes: > Btw, there's a lot of strange stuff going on with this. > > First, we accept (:box N) and (:box (N . M)) forms, but they are not > documented. The form (:box (:line-width (N . M))) is also > undocumented. The latter is mentioned: @item @code{(:line-width (@var{vwidth} . @var{hwidth}) :color @var{color} :style @var{style})} But I've now clarified that the elements are optional. I don't think we need to document the alternate forms -- I think they just confuse the issue. > Second, we disallow zero in (:box 0), but allow it in (:box (0 . 0)) > and also in (:box (:line-width 0)) (and handle them as if 1 was > specified instead of zero). And I don't think we need to mention this, either. 0 is a degenerate value for something that's supposed to be about a box's line widths, so either erroring out or defaulting to 1 makes as much sense (which is probably why one form does the one thing and the other does the other). Po Lu <luangruo@yahoo.com> writes: > Eli Zaretskii <eliz@gnu.org> writes: > >> Thanks, should be fixed now. > > Works as advertised, thanks :) So I'm closing this bug report. ^ permalink raw reply [flat|nested] 5+ messages in thread
* bug#50699: 28.0.50; Assertation failed when built with checking and mouse face is set to :box 0 2021-09-20 12:23 ` Eli Zaretskii 2021-09-20 12:54 ` Eli Zaretskii @ 2021-09-21 0:35 ` Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors 1 sibling, 0 replies; 5+ messages in thread From: Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors @ 2021-09-21 0:35 UTC (permalink / raw) To: Eli Zaretskii; +Cc: 50699 Eli Zaretskii <eliz@gnu.org> writes: > Thanks, should be fixed now. Works as advertised, thanks :) ^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2022-08-26 12:30 UTC | newest] Thread overview: 5+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- [not found] <874kaf4h94.fsf.ref@yahoo.com> 2021-09-20 11:29 ` bug#50699: 28.0.50; Assertation failed when built with checking and mouse face is set to :box 0 Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors 2021-09-20 12:23 ` Eli Zaretskii 2021-09-20 12:54 ` Eli Zaretskii 2022-08-26 12:30 ` Lars Ingebrigtsen 2021-09-21 0:35 ` Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors
Code repositories for project(s) associated with this external index https://git.savannah.gnu.org/cgit/emacs.git https://git.savannah.gnu.org/cgit/emacs/org-mode.git This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.