On Fri, Jan 16, 2015 at 21:43, Miod Vallat wrote: >> Floating point constants default to double precision and can be made >> single precision with a 'f' or 'F' suffix or long double precision >> with 'l' or 'L'. >> >> It turns out indent only knows about 'u'/'U' and 'l'/'L' suffixes and >> will add a space between floating point constants and the suffix if >> 'f' or 'F' is used which results in code that won't compile. > > Your diff will allow for `fl' suffixes. What about doing > seensfx |= 1 | 2; > to make sure a forthcoming `l' suffix gets rejected?
I'm not sure indent needs a precise parser. Isn't that what the compiler is for? As long as it doesn't *generate* incorrect code, I think it's fine. If for some reason the source file I pass to indent has 0.0fl in it, let it through; cc will tell me about it soon enough.