PATCH RFA: Avoid enum bitfield in C++ frontend
Mark Mitchell
mark@codesourcery.com
Fri Apr 17 03:40:00 GMT 2009
Jason Merrill wrote:
> Ian Lance Taylor wrote:
>> This is how I think the code looks best--I don't see a need for a
>> special type to hold this combination of bitfields. However, it would
>> be a smaller change to let the C++ frontend continue to use
>> tsubst_flags_t, and simply do "typedef int tsubst_flags_t". If the C++
>> frontend maintainers would prefer that patch, I can do that instead.
>
> I think I would prefer that; the typedef provides documentation about
> what the complain parameter means.
FWIW, I agree. I'd rather we used typedefs for basic types more in GCC,
rather than less; it's an easy way to get some self-documentation in the
code. "a_register r" is a lot easier to understand than "int r".
My two cents,
--
Mark Mitchell
CodeSourcery
mark@codesourcery.com
(650) 331-3385 x713
More information about the Gcc-patches
mailing list