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

names at top level should be limited to names exported



Regarding the reference implementation for SRFI-45:

I recommend changing

(define (strict x) (cons 'value x))

(define-syntax delay
  (syntax-rules ()
    ((delay exp) (lazy (strict exp)))))

to

(define-syntax delay
   (syntax-rules ()
      ((delay exp) (lazy (cons 'value x)))))

In order to avoid binding the name 'strict', which is not
part of the proposed interface.

This is because 'strict' is a word that may be used in other
code or modules that someone is trying to interface with, and
confusion or breakage could result from potential multiple
definitions.

Overall, this is good work.

				Bear