On Mon, Sep 24 2012, Austin Clements <[email protected]> wrote: > Quoth Justus Winter on Sep 24 at 5:21 pm: >> Fix the COERCE_STATUS macro to handle _internal_error being declared >> as void function. >> >> Note that the function _internal_error does not return. Evaluating to >> NOTMUCH_STATUS_SUCCESS is done purely to appease the compiler. >> >> Signed-off-by: Justus Winter <[email protected]> >> --- >> lib/notmuch-private.h | 10 +++++++--- >> 1 file changed, 7 insertions(+), 3 deletions(-) >> >> diff --git a/lib/notmuch-private.h b/lib/notmuch-private.h >> index bfb4111..7a409f5 100644 >> --- a/lib/notmuch-private.h >> +++ b/lib/notmuch-private.h >> @@ -136,13 +136,17 @@ typedef enum _notmuch_private_status { >> * to or greater than NOTMUCH_STATUS_LAST_STATUS. (The idea here is >> * that the caller has previously handled any expected >> * notmuch_private_status_t values.) >> + * >> + * Note that the function _internal_error does not return. Evaluating >> + * to NOTMUCH_STATUS_SUCCESS is done purely to appease the compiler. >> */ >> #define COERCE_STATUS(private_status, format, ...) \ >> ((private_status >= (notmuch_private_status_t) >> NOTMUCH_STATUS_LAST_STATUS)\ >> ? >> \ >> - (notmuch_status_t) _internal_error (format " (%s).\n", \ >> - ##__VA_ARGS__, >> \ >> - __location__) >> \ >> + _internal_error (format " (%s).\n", \ >> + ##__VA_ARGS__, >> \ >> + __location__), >> \ >> + (notmuch_status_t) NOTMUCH_PRIVATE_STATUS_SUCCESS >> \ > > Just a nit: why not simply NOTMUCH_STATUS_SUCCESS? > > Otherwise, this series LGTM. No need to roll another version just for > this comment.
I agree. LGTM. > >> : >> \ >> (notmuch_status_t) private_status) _______________________________________________ notmuch mailing list [email protected] http://notmuchmail.org/mailman/listinfo/notmuch
