Clojure

Clarify limits of clojure.walk/macroexpand-all in docstring

Details

  • Type: Enhancement Enhancement
  • Status: Open Open
  • Priority: Trivial Trivial
  • Resolution: Unresolved
  • Affects Version/s: None
  • Fix Version/s: None
  • Component/s: None
  • Labels:
  • Patch:
    Code

Description

The clojure.walk/macroexpand-all function appears to be a general recursive macroexpansion, but it is not because it doesn't understand special forms such as let.

Current patch: 0001-CLJ-1240-Note-limits-of-clojure.walk-macroexpand-all.patch

The modified docstring in this patch notes that clojure.walk/macroexpand-all is not identical to macroexpansion by the Clojure compiler and should be used for development only.

Activity

Stuart Sierra made changes -
Field Original Value New Value
Patch Code [ 10001 ]
Description The {{clojure.walk/macroexpand-all}} function appears to be a general macroexpansion, but it is not because it doesn't understand special forms such as {{let}}.

The {{clojure.walk/macroexpand-all}} function appears to be a general recursive macroexpansion, but it is not because it doesn't understand special forms such as {{let}}.

*Current patch:* 0001-CLJ-1240-Note-limits-of-clojure.walk-macroexpand-all.patch

The modified docstring in this patch notes that {{clojure.walk/macroexpand-all}} is not identical to macroexpansion by the Clojure compiler and should be used for development only.
Attachment 0001-CLJ-1240-Note-limits-of-clojure.walk-macroexpand-all.patch [ 12079 ]
Assignee Stuart Sierra [ stuart.sierra ]
Summary Clarify docstring of clojure.walk/macroexpand-all Clarify limits of clojure.walk/macroexpand-all in docstring
Alex Miller made changes -
Labels walk

People

Vote (0)
Watch (0)

Dates

  • Created:
    Updated: