From: Carl Worth Date: Mon, 19 Oct 2009 20:09:19 +0000 (-0700) Subject: date.c: Remove all occurrences of g_return_val_if_fail X-Git-Tag: 0.1~835 X-Git-Url: https://git.notmuchmail.org/git?p=notmuch;a=commitdiff_plain;h=dbadca9a63ce9e97dc2d80fb0fee7ba2f8640316;hp=dbadca9a63ce9e97dc2d80fb0fee7ba2f8640316;ds=sidebyside date.c: Remove all occurrences of g_return_val_if_fail That's got to be one of the hardest macro names to read, ever, (it's phrased with an implicit negative in the condition, rather than something simple like "assert"). Plus, it's evil, since it's a macro with a return in it. And finally, it's actually *longer* than just typing "if" and "return". So what's the point of this ugly idiom? ---