Benjamin Bannier

Results 194 comments of Benjamin Bannier

Thanks for taking a look @HerringtonDarkholme. I agree that it is weird how this is not provided by `tree-sitter`, but only `tree-sitter-cli`. The relevant functions are however `pub`, so one...

A naive fix here would be to pass a deterministic time to `mtime`, e.g., the start of the epoch, ```cpp auto mtime = hilti::expression::Ctor(hilti::ctor::Time(hilti::rt::Time())); ``` The bigger question is what...

To clarify, I meant live leaks here where handles (still valid) stick around consuming resources without being used. An example would be an analyzer parsing long-running connection; for each new...

One way to implement this would be Spicy functions in the `zeek` module which get Zeek values (maybe returning an `optional`). In order to make type checking work we might...

> I think just about all the common used ones have the prefix: > > - ldap > ... We'll remove the `spicy_` prefix when moving spicy-ldap into Zeek proper...

Just as a comment, in practice such issues might be easier to debug with a tree-sitter-based grammar since tree-sitter supports [slightly more advanced error recovery](https://github.com/tree-sitter/tree-sitter/issues/224#issuecomment-436731626). For this code `zeek-script` reports:...

> The underlying issue is that `@load` is really like `#include` in C and not like `import` in python. And all this seems to happen already in the parser, so...

> Unless we need the new features, I am tempted to leave things as they are. While the python versions are EOL - they are still maintained by the distributions....