<div dir="ltr">No worries, I wouldn't spend much time rewriting old tests.  I'd write new tests this way, though.</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Apr 18, 2014 at 11:47 AM, Timur Iskhodzhanov <span dir="ltr"><<a href="mailto:timurrrr@google.com" target="_blank">timurrrr@google.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">OK, I'll continue this boring task next week then :D<br>
<br>
2014-04-18 22:44 GMT+04:00 Reid Kleckner <<a href="mailto:rnk@google.com">rnk@google.com</a>>:<br>
<div class="HOEnZb"><div class="h5">> On Fri, Apr 18, 2014 at 8:17 AM, Timur Iskhodzhanov <<a href="mailto:timurrrr@google.com">timurrrr@google.com</a>><br>
> wrote:<br>
>><br>
>> 2014-04-18 1:38 GMT+04:00 Reid Kleckner <<a href="mailto:rnk@google.com">rnk@google.com</a>>:<br>
>> > ================<br>
>> > Comment at:<br>
>> > test/CodeGenCXX/microsoft-abi-vtables-virtual-inheritance.cpp:459<br>
>> > @@ +458,3 @@<br>
>> > +namespace Test12 {<br>
>> > +struct X : B, A { };<br>
>> > +<br>
>> > ----------------<br>
>> > Going forward, I'd really prefer it if our inheritance hierarchy tests<br>
>> > could all be standalone.  See the microsoft-abi-vbtables.cpp test cases,<br>
>> > where the contents of any namespace can be cut-and-paste into a standalone<br>
>> > .cpp file and compiled.<br>
>> ><br>
>> > In this test, I have to go lookup ::A and ::B in the right namespace to<br>
>> > figure out what kind of record this is going to be.  It's worth a few lines<br>
>> > of duplicated test code to avoid that.<br>
>><br>
>> Like r206614?<br>
><br>
><br>
> Yep, thanks!<br>
</div></div></blockquote></div><br></div>