Consistent Interface parameter placement

[razcore-rad]

With this PR I'm just trying to align the placement of Interface in
the API, it's just cosmetic, but I find that having Interface either
at the beginning (this PR's option) or at the end is desirable.

[garyb]

I would suggesting putting Interface at the end consistently is the better option - it's the way we usually do it when binding to an effectful JS API.

The reason for that is it plays better with bind and traverse, since you can do things like foo =<< interface... I think for this API that probably isn't going to get much use, but I'd still suggest following it because it's the more common convention. All the web libraries use that convention - I'm not sure about the other node ones that have this kind of situation, but if they differ I'd say they should be changed similarly.

[garyb]

Hah, I see @thomashoneyman said otherwise in #21 (comment), but personally I still think it's right to put Interface at the end for the reasons I mentioned above.

[thomashoneyman]

@garyb I'm happy to defer :) I sometimes put these sorts of values as the first argument because it's easier to scan over them if they're in a consistent position (vs. depending on the number of arguments) and because you can partially-apply if you're going to be using the function several times (I think? It's been a while since I've done this). Given that the web libraries already use your suggested placement I think that's enough reason to stick with that, let alone your other reasons.

[razcore-rad]

I put it in the first position cause I was thinking about partial application as well, but I'm happy with either as long as it's consistent.

[thomashoneyman]

@razcore-art let's go with @garyb's suggestion!

[razcore-rad]

I've updated the PR, should be good to go.

[thomashoneyman]

This can use a bit less indentation with:

Suggested change

	setLineHandler (\s -> if s == "quit"
	then close interface
	else do
	log $ "You typed: " <> s
	prompt interface) interface
	interface # setLineHandler \s ->
	if s == "quit" then
	close interface
	else do
	log $ "You typed: " <> s
	prompt interface

[razcore-rad]

I updated the whitespace so it isn't like either of us! :P

[thomashoneyman]

I'd like a look from @garyb but otherwise 👍