the libc free() (man 3 free) allows free'ing NULL pointers. I think the quality
of the codebase is improved if we stick to this standard.
Your suggestion is excellent. We definitely have to remove all `if (buf)` tests!
---
Reply to this email directly or view it on GitHub:
https://github.com/Ope
Shouldn't you fix the caller that pkg_free's without checking for NULL instead?
If you "allow" it, then you might as well drop all the `if(buf)` before every
`pkg_free(buf)`. Allowing both makes the code inconsistent.
Right?
---
Reply to this email directly or view it on GitHub:
https://github.