APPROVE
This is great, Jerry. I notice my test case in
http://mid.gmane.org/5157.0fd6.c7927.5602@parhasard.net now works.
With the xbm code, I note that this code now succeeds (where it shouldn’t),
because the product of most-positive-fixnum and (/ most-positive-fixnum 2)
is a negative C integer. We could check for that explicitly or multiply
unsigned integers.
(set-face-background-pixmap
'left-margin
`[xbm :data (,(/ most-positive-fixnum 2) ,most-positive-fixnum
"random-text")])
Ar an ceathrú lá déag de mí Meitheamh, scríobh Jerry James:
PATCH 21.5
I did not intend to take this long, or produce such a large patch, but it kind
of got away from me. This patch started life as an effort to fix the problem
that C integers larger than a machine word are not converted to bignums
properly. Then as I stumbled across various bugs and problems, it grew into a
larger cleanup of some of the bignum code. [...]
--
‘Liston operated so fast that he once accidentally amputated an assistant’s
fingers along with a patient’s leg, […] The patient and the assistant both
died of sepsis, and a spectator reportedly died of shock, resulting in the
only known procedure with a 300% mortality.’ (Atul Gawande, NEJM, 2012)
_______________________________________________
XEmacs-Patches mailing list
XEmacs-Patches(a)xemacs.org
http://lists.xemacs.org/mailman/listinfo/xemacs-patches