ClientCallable.call() signature

[zlangbert]

This is a bit of a nitpick but when call() is expanded it includes a call to Future.map which requires an execution context. My IDE kept deleting the import because it didn't know it needed it. Would it be possible to change the signature of call() to something like call(implicit executor: ExecutionContext)?

[lihaoyi]

Sure

[lihaoyi]

Probably call()(implicit executor: ExecutionContext) to be source-compatible, unless you want to remove the parens at the end of the call, which is probably unimportant

[zlangbert]

No you're right, that makes more sense. Thanks!

[caente]

On this note, is there a way to pass the ExecutionContext explicitly to call?

[lihaoyi]

Not unless we change the signature =P Which nobody has done yet...

[Daxten]

#57 closes this