>  */
>  GEANY_API_SYMBOL
>  void tm_source_file_free(TMSourceFile *source_file)
>  {
> -     if (NULL != source_file)

I generally agree with you. IMO for API funtions, erroneous calls should be 
catched by something that isn't easily compiled out like g_return_*. I just did 
that here because we do it like this for all other API functions.

Fine, I'll remove this part of the change and leave it for later audit.

---
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/839/files#r50097974

Reply via email to