[Feature Request]: IGpsManager.GetCurrentPosition() extension method needs a timeout parameter.

[JVanloofsvelt]

[Feature Request]: IGpsManager.GetCurrentPosition() extension method needs a timeout parameter.

[aritchie]

You don't need this with reactive extensions. You can chain .Timeout(TimeSpan.FromSeconds(5)).ToTask()

[JVanloofsvelt]

Hello aritchie. It is inside the implementation of this extension method that ToTask() is used, and there is no use of the Rx Timeout operator there.

While I can use another timeout operator on the observable that is returned from this method, like you suggest, the internals of the observable use this task returned from ToTask() that never completes , and hence doesn't stop the GPS manager or releases the semaphore .

See here for the implementation:
https://github.com/shinyorg/shiny/blob/master/src/Shiny.Locations/Extensions.cs

[aritchie]

There is a cancellation token that is watched on the semaphore and the totask listener.

This is just an extension method, you can write your own if you feel it isn't working.

[JVanloofsvelt]

Oh I hadn't noticed the cancellation token being passed to ToTask()! Makes sense then

Thanks