[cfe-dev] Adding -add-override functionality to cpp11-migrate tool

[Philip Dunstan]

I've attached patches that include this new test:



Phil
--
Philip Dunstan
phil at philipdunstan.com
www.philipdunstan.com


On Tue, Feb 5, 2013 at 7:23 PM, Dmitri Gribenko <gribozavr at gmail.com> wrote:

> On Tue, Feb 5, 2013 at 7:44 PM, Philip Dunstan <phil at philipdunstan.com>
> wrote:
> > I haven't modified yet how the insertion point for the override keyword
> is
> > calculated. Dmitri suggested FunctionTypeLoc::getRParenLoc() but I don't
> > think that this will work if there is a trailing const keyword or
> deferred
> > return type.
>
> Could you please add a test with a trailing return type anyway?
>
> class A {
> public:
>   virtual int h();
> };
>
> class B {
> public:
>   auto h() -> int;
> };
>
> Dmitri
>
> --
> main(i,j){for(i=2;;i++){for(j=2;j<i;j++){if(!(i%j)){j=0;break;}}if
> (j){printf("%d\n",i);}}} /*Dmitri Gribenko <gribozavr at gmail.com>*/
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-dev/attachments/20130205/4e9658fd/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: add-override.clang_tools_extra_174386.patch
Type: application/octet-stream
Size: 16671 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-dev/attachments/20130205/4e9658fd/attachment.obj>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: add-override.clang_174386.patch
Type: application/octet-stream
Size: 2369 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-dev/attachments/20130205/4e9658fd/attachment-0001.obj>