Austin Clements <[email protected]> writes: > + message->flags &= ~(1 << NOTMUCH_MESSAGE_FLAG_GHOST);
What do you think about using bit set / clear / read macros? I don't insist, but I wonder if it would make this part more readable. > + else if (*i == "Tghost") > + message->flags |= (1 << NOTMUCH_MESSAGE_FLAG_GHOST); > + else It makes me faintly unhappy to have the prefix hardcoded here. Not sure if there is a sensible solution. _______________________________________________ notmuch mailing list [email protected] http://notmuchmail.org/mailman/listinfo/notmuch
