Timeout Overloads

[Aidan63]

Currently you can only timeout a lambda node, but it might be useful to for timeout to also work on IStartableCoroTask (I assume this would start the task, not return a new IStartableCoroTask which has a timeout which starts once its started).

Since we lack real overloads I think the best we can do is something along the lines of timeoutLambda and timeoutStartableTask. I don't know if there are any other overloads we would want but I'm guessing we'll also come across this issue with other functions we add. To prevent Coro becoming very bogged down might it be worth having a Timeout.hx file which has all these functions in it?
User would then have to do import hxcoro.Timeout.* to get all the functions, although module level fields might help here.

[Simn]

Haxe actually supports overloads if you make them extern inline, because in that case we can guarantee that the runtime doesn't need to be involved:

import hxcoro.ICoroTask;
import hxcoro.ds.Channel;
import haxe.Timer;
import hxcoro.CoroRun;
import hxcoro.Coro.*;
import hxcoro.NodeLambda;

class TimeoutApi {
	static public overload extern inline function timeout(ms:Int, lambda:NodeLambda<Any>) {
		trace("lambda timeout");
	}

	static public overload extern inline function timeout<T>(ms:Int, task:IStartableCoroTask<T>) {
		trace("task timeout");
	}
}

function main() {
	CoroRun.runScoped(node -> {
		final child = node.lazy(_ -> {});
		TimeoutApi.timeout(10, child); // task timeout
		TimeoutApi.timeout(10, _ -> {}); // lambda timeout
	});
}

But this doesn't work with @:coroutine functions yet, which is why I opened #119. Once this works we can utilize it to dispatch to the actual implementations from a set of overloads, which should be quite nice in the user-facing API.

[Aidan63]

I wasn't sure if we wanted to use those "compile time" overloads since iirc you can't do stuff like pass one into, say, a Int->NodeLambda<T>->T function argument.
Thinking about it I'm not sure how often you would want to do that with these though, they seem to be used like building blocks and not things you pass around. So it's probably a nice solution.