Tony Morris

@ryanlecompte @jedws mapValues is only available on Map. It’s also VERY broken. Don’t use it if you care about performance (or correctness).

@djspiewak @ryanlecompte what correctness issues do you have? I can see that the laziness may be a prob if recomputing values all the time…

@jedws @ryanlecompte Recomputation *is* a correctness issue in an impure language.

@djspiewak @ryanlecompte oh, the dickhead problem… solved by a large cricket bat!

@jedws @ryanlecompte lol Side effects happen.

@djspiewak @ryanlecompte indeed. but it is somewhat pessimistic though… any function in any impure language has the same problem

@jedws @ryanlecompte Yes, but mapValues is not expected to be lazy, and thus the fact that it’s a function is unexpected.

@djspiewak @ryanlecompte why do you not expect it to be lazy? why does it matter? (in the sense that a program should not care)