quicklisp / quicklisp-client

Quicklisp client.
http://www.quicklisp.org/
MIT License
298 stars 75 forks source link

Fix non-conforming code. #215

Closed karlosz closed 2 years ago

karlosz commented 2 years ago

According to the LOAD-TIME-VALUE entry for CLHS http://clhs.lisp.se/Body/s_ld_tim.htm, "It is guaranteed that the evaluation of form will take place only once when the file is loaded, but the order of evaluation with respect to the evaluation of top level forms in the file is implementation-dependent." Therefore, doing LOAD-TIME-VALUE of a function call where the function is defined in the same file is not portable code.

This is important to fix because old versions of CMU CL do take advantage of this fact, and future versions of SBCL may as well.

quicklisp commented 2 years ago

Hi @karlosz - why did you close the PR?

karlosz commented 2 years ago

This was by accident. Re-opening.