Re:Lightweight concurrency library for C, featuring symmetri...
by jdmt on Friday October 18, 2013 @02:40PM
(#6573181)
Failing immediately is certainly an easy way to handle unrecoverable errors. However, it seems as though your library assumes NDEBUG is never defined. Maybe you should make your own function rather than using assert()?
[ Parent
| on HN ]
Re:Lightweight concurrency library for C, featuring symmetri...
by halayli on Friday October 18, 2013 @02:54PM
(#6573281)
The asserts are in the .c files. Defining NDEBUG in your program won't change anything in lthread because the library was compiled without NDEBUG defined.
If you want to define NDEBUG when compiling lthread, that's up to you.
Re:Lightweight concurrency library for C, featuring symmetri...
by sebcat on Friday October 18, 2013 @03:24PM
(#6573469)
If you compile
assert(foo(bar) == 0);
with NDEBUG, foo will never even be called. I've seen some funky bugs from this in the past, like people calling malloc inside an assertion... Works great during dev, release build has NDEBUG defined, no test coverage... You get the picture.
A better idea would be to capture the return value in a var and do the assert on the var.
Re:Lightweight concurrency library for C, featuring symmetri...
by halayli on Friday October 18, 2013 @04:12PM
(#6573754)
Yes I understand that foo() won't be called if you defined NDEBUG. But when compiling lthread, NDEBUG is not defined anywhere in the code / makefiles so it's guaranteed to be called and I want to assert the returned value.