On 16/03/2010 09:15, Sean D'Epagnier wrote:
> I went through the c++ parser and added support for fixed point there.
> Everything seems to be working, and I am able to use fixed-point
> numbers in c++.
>
> The c++ parser is kind of complex and it is possible I missed
> something. I would love to get feedback on this patch, and hopefully
> it can get committed to gcc.
Do you have a copyright assignment on file for GCC? For a non-trivial
contribution of this size, it's a requirement before the patch can be accepted.
Once that is established, your patch would need a changelog entry in the
standard format, and you need to have built the compiler both patched and
clean and run a full C and C++ testsuite on both to check for regressions.
Oh, and sending it to the -patches list is the best way to make clear that
it's ready for review, once we've got the rest sorted out.
cheers,
DaveK
