<div dir="ltr">LGTM for the matcher patch.</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Feb 8, 2013 at 10:01 PM, Philip Dunstan <span dir="ltr"><<a href="mailto:phil@philipdunstan.com" target="_blank">phil@philipdunstan.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div>Hello</div><div><br></div><div><div style="font-family:arial,sans-serif;font-size:12.800000190734863px">
These patches continue some work that I started back in July (before RL kicked in) to create a C++11 migration tool that would add the override specifier to suitable member functions where it could. I have reworked the tool to integrate it into the cpp11-migrate tool in the tool/extra repository.<br>


</div><div style="font-family:arial,sans-serif;font-size:12.800000190734863px"><br></div><div style="font-family:arial,sans-serif;font-size:12.800000190734863px">There are two patches - one on the clang repository with some AST matchers and the other on the tool/extra repo.</div>


<div style="font-family:arial,sans-serif;font-size:12.800000190734863px"><br></div><div style="font-family:arial,sans-serif;font-size:12.800000190734863px">add-override-clang_174296.patch<br></div><div style="font-family:arial,sans-serif;font-size:12.800000190734863px">


Adds ASTMatchers isVirtual and isOverride </div><div style="font-family:arial,sans-serif;font-size:12.800000190734863px"><br></div><div style="font-family:arial,sans-serif;font-size:12.800000190734863px">add-override-clang_tools_extra_174296.patch<br>


</div><div style="font-family:arial,sans-serif;font-size:12.800000190734863px">Adds the -add-override refactor to the cpp11-migrate tool</div></div><br clear="all"><div>Phil<br>--<br>Philip Dunstan<br><a href="mailto:phil@philipdunstan.com" target="_blank">phil@philipdunstan.com</a><br>


<a href="http://www.philipdunstan.com" target="_blank">www.philipdunstan.com</a></div>
</div>
</blockquote></div><br></div>