Arne Welzel
Arne Welzel
Chatting with @rsmmr , one idea came up to prevent copying data for UDP/block analyzers: https://github.com/zeek/spicy/blob/26b3b791e3f9b34258e2b9494e17d5aaa70e7120/spicy/runtime/src/driver.cc#L259-L281 If any iterators into the stream are invalidated after `parse1`, seems the stream would...
This is again a micro-benchmark, but I think an interesting observation. Relates to zeek/zeek#3379. When running `spicy-driver` for micro-benchmarking, [glibc is running in single-threaded mode](https://www.gnu.org/software/libc/manual/html_node/Single_002dThreaded.html), avoiding usage of atomic instructions...
Similar to #1592 (this is using the same netsted.spicy reproducer), patching the generated code to remove the default initialization of the std::tuple `__result` in the `__parse` functions results in significant...
Hope that's a valid test. I'm not sure why there's all the redundant `(*__self).__error = __error` assignments, but it doesn't come for free. I took the generated C++ code of...
Forgot to remove a `backtrack()` from a unit and eventually found a violation in the `analyzer.log` explaining the problem. Would think this is an error that should make it into...
See discussion below - I'm uncertain if this is a Spicy or Zeek issue, putting it in Spicy for now. https://github.com/zeek/zeek/pull/3277#discussion_r1316876424_ Chatting with @bbannier in referenced PR, it would be...
The `zeek::confirm_protocol()` method is documented as "deprecated". Compiling a `.spicy` file with a hook that calls `zeek::confirm_protocol()` does not trigger a deprecation warning or some form of notification to the...
When using a unit type for `%context`, the unit's `%init` hook is not invoked. Ran into it when attempting to instantiate an analyzer from within a context's `%init` to have...
Not sure this is suitable for tracking in zeek/spicy, but seem to trigger in `type_erase.h`, so creating it here. When compiling Zeek with `./configure ... --sanitizers=undefined,address`, compiling fails during compilation...
@bbannier suggested to open a ticket. Probably all false positives, but can't compile with -Werror (not a requirement, just mentioning this). ``` $ gcc --version gcc (Debian 10.2.1-6) 10.2.1 20210110...