search

serge-sans-paille / frozen

a header-only, constexpr alternative to gperf for C++14 users
Apache License 2.0
1.27k stars 103 forks source link

minor fixes and improvements #158

Closed muggenhor closed 11 months ago

muggenhor commented 1 year ago

This is a set of fixes and minor refactors of issues that I encountered while trying to get this library to produce zero load-time relocations when compiling a container storing strings with -fPIC.

I thought it better to extract these to this separate PR, as they're useful regardless, before creating a massive PR for that.

FYI: I'm not fully happy yet with my changes for zero-reloc string containers so won't be creating an immediate follow-up PR for that just yet. But if you want to have a look at that, I'm keeping those changes in my feat/zero-reloc-string-containers branch.

muggenhor commented 11 months ago

Please let me know if there's any other changes needed before merging this. Or you'd like some of these changes split off from this PR. (I've purposefully kept these changes in a bunch of separate commits).

serge-sans-paille commented 11 months ago

Thanks a lot for both your patience and the great code base improvement!