[squeak-dev] Decompiler reordering temps

Frank Shearar frank.shearar at gmail.com
Sun Jun 23 09:09:22 UTC 2013


On 23 June 2013 01:04, David T. Lewis <lewis at mail.msen.com> wrote:
> On Sat, Jun 22, 2013 at 11:38:46PM +0100, Frank Shearar wrote:
>> I happened to have Metacello loaded when I ran the DecompilerTests. A
>> method (MetacelloCommonMCSpecLoader >>
>> #retryingResolvePackageSpecReferences:gofer:) has these locals
>> declared:
>>
>> | references repositoryString retryCount repositoryError |
>>
>> and when decompiled the list looks like this:
>>
>> | references retryCount repositoryError repositoryString |
>>
>
> What's wrong with that?

It breaks a test. The Decompiler tests imply that foo's source and
foo's decompiled source are identical. This is another case where
that's not true.

So if you like, there's nothing wrong with it, and it's the test
that's broken (by being too strict). Either way there's a problem.

frank

> Dave
>
>


More information about the Squeak-dev mailing list