Update higher-order-functions.md

[cclaudiu81]

enhanced doc on methods that take fns as args

[dwijnand]

Perhaps it's a good idea to express how easy it is to close over and leak internal state with closures, but my thinking is if we want to do this we should do it in a better place, perhaps in the introduction. Also, I think this could use some rewording (functions leak internal state, not "escape" the internal state).

[cclaudiu81]

you can do it in any place you want :) i just wanted to emphasize the idea that overusing closures when objects are used to represent the domain could produce some dangerous things... they can easily become antipatterns just like everything which is overused...In this context "leak internal state" i see it as a synonym for "escape internal state"...but again if you consider "leak internal state" is more appropriate then go ahead and use it.

[SethTisue]

@cclaudiu81 there isn't anyone who's job it is to take pull requests and put them into final form. if you're interested in responding to the review feedback yourself, great. if not, we'll close the PR, while inviting other volunteers to submit their own versions if they like.

[cclaudiu81]

@SethTisue will add this information as PR changes. thanks!

[dwijnand]

Thanks!