On 22/12/13 21:21, Behdad Esfahbod wrote: > + if (!(frac_mask | numr_mask | dnom_mask)) > + return; Maybe it would be better to require *both* numr and dnom to be present if they're to be used. Something more like if (!frac_mask && (!numr_mask && !dnom_mask)) return; perhaps? JK