search

thephoeron / let-over-lambda

Doug Hoyte's "Production" version of macros from Let Over Lambda, ready for ASDF and Quicklisp.
Other
131 stars 25 forks source link

Add defun! variant of defmacro! #13

Closed EuAndreh closed 9 years ago

EuAndreh commented 9 years ago

You think that's worth adding?

EuAndreh commented 9 years ago

Now one can write:

(defmacro! my-macro (a b)
  (declare (ignore a)) 
    ...)

... and get the expected behaviour, with or without docstrings.

PuercoPop commented 9 years ago

May I suggest alexandria:parse-body a less concrete way to extract declarations and docstrings?

EuAndreh commented 9 years ago

Nice tip @PuercoPop =]

thephoeron commented 9 years ago

@EuAndreh: Before we merge in this pull request, can you add some tests for the new forms?

EuAndreh commented 9 years ago

Sure. I'll do it tonight, as soon as I get home.

EuAndreh commented 9 years ago

Is that what you had in mind? Do you suggest adding some more things?

EuAndreh commented 9 years ago

Now one can test with (asdf:test-system :let-over-lambda).

thephoeron commented 9 years ago

Nice! thanks