adamritter / fastgron

High-performance JSON to GRON (greppable, flattened JSON) converter
MIT License
592 stars 10 forks source link

Did you forget to ‘#include <cstring>’? #9

Closed kseistrup closed 1 year ago

kseistrup commented 1 year ago

Does 035377cbbd380546b8cf4acef18377a15a22d07c build?

On my ArchLinux box I get a lot of errors about memcmp and memcpy with messages like:

…/fastgron/src/growing_string.hpp:25:16: error: ‘memcmp’ was not declared in this scope
…/fastgron/src/growing_string.hpp:5:1: note: ‘memcmp’ is defined in header ‘<cstring>’; did you forget to ‘#include <cstring>’?
CXXFLAGS=-march=x86-64 -mtune=native -O2 -pipe -flto -fno-plt -fexceptions -Wp,-D_FORTIFY_SOURCE=2 -Wformat -Werror=format-security -fstack-clash-protection -fcf-protection -Wp,-D_GLIBCXX_ASSERTIONS
LDFLAGS=-Wl,-O1,--sort-common,--as-needed,-z,relro,-z,now
adamritter commented 1 year ago

Thanks, I just pushed this change recently, maybe I shouldn't directly work on main branch. I pushed a fix