Michele Caini
Michele Caini
> Should be easy to adapt the dense map. I tried to implement this and it basically turned out that the underlying data structure (technically, a glorified sparse set) isn't...
This is finally available on branch `wip` where the static storage in the const `assure` is gone. 👍
Mmm any chance you can reproduce it? I don't know what's going on there otherwise. 😞
> Mmm any chance you can reproduce it? I don't know what's going on there otherwise. 😞 @stefanofiorentino ping 🙂 I'd like to merge this one upstream but I can't...
> I'm wondering, why we don't have USE_*SAN as github workflows? Testing a workflow for that on this branch. 👍
Cool @stefanofiorentino at least I can reproduce the error on the CI. Good catch. 👍
Oh, wait, I suspect the test is wrong actually. We're closing the handle after the loop. 🤦♂️ Gotta go now but I'll give it a try during the weekend if...
What error do you get @stefanofiorentino `Loop.walk`, a wrong counter?
Yeah @stefanofiorentino that's this line: ```cpp loop->resource(0, true); ``` If you run it from the command line, it causes your error. If you can confirm it, we can also drop...
Any other concerns? I'd make this the upstream version. It has indecently faster compile times too.