From: Han-Wen Nienhuys <hanwen@xs4all.nl>
Subject: Re: stumped by scm_car/scm_cdr
Date: Thu, 7 Oct 2004 20:52:58 +0200 [thread overview]
Message-ID: <16741.37002.668216.221013@byrd.xs4all.nl> (raw)
In-Reply-To: <16740.33606.743452.458667@byrd.xs4all.nl>
hanwen@xs4all.nl writes:
> > One more idea: it might be that you (harmlessly) used SCM_CAR or
> > SCM_CDR on a non-pair but scm_car/scm_cdr are now complaining about
> > this by throwing an error. That error might happen in a context where
> > throwing does not work, or might lead to endless recursion, or some
> > other anomaly. Try replacing the call to scm_wrong_type_arg_msg in
> > scm_car and scm_cdr with a simple abort.
>
> I tried to remove all fishy looking places where SCM_C[AD]R were used,
> and now it seems to work OK. You might want to consider just doing
> abort() or a something similar as a standard. If the C code uses the
> API incorrectly, then (IMO) this is not something that should be
> signaled at Scheme level using scm_error(), because no amount of
> Scheme coding can fixup such errors.
for the record, these fishy places were mostly in GC mark functions,
where I did
smobbed_type * p = (smobbed_type*) SCM_CDR(scm)
iso.
smobbed_type * p = (smobbed_type*) SCM_GC_CELL_WORD1(scm)
--
Han-Wen Nienhuys | hanwen@xs4all.nl | http://www.xs4all.nl/~hanwen
_______________________________________________
Guile-devel mailing list
Guile-devel@gnu.org
http://lists.gnu.org/mailman/listinfo/guile-devel
next prev parent reply other threads:[~2004-10-07 18:52 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2004-09-24 23:29 stumped by scm_car/scm_cdr Han-Wen Nienhuys
2004-09-25 15:54 ` Marius Vollmer
2004-09-25 23:19 ` Han-Wen Nienhuys
2004-09-29 15:37 ` Marius Vollmer
2004-10-06 23:44 ` Han-Wen Nienhuys
2004-10-07 18:52 ` Han-Wen Nienhuys [this message]
2004-10-07 21:55 ` Marius Vollmer
2004-10-07 22:12 ` Marius Vollmer
2004-10-07 22:51 ` Han-Wen Nienhuys
2004-10-07 23:41 ` Han-Wen Nienhuys
2004-10-18 17:47 ` Marius Vollmer
2004-10-24 12:25 ` Han-Wen Nienhuys
2004-11-04 16:49 ` Marius Vollmer
2004-11-08 6:48 ` Dirk Herrmann
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/guile/
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=16741.37002.668216.221013@byrd.xs4all.nl \
--to=hanwen@xs4all.nl \
/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.
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).