() Stefan Monnier () Tue, 29 Apr 2014 01:39:39 -0400 > + return 1; > + return 0; Please make the return type "bool", then. And use "true" and "false" rather than 1 and 0. Also, you can apply eta-reduction to the above code and just write return (XftCharExists (xftfont_info->display, xftfont_info->xftfont, 0x4E00) || XftCharExists (xftfont_info->display, xftfont_info->xftfont, 0xAC00)); [ Tho that would step over the 80 columns limit, so you may then want to introduce a local var to hold xftfont_info->display, maybe. ] Another wrinkle is func ‘XftCharExists’ returns ‘FcBool’, so to be excruciatingly correct, we would want to compare each call's rv against ‘FcTrue’, something like: