Re: [PATCH] Fix (letrec ((ignore)))

[John Wiegley]

>>>>> Kaushal Modi <address@hidden> writes:

> Would it be better instead to fix the org-mode files for consistency?
> - Replace all instances of
> (letrec ((SOMEVAR)) ..)
> with
> (letrec ((SOMEVAR nil)) ..)

(letrec ((SOMEVAR))) should be an error, since the value is missing. If one
wishes to declare SOMEVAR with a nil value, the correct forms are:

    (letrec ((SOMEVAR nil)))
    (letrec (SOMEVAR))

> Question to emacs-devel: What would be the right approach?

We should give `letrec' the same treatment we've given to `setq' and `setf',
and then Org and other users should be corrected.

-- 
John Wiegley                  GPG fingerprint = 4710 CF98 AF9B 327B B80F
http://newartisans.com                          60E1 46C4 BD1A 7AC1 4BA2

signature.asc
Description: PGP signature