Thanks, Frank. Fixed.<br><br><div class="gmail_quote">On Fri, Feb 8, 2013 at 12:08 PM, Frank Shearar <span dir="ltr"><<a href="mailto:frank.shearar@gmail.com" target="_blank">frank.shearar@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On 8 February 2013 19:34, <<a href="mailto:commits@source.squeak.org">commits@source.squeak.org</a>> wrote:<br>
> Eliot Miranda uploaded a new version of Compiler to project The Trunk:<br>
> <a href="http://source.squeak.org/trunk/Compiler-eem.248.mcz" target="_blank">http://source.squeak.org/trunk/Compiler-eem.248.mcz</a><br>
><br>
> ==================== Summary ====================<br>
><br>
> Name: Compiler-eem.248<br>
> Author: eem<br>
> Time: 8 February 2013, 11:33:58.616 am<br>
> UUID: c4da1b1b-b487-44bb-b8ef-392432ca421d<br>
> Ancestors: Compiler-cwp.247<br>
><br>
> Slightly less circuitous generation of methods with temp<br>
> names. Since schematic temp names string is computed<br>
> during generation, best to generate a method and copy<br>
> it with the computed temp names than generate two<br>
> separate methods.<br>
><br>
> =============== Diff against Compiler-cwp.247 ===============<br>
<br>
</div>It'd be great to automate reporting of test regressions. This (*)<br>
causes two regressions:<br>
<br>
Tests.Compiler.ClosureCompilerTest.testBlockDoitDecompilation [1]<br>
Tests.Compiler.ClosureCompilerTest.testDecompiledDoitMethodTempNames [2]<br>
<br>
[1] <a href="http://build.squeak.org/job/SqueakTrunk/160/testReport/junit/Tests.Compiler/ClosureCompilerTest/testBlockDoitDecompilation/" target="_blank">http://build.squeak.org/job/SqueakTrunk/160/testReport/junit/Tests.Compiler/ClosureCompilerTest/testBlockDoitDecompilation/</a><br>
[2] <a href="http://build.squeak.org/job/SqueakTrunk/160/testReport/junit/Tests.Compiler/ClosureCompilerTest/testDecompiledDoitMethodTempNames/" target="_blank">http://build.squeak.org/job/SqueakTrunk/160/testReport/junit/Tests.Compiler/ClosureCompilerTest/testDecompiledDoitMethodTempNames/</a><br>
<br>
frank<br>
<br>
(*) "This" really just being "this with high probability" because<br>
there's currently no indication in Jenkins as to how build N+1 differs<br>
source-wise from build N.<br>
<br>
</blockquote></div><br><br clear="all"><div><br></div>-- <br>best,<div>Eliot</div>