b4n commented on this pull request.

Looks fine

@@ -383,6 +381,10 @@ GeanyDocument *document_get_current(void)

void document_init_doclist(void) { documents_array = g_ptr_array_new(); +#ifdef GEANY_DEBUG + // add a dummy invalid document to catch naive iteration + g_ptr_array_add(documents_array, g_new0(GeanyDocument, 1));

If I'm not mistaken you could simply use `new_doc_index()` now