Open msterin opened 3 years ago
Glad it has been useful.
The code change looks good. I will probably rework it a bit before integrating it in the next release (probably combining common elements with GREATEST_ASSERT_EQ_FMTm
), but that's because of my self-imposed line of code limit, which isn't your problem. :)
Glad it has been useful.
The code change looks good. I will probably rework it a bit before integrating it in the next release (probably combining common elements with
GREATEST_ASSERT_EQ_FMTm
), but that's because of my self-imposed line of code limit, which isn't your problem. :)
awesome, thanks.
There are nice new macros ASSERT_NEQ and ASSERT_NEQm in 1.15. This PR adds missing FMT macros for NEQ comparison.
FWIW we are using this in our test suite (including support similar to NEQ) and it's a very nice tool - thanks to the author and everyone contributing.
I did not find any special requirements for PR (other that to submit to
develop
branch), so submitting as is.Thanks !