Feature request: Ability to use sigmatch without compile time literals using runtime variables instead about sigmatch

[SpriteOvO]

""_sig only accepts string from compile-time, it needs changes to support runtime signature parsing.

[MattMills]

OK, I've changed the title to represent a feature request instead, assuming you are OK with that... if not I can try to fork and see if I can manipulate the existing code to implement it myself, but to be honest your C++ is much more advanced than my understanding, I haven't learned many of these newer features.

[SpriteOvO]

Some hints:

1. Changing the consteval to constexpr for all function involved in _sig function, this makes them can be called at both runtime and compile-time.

2. During the first step, you need to figure out how to reimplement the macro SIGMATCH_CONSTEVAL_STATIC_ASSERT in order to report an error to caller for runtime parsing. (Probably have to use if constexpr and std::is_constant_evaluated)

3. Make a static member function for signature for runtime parsing, e.g.

   signature signature::parse(std::string_view) noexcept

Feel free to try it out if you wish, just make it work for yourself and then you can learn a lot from it ;)

I will try to officially implement it in this repo after I finish my other work.

[MattMills]

Thanks for the tips, I've messed with it for a while but I need to go learn more modern C++; I think I probably learned on C++98...

[MattMills]

@SpriteOvO if I can bother you for another hint, I've spent some more time on this and where I'm stuck right now is what the heck to do with consteval_str_buffer... since it takes size as a template arg and that needs to be a compile time constant; am I looking at this in the wrong way, or do I just need to add a second struct with a dynamically sized array?

[SpriteOvO]

do I just need to add a second struct with a dynamically sized array?

@MattMills No, constexpr can handle std::string_view well, so consteval_str_buffer is only needed for the entry consteval function parse_sig_str. For deeper functions, the NTTP consteval_str_buffer can be simply replaced with a normal parameter std::string_view.

Don't worry, I have started trying implement it in this repo officially.

[SpriteOvO]

Implemented in #5.