Remove unnecessary null pointer checks

[elfring]

An extra null pointer check is not needed in functions like the following.

• bytechunk_free
• _logrecord_free

How do you think about to apply a corresponding update suggestion which will eventually be generated by the software "Coccinelle"?

@Deletion@
expression x;
@@
-if (\(x != 0 \| x != NULL\))
    g_free(x);

[pastly]

If you have the time to open issues for trivial-to-fix non-issues, please also find the time to submit pull requests to solve them.

[elfring]

I find it unlikely at the moment that I am going to contribute pull requests for this software because I do not like specific properties of the corresponding licence.

[robgjansen]

Thanks for pointing this out. We plan to remove glib as a dependency soon, in which case the NULL checks may become useful. We'll consider this as we clean up our code in future releases.

[elfring]

I suggest to consider additional development tools and software design approaches so that more unnecessary checks can be avoided.

[stevenengler]

Closing since we're moving from C to rust.