Francisco Giordano
Francisco Giordano
Hi @paulrberg, glad you've tried it. It's taking me longer than I wanted to finish this new version and work on the documentation including a migration guide, but it's pretty...
The feature hasn't been implemented. The codebase was entirely rewritten since the original issue was opened so the patches no longer apply. The tool is much more robust, though, so...
I still think this is a good idea, but there are no plans to do this. If someone wants to take it on I'm happy to guide through it.
Not directly, but you can do it by accessing the AST node of the event definition. Something like this should work: ``` {{#each ownEvents}} {{#each astNode.parameters.parameters}} {{#if indexed}} indexed {{/if}}...
Let's leave it open, it would be nice to expose this somehow.
What about a `ChildExt` trait implemented for `std::process::Child` that adds a `pid(&self) -> Pid` method? It's a real pity that we need to cast to obtain a `Pid` instance.
It looks like this is the code that's necessary to fix this: https://github.com/tldraw/tldraw/blob/9ba4f7cf2a25039df26aff0f8988e78e7b19e0d0/apps/examples/src/examples/static-assets/StaticAssetsExample.tsx
I don't see a way to fix this without a breaking change. Another possible fix would be to add a new internal function. But I think just adding a return...
I think it would be ok to leave the previous `_extractUint128` if we mark it as `memory-safe`, along with a comment warning that the caller must validate `pos`.
We could also partition the dictionary based on the first few bytes. This is a variation on option 3.