Closed kjvalencik closed 1 year ago
There's a pretty big footgun in debug_assert_eq!. When compiled out, in addition to the assertion being removed, the included expressions do not evaluate at all.
debug_assert_eq!
This fix moves the necessary scope closes outside of the assertion macro.
This bug was introduced in https://github.com/neon-bindings/neon/commit/e1743934a2cf1ece9b3730ddc50c9931b8360eaf and currently only exists in the 1.0 pre-release.
There's a pretty big footgun in
debug_assert_eq!
. When compiled out, in addition to the assertion being removed, the included expressions do not evaluate at all.This fix moves the necessary scope closes outside of the assertion macro.
This bug was introduced in https://github.com/neon-bindings/neon/commit/e1743934a2cf1ece9b3730ddc50c9931b8360eaf and currently only exists in the 1.0 pre-release.