Skip to content

Propagate Duration from request_timer to the backends #848

Open
@luleyleo

Description

@luleyleo

In #847 the api of request_timer was changed to work with Duration instead of Instant.
This was done quickly in order to make the CI work again, by changing request_timer to

pub fn request_timer(&self, deadline: Duration) -> TimerToken {
    self.0.request_timer(instant::Instant::now() + deadline)
}

Which means the back ends were not affected by this.
As @xStrom mentioned on Zulip:

I quickly checked and all platform native functions seem to take duration. Which makes me wonder why druid ended up using Instant at all.

So it would make sens to handle the Duration properly in each backend.

Metadata

Metadata

Assignees

No one assigned

    Labels

    maintenancecleans up code or docsshellconcerns the shell abstraction

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions