Re: fixed-point support in c++

[Dave Korn <dave dot korn dot cygwin at googlemail dot com>]

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