use EMPTY_STRING in _parent_via_in_reply_to

This is a review suggestion [1] of Tomi. I decided not to squash it
so that the code movement remains clear.

[1]: id:m2pnxxgf5q.fsf@guru.guru-group.fi
This commit is contained in:
David Bremner 2018-09-03 12:26:10 -03:00
parent b9d4eb0412
commit ac2146118b

View file

@ -402,7 +402,7 @@ _parent_via_in_reply_to (notmuch_thread_t *thread, notmuch_message_t *message) {
THREAD_DEBUG("checking message = %s in_reply_to=%s\n", THREAD_DEBUG("checking message = %s in_reply_to=%s\n",
notmuch_message_get_message_id (message), in_reply_to); notmuch_message_get_message_id (message), in_reply_to);
if (in_reply_to && strlen (in_reply_to) && if (in_reply_to && (! EMPTY_STRING(in_reply_to)) &&
g_hash_table_lookup_extended (thread->message_hash, g_hash_table_lookup_extended (thread->message_hash,
in_reply_to, NULL, in_reply_to, NULL,
(void **) &parent)) { (void **) &parent)) {