<div dir="ltr">Hi Eric,<div><br></div><div>your change builds fine on VS 2010.</div><div><br></div><div>Thank you!</div><div><br></div><div>Thomas</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Nov 4, 2015 at 4:04 PM, Erik Helin <span dir="ltr"><<a href="mailto:erik.helin@oracle.com" target="_blank">erik.helin@oracle.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 2015-11-04, Thomas Schatzl wrote:<br>
> Hi,<br>
><br>
> On Wed, 2015-11-04 at 12:12 +0100, Volker Simonis wrote:<br>
> > Hi Erik,<br>
> ><br>
> > thanks a lot for your understanding :)<br>
> ><br>
> > Your suggestion is perfectly fine for me and we can live with both<br>
> > versions of the fix.<br>
><br>
>   I went for the change in the globalDefinitions* file.<br>
><br>
> New webrevs:<br>
> <a href="http://cr.openjdk.java.net/~tschatzl/8141134/webrev.1/" rel="noreferrer" target="_blank">http://cr.openjdk.java.net/~tschatzl/8141134/webrev.1/</a><br>
> <a href="http://cr.openjdk.java.net/~tschatzl/8141134/webrev.0_to_1/" rel="noreferrer" target="_blank">http://cr.openjdk.java.net/~tschatzl/8141134/webrev.0_to_1/</a><br>
<br>
</span>Looks good to me, Reviewed.<br>
<span class=""><br>
> It built fine on the official MS compiler, but I did not check on VS<br>
> 2010.<br>
<br>
</span>Volker, can you check if this compiles on VS 2010?<br>
<br>
Thanks,<br>
Erik<br>
<br>
> Thanks,<br>
>   Thomas<br>
><br>
><br>
</blockquote></div><br></div>