[PATCH v6 06/14] reindex: drop all properties named with prefix "index."

David Bremner david at tethera.net
Thu Oct 19 04:52:46 PDT 2017


Daniel Kahn Gillmor <dkg at fifthhorseman.net> writes:

> +    ret = notmuch_message_remove_all_properties_with_prefix (message, "index.");
> +    if (ret) {
> +	INTERNAL_ERROR ("failed to remove index.* properties");
> +	goto DONE;
> +    }

1) INTERNAL_ERROR is fatal, so the goto is unneeded / confusing.

2) Is there no sensible return value here? The function already has an
error return path. I see there is one INTERNAL_ERROR there so maybe
similar reasoning applies and a TODO is enough.


More information about the notmuch mailing list