Simplify SLOT-NAME-LISTS-FROM-SLOTS and friends
commit93b0a4e15aed97e43a22d4cc5fdf75faf537d533
authorJan Moringen <jmoringe@techfak.uni-bielefeld.de>
Tue, 15 Mar 2016 11:21:18 +0000 (15 12:21 +0100)
committerJan Moringen <jmoringe@techfak.uni-bielefeld.de>
Sun, 10 Jul 2016 12:26:37 +0000 (10 14:26 +0200)
treea1dae96e369f17405f7a30ed28837faffe2c61bd
parent4b8e2041f13fb82285c9c745325725691b35512e
Simplify SLOT-NAME-LISTS-FROM-SLOTS and friends

* Removed SYMBOL-LESSP and its helper SYMBOL-PKG-NAME

* SYMBOL-OR-CONS-LESSP compares symbol hashes instead of calling
  SYMBOL-LESSP

A FIXME comment suggested this change, giving the following rationale:

  Because of the existence of UNINTERN and RENAME-PACKAGE, the part of
  this ordering which is based on SYMBOL-PKG-NAME is not stable. This
  ordering is only used in to SLOT-NAME-LISTS-FROM-SLOTS, where it serves
  to "canonicalize the PV-TABLE's a bit and will hopefully lead to having
  fewer PV's floating around", so it sounds as though the instability
  won't actually lead to bugs, just small inefficiency. But still, it
  would be better to reimplement this function as a comparison based on
  SYMBOL-HASH:

  * stable comparison
  * smaller code (here, and in being able to discard SYMBOL-PKG-NAME)
  * faster code.
src/pcl/vector.lisp