STInst regression in aa62468d8c092f368ebb98036802585d35c08dce

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
3 messages Options
Reply | Threaded
Open this post in threaded view
|

STInst regression in aa62468d8c092f368ebb98036802585d35c08dce

Holger Freyther
Hi Paolo,

the above commit broke the import of Squeak sourcecode. Do you still remember
the testcase you had for the Behavior>>#evaluate: fix? I could create unit
tests to make sure that we don't regress in any of the two ways.

holger

_______________________________________________
help-smalltalk mailing list
[hidden email]
http://lists.gnu.org/mailman/listinfo/help-smalltalk

parser.st (176 bytes) Download Attachment
Reply | Threaded
Open this post in threaded view
|

Re: STInst regression in aa62468d8c092f368ebb98036802585d35c08dce

Paolo Bonzini-2
On 02/01/2011 10:18 PM, Holger Hans Peter Freyther wrote:
> Hi Paolo,
>
> the above commit broke the import of Squeak sourcecode. Do you still remember
> the testcase you had for the Behavior>>#evaluate: fix? I could create unit
> tests to make sure that we don't regress in any of the two ways.

Yes, see

http://smalltalk.gnu.org/project/issue/527

Paolo

_______________________________________________
help-smalltalk mailing list
[hidden email]
http://lists.gnu.org/mailman/listinfo/help-smalltalk
Reply | Threaded
Open this post in threaded view
|

Re: STInst regression in aa62468d8c092f368ebb98036802585d35c08dce

Holger Freyther
On 02/02/2011 08:47 AM, Paolo Bonzini wrote:

>
> http://smalltalk.gnu.org/project/issue/527


Hi Paolo,

this makes the manual test of the Compiler work again, it then moves this and
the code from the above issue into an autotest. I have also added the snippet
for the Squeak code that is not parsable anymore.

Now I just need to fix the regression itself..

z.

_______________________________________________
help-smalltalk mailing list
[hidden email]
http://lists.gnu.org/mailman/listinfo/help-smalltalk

0001-stinst-Make-the-manual-test-work-again.patch (1K) Download Attachment
0002-stinst-Start-a-autotest-test-for-the-STInST-parser-a.patch (6K) Download Attachment