diff options
author | Kaz Kylheku <kaz@kylheku.com> | 2020-06-19 06:29:45 -0700 |
---|---|---|
committer | Kaz Kylheku <kaz@kylheku.com> | 2020-06-19 06:29:45 -0700 |
commit | 09091bedcc6dd570262de4a72c362b161a492df7 (patch) | |
tree | 5d68d965c3a7b28a7908254cbe3e9e3e9b9589aa /lib.c | |
parent | 2f2d654c20c75a7cc6e5c1c28803f475c929abba (diff) | |
download | txr-09091bedcc6dd570262de4a72c362b161a492df7.tar.gz txr-09091bedcc6dd570262de4a72c362b161a492df7.tar.bz2 txr-09091bedcc6dd570262de4a72c362b161a492df7.zip |
itypes: remove silly itypes_little_endian.
Curiously, the itypes_little_endian variable was introduced in
April 2017, in a the first commit implementing buffers. The
variable was initialized, but never referenced. About a month
after that, I introduced HAVE_LITTLE_ENDIAN config.h macro,
which is always present and defined as 0 or 1. This was used
right away in the implementation of FFI bitfields, and later
on in other FFI work. In spite of HAVE_LITTLE_ENDIAN, almost
exactly a year after itypes_little_endian was introduced, I
mistakenly put that variable to use instead of recognizing it
as superfluous and removing it in favor of using
HAVE_LITTLE_ENDIAN. That is what I'm doing now.
* itypes.c (itypes_little_endian): Variable removed.
(itypes_init): Function removed, since its only job is to
initialize this variable.
* itypesl.h (itypes_little_endian, itypes_init): Declarations
removed.
* lib.c (init): Call to itypes_init removed.
* parser.c (read_file_common): Instead of
itypes_little_indian, use HAVE_LITTLE_ENDIAN, which
the configuration script always exists and is defined
as 0 or 1.
Diffstat (limited to 'lib.c')
-rw-r--r-- | lib.c | 1 |
1 files changed, 0 insertions, 1 deletions
@@ -13086,7 +13086,6 @@ void init(val *stack_bottom) hash_init(); struct_init(); tree_init(); - itypes_init(); buf_init(); ffi_init(); sysif_init(); |