Can't build with clang, error: "jthread is supported now, remove custom definition"

[FabioLolix]

Can't build with clang, error: "jthread is supported now, remove custom definition"

[heinezen]

Ah looks like clang (or the stdlib it uses) finally supports jthread now.

[TheJJ]

Short term, you probably should patch out this error for the arch build for now, i.e. just remove the error preprocessor directive.
We added it so we can remove our custom jthread implementation once clang supports it, but I fear we have to keep it as long as we support clangs that don't have it yet...
To properly solve it, we have to remove the error too and remember deleting the custom jthread impl once we raise the minimum clang version 😎

[heinezen]

We already do the jthread C++ feature check, so if std::jthread is defined, it's always used. And it's not like our custom jthread takes a lot of space away in the code. We just have to remember to remove it when Apple finally updates their clang version in the next century ;)

[FabioLolix]

Builds fine thanks