On 2023-07-22 at 16:06 -03, David Bremner <[email protected]> wrote: > My first thought was that this should be static, but maybe it doesn't > matter in C++; I see the other inline functions in that file are not > declared static.
True, I should have marked this function static. I did the same for the others. > I stumbled over the logic this code. I would prefer not to use assert > for this. In a few other places we call INTERNAL_ERROR from a default: > case. Thanks for the pointer, I removed the assert. _______________________________________________ notmuch mailing list -- [email protected] To unsubscribe send an email to [email protected]
