On Mon Apr 3 08:53:31 2023 +0000, Mohamad Al-Jaf wrote:
Yeah, I realized that after pushing and was going to add a comment. Though, I'm curious why there's no error checking in the helper functions. Wouldn't it be better to check if the parameter is NULL there?
It was never needed, so you can then say that it's an error to call it with a `NULL` parameter. I think it makes sense to make the check here, as the base class is optional.