This is the mail archive of the guile@sourceware.cygnus.com mailing list for the Guile project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]

Re: patches to backtrace.c


Dirk Herrmann <dirk@ida.ing.tu-bs.de> writes:

> Hey, more patches and thoughts -- and it's not christmas yet.
> 
> Again, I eliminated redundant SCM_NIMP tests, the patches are appended
> below.  The patch of function display_expression is not obviously valid,
> but display_expression is static and only called from one single place,
> where the predicate is always fulfilled.  Further, scm_unmemoize performs
> a SCM_NIMP test itself.

Well, I'd rather not make that one change simply because it's making
assumptions about the caller that are not stated clearly, and hard to
enforce reliably (after all, my change was about safety, not
performance).  The other two changes I've applied to my copy and will
commit soon.

Thanks!
Greg

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]