chicken-hackers
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Chicken-hackers] [PATCH] Drop obsolete C functions kept for bootstrappi


From: Peter Bex
Subject: [Chicken-hackers] [PATCH] Drop obsolete C functions kept for bootstrapping and simplify flonum literal decoding
Date: Fri, 19 May 2017 16:09:01 +0200
User-agent: Mutt/1.5.23 (2014-03-12)

Hi all,

Now we have a bootstrapping tarball, we can finally (FINALLY!) drop all
those unused C functions that have been lying around since the numeric
tower integration.

The first patch is very large, but it just removes unused code, and it
should be pretty straightforward.  We don't need to remove much at
the Scheme level because almost all these C functions are things used
by rewrites in c-platform.scm (or types.db) in the old CHICKEN.
I noticed that C_i_foreign_unsigned_ranged_integer_argumentp was
incorrectly marked OBSOLETE, so I've removed the comment there.

The second patch is to get rid of the "big fixnum" literal encoding
special handling, which was necessary when compiling a program with
fixnum literals on a 64-bit machine; if it was run on a 32-bit machine
such a large fixnum literal could result in a flonum.  This is no longer
needed, because in such cases we now use bignums, instead.

The third patch uses literals for +i, +2i and -i instead of jumping
through hoops to cope with a compiler with a reader that doesn't support
complex number literals.

The final patch cleans up and prepares us for dropping the remaining
OBSOLETE functions that I forgot to actually remove.  After the next
snapshot (pre2? rc1?), we can drop the matching C code.

Cheers,
Peter

Attachment: 0001-Remove-obsolete-unused-numeric-procedures.patch
Description: Text Data

Attachment: 0002-Replace-convert_string_to_number-with-simpler-flonum.patch
Description: Text Data

Attachment: 0003-Replace-explicit-complex-number-construction-with-li.patch
Description: Text Data

Attachment: 0004-Ensure-some-functions-marked-OBSOLETE-are-really-unu.patch
Description: Text Data

Attachment: signature.asc
Description: Digital signature


reply via email to

[Prev in Thread] Current Thread [Next in Thread]