[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