Added NDEBUG flag for Windows builds to strip assertion checking

CoreyKaylor / Lightning.NET

.NET library for LMDB key-value store

Other

397 stars 82 forks source link

Added NDEBUG flag for Windows builds to strip assertion checking #172

Closed marcotod1410 closed 1 month ago

marcotod1410 commented 1 month ago

Addresses #171

CoreyKaylor commented 1 month ago

Thanks for this. I'll be making a few more updates to the script, targeting 0.9.33, and incorporating the feedback from https://github.com/CoreyKaylor/Lightning.NET/issues/168. Then I'll publish a new version to nuget. Again, thanks for taking the time to report and send a PR.

CoreyKaylor commented 3 weeks ago

Looks like the -DNDEBUG on the AutoResize test fails. I did a quick glance and didn't see any of that behavior behind debug symbols, but I didn't spend much time either.