paleolimbot commented on PR #421: URL: https://github.com/apache/arrow-nanoarrow/pull/421#issuecomment-2046331319
> To get this added to meson all we would have to do is add werror=true to the project default_options configuration in the top level meson.build file Nice! I'm not sure what the best balance is...it's good to catch errors in nanoarrow code but we also don't want to fail if somebody can't install a specific version of Arrow or Google Benchmark (which I think is responsible for one of the unused warnings). We also don't want to fail on a release build in case some new compiler or some very old compiler finds something we didn't think of. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
