wanted: push :BIG-ENDIAN to *FEATURES*
Bug #901661 reported by
Luís Oliveira
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
SBCL |
Fix Released
|
Undecided
|
Unassigned |
Bug Description
The attached patch pushes :big-endian to *features* when appropriate. I offer three rationales:
1. Correctness. :big-endian and :little-endian are not the only possible endianness modes, although that is admittedly rather hypothetical.
2. Practicality. trivial-features[1] decided to push :big-endian (mostly because CCL did it), a handful of libraries depend on that and we want to turn trivial-features into a no-op some day.
3. Style. If big endian is The One True Endianness, then one could argue that using #+big-endian and #-big-endian is nicer than the alternative. :-)
Changed in sbcl: | |
status: | Fix Committed → Fix Released |
To post a comment you must log in.
Looks good to me, except for missing a commit message...