Skip to content

SomeAsync and value replacement #144

Open

Description

@treeowl

cancelMany :: [Async a] -> IO () is ... okay, but it's annoying when cancelling Asyncs with multiple types. Some options:

-- Not very nice. Allocates wrappers unnecessarily.
data SomeAsync = forall a. SomeAsync !(Async a)
cancelSomeAsyncs :: [SomeAsync] -> IO ()

-- Is a `some` dependency okay, or copying stuff from there?
cancelSomeAsyncs :: [Some Async] -> IO ()

-- Potentially expensive if there's a lot of `fmap`ping and not a lot of inlining.
cancelAsyncs [() <$ thisOne, () <$ thatOne]

-- Can we make <$ cheaper? Yes, but it makes `Async` bigger. Any we need
-- to unsafeCoerce with Any to allow Async to unbox (ugh).
data Async a = Async !ThreadId !(TMVar (Either SomeException Any)) (Any -> a)
a <$ Async tid mv fun = Async tid mv (const a)

Nothing I'd call a beautiful solution.

Activity

  1. treeowl commented on Mar 3, 2023

    @treeowl
    ContributorAuthor

    Oh yeah, there's also the HListy option:

    data Asyncs :: [Type] -> Type where
      Nil :: Asyncs '[]
      (:<) :: {-# UNPACK #-} (Async a) -> Asyncs as -> Asyncs (a ': as)

    But a custom heterogeneous list type doesn't seem appropriate for this package.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions