geany/geany — comment on commit f31a84869d45980a7ecb27f5a4d6c3588c547eee
File: src/document.c line 2372
By b4n:

IMO this one was right, although the whole `app->tm_workspace != NULL` check is quite pointless since there is *no* code path that can lead to this situation AFAICT.

So I'd say either remove the check or keep use `g_return_if_fail()` (which implicitly uses `G_UNLIKELY()`) or alike: it's not a situation we want to manage, I guess the check here is only defensive programming before dereferencing the pointer.