Benjamin Bannier

> Can we leave valid unicode alone? There might still be valid Unicode which we'd want to encode, e.g., Null `U+0000` is [valid unicode](https://unicodeplus.com/U+0000). As an external data point for...

There is an explicit customization point just two lines down with `SPICY_FLAGS`; if one wants debug builds of Spicy analyzer one can always set that to `-d`. I cannot point...

As far as I can tell `strptime` still does not support extracting timezone information. Unless I am missing something let's leave this open @Mohan-Dhawan.

Your patched code definitely looks better, but the question is how hard it is to generate that code in general (i.e., the responsible section in the parser builder might create...

Like we discussed offline, one approach of not dealing with the different code paths would be to trade work on running the default constructors of the tuple elements for default...

I think we had a few instances already where users attempted to use a unit-type as context. Units come with additional code for parsing and hooks which are execute during...

> Mainly spurred [by this snippet](https://github.com/fox-ds/spicy-quic/blob/cb19f52fbe6e8a029a951ddd2b9d16dfcea94124/analyzer/QUIC.spicy#L21-L26) that I happily tried to extend. Yes, this definitely should use a struct instead of a unit. This specific snippet seems to initialize all...

_No custom default constructor_ syntax ;)

> I think that this either should be supported, or raise a compilation error. Just silently not executing %init seems very unexpected to me. For custom data to pass as...

Thanks for this issue @awelzel, this is a useful microbenchmarking result. We use `std::shared_ptr` extensively in the runtime library, e.g., as control blocks in views and our safe iterators which...