[#CLJ-1043] Unordered literals does not preserve left-to-right evaluation of arguments - Clojure JIRA

Details

Type: Defect
Status: Open
Priority: Minor
Resolution: Unresolved
Affects Version/s: Release 1.4
Fix Version/s: None
Component/s: None
Labels:
None

Description

Given: (defn f [x] (println x) x)

#{(f 2) (f 1)}

Prints:

1
2

But expected would be:

2
1

This issue is related to CLJS-288

Activity

Ascending order - Click to sort in descending order

Hide

Permalink

Andy Fingerhut added a comment - 23/Sep/12 6:01 PM

I have the same question as David Nolen for CLJS-288: Is this a bug, or just behavior you didn't expect?

It seems that vectors preserve the order of evaluation, so if you really want to control evaluation order you could use something like (set [(f 2) (f 1)]) or (set (map f [2 1])).

Show

Andy Fingerhut added a comment - 23/Sep/12 6:01 PM I have the same question as David Nolen for CLJS-288: Is this a bug, or just behavior you didn't expect? It seems that vectors preserve the order of evaluation, so if you really want to control evaluation order you could use something like (set [(f 2) (f 1)]) or (set (map f [2 1])).

Hide

Permalink

Brandon Bloom added a comment - 23/Sep/12 7:38 PM

I'd consider the expected default behavior of any syntax or macro to evaluate each sub-form once each, from left to right. Conditional, repeated, or out-of-order evaluation should be documented as deviations from that norm. If you buy that, then this is either a code or a documentation bug. My vote is for code bug.

Show

Brandon Bloom added a comment - 23/Sep/12 7:38 PM I'd consider the expected default behavior of any syntax or macro to evaluate each sub-form once each, from left to right. Conditional, repeated, or out-of-order evaluation should be documented as deviations from that norm. If you buy that, then this is either a code or a documentation bug. My vote is for code bug.

People

Assignee:

Unassigned

Reporter:

Brandon Bloom

Vote (0)

Watch (1)

Dates

Created:

16/Aug/12 9:25 PM

Updated:

23/Sep/12 7:38 PM