(keys (LazyMap. {:a (delay "B")})) will eagerly instantiate MapEntrys about potemkin HOT 4 CLOSED

Hmmm. This appears to be due to how clojure.lang.RT/keys is defined:

static public ISeq keys(Object coll){
    return APersistentMap.KeySeq.create(seq(coll));
}

It takes all the entries, just to get at the keys. I can make the val lookup on each entry lazy, though, which I think is the semantically "right" way to give ourselves indirection. Thanks for pointing this out, I had completely missed this.

from potemkin.

No wonder I couldn't figure it out. I was looking at trunk, where I think keys might have changed implementation. Stared at that for the longest time. Thanks.

Are you saying that the MapEntrys that come back from seq and entryAt should overload val and getValue to lazily call valAt?

from potemkin.

Yeah, that was my thought.

On Thu, Jul 10, 2014 at 12:38 PM, savagematt [email protected]
wrote:

No wonder I couldn't figure it out. I was looking at trunk, where I think
keys might have changed implementation. Stared at that for the longest
time. Thanks.

Are you saying that the MapEntrys that come back from seq and entryAt
should overload val and getValue to lazily call valAt?

—
Reply to this email directly or view it on GitHub
#23 (comment).

from potemkin.

Merged, sorry for the delay.

from potemkin.