laughinghan/mathquill — comment on commit d4ded2cd815d57d1d9c6272690c1366b6bc135b8
File:  line 0
By laughinghan:

a little weird for `map` to map a Range to an Array, no? Shouldn't it map a Range to a Range, if anything?

is there a reason you wanted a `map`? I've never wished for it or I would've written it, the only use I can think of for it is `MathBlock.prototype.latex = function(){ this.map(function(e){ return e.latex(); }).join('');`, but I think `fold` is actually more readable because `map` returning an Array is so weird.

the `return false` is like `break` is following jQuery; `each` doesn't have any other use for the return value of the function, so it's fine, but I see why you'd want to make it consistent with `map`. I feel like the most important point here is that we've never, and I don't foresee ever, actually having a reason to `return false` in an `each` iteration, but I get exceptions thrown inside `each` iteration due to `undefined` property access etc all the time, and this would make debugging those a good deal more annoying.