From: Eli Zaretskii <eliz@gnu.org>
To: Paul Eggert <eggert@cs.ucla.edu>
Cc: 21999@debbugs.gnu.org, dima@secretsauce.net, deng@randomsample.de
Subject: bug#21999: 25.0.50; Binary with --enable-checking immediately aborts with '0<=size'
Date: Wed, 25 Nov 2015 19:34:00 +0200 [thread overview]
Message-ID: <83610qx9tj.fsf@gnu.org> (raw)
In-Reply-To: <565568D9.2010005@cs.ucla.edu>
> Cc: deng@randomsample.de, 21999@debbugs.gnu.org
> From: Paul Eggert <eggert@cs.ucla.edu>
> Date: Tue, 24 Nov 2015 23:52:57 -0800
>
> Eli Zaretskii wrote:
> > After all, the size field of the pseudo-vector object is not
> > really a size, but a bunch of bitfields, so I'm not sure testing it in
> > its entirety makes sense. Paul?
>
> The point of the eassume is that it improves code quality in the common case
> where the size field is actually a size, since the compiler can generate better
> code when it knows that sizes are nonnegative. Also, it should improve checking
> a bit, in case someone trashes a size field.
>
> Looks like this is more trouble than it's worth, and we should get rid of that
> eassume, at least in emacs-25. Or have you fixed it?
I fixed this particular issue, which happened because several macros
on font.h used ASIZE and were called in the context of GC. Now each
such macro has a GC_* variant that calls gc_asize instead. If that's
a clean enough solution, we can safely move on. I was just worried
that ASIZE is no longer safe in GC, and any macro that uses it becomes
unsafe. But currently, any such remaining macros aren't called by GC
code, so there are no more known problems, just a subtlety.
prev parent reply other threads:[~2015-11-25 17:34 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-11-23 19:52 bug#21999: 25.0.50; Binary with --enable-checking immediately aborts with '0<=size' David Engster
2015-11-23 20:22 ` Eli Zaretskii
2015-11-23 21:05 ` David Engster
2015-11-24 9:55 ` Dima Kogan
2015-11-24 16:12 ` Eli Zaretskii
2015-11-24 16:32 ` David Engster
2020-08-15 6:05 ` Stefan Kangas
2015-11-25 7:52 ` Paul Eggert
2015-11-25 17:34 ` Eli Zaretskii [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
List information: https://www.gnu.org/software/emacs/
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=83610qx9tj.fsf@gnu.org \
--to=eliz@gnu.org \
--cc=21999@debbugs.gnu.org \
--cc=deng@randomsample.de \
--cc=dima@secretsauce.net \
--cc=eggert@cs.ucla.edu \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this public inbox
https://git.savannah.gnu.org/cgit/emacs.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).