[CLJS-1518] Case macro expansion evaluates expression twice Created: 21/Dec/15 Updated: 09/May/17 Resolved: 09/May/17
|Affects Version/s:||1.7.145, 1.7.228|
|Reporter:||Darrick Wiebe||Assignee:||David Nolen|
The issue is present in version 1.7.189.
|Patch:||Code and Test|
The value being checked is evaluated twice if all of the test values are keywords.
|Comment by Mike Fikes [ 31/Jan/16 11:38 PM ]|
Patch takes advantage of the existing gensym as a temp place to stash the evaluated value before test / FQN conversion.
Adds a unit test specifically checking for single evaluation in this case.
|Comment by Mike Fikes [ 31/Jan/16 11:40 PM ]|
With the patch, Darrick's macroexpansion example becomes:
|Comment by António Nuno Monteiro [ 07/Nov/16 10:11 AM ]|
Still present in 1.9.293.
|Comment by David Nolen [ 09/May/17 8:20 AM ]|