Added toFixed binding.

[Fresheyeball]

No description provided.

[paf31]

Ok, I'm going to be super picky here :)

Can we reorder the function arguments? From a partial application viewpoint, it seems more likely that you'd want to partially apply to the number of decimal places:

printThree x y z = 
  formatDecimal x ++ ", " 
  formatDecimal y ++ ", " 
  formatDecimal z
  where
  formatDecimal = toFixed 2

[Fresheyeball]

I'm pretty sure that's the current order of the arguments

[paf31]

Oh. I was confused because of the argument names. I assumed decimal meant "the decimal I'm formatting". Sorry :)

[paf31]

Looks good to me. @garyb any comments?

[garyb]

I'm not sure why, but we had toFixed once before, and then removed it: 00d2191. I'll have a think about why before re-merging.

[garyb]

Sorry, forgot to get back to this... the problem is toFixed throws runtime exceptions if you call it with an argument < 0 or > 20, so ideally we should be using Eff or something to represent/handle that situation.

[Fresheyeball]

Actually, it sounds like having toFixed is the wrong approach for PureScript then. It might be better to put up some functions that round to a set number of decimals than use this JavaScript native method.