bugKawa - Bugs: bug #47651, Scope issue calling for-each from...

 
 

bug #47651: Scope issue calling for-each from a class method

Submitted by:  Tom Bousso <tbousso>
Submitted on:  Fri 08 Apr 2016 09:24:55 PM UTC  
 
Category: Code generationSeverity: 3 - Normal
Item Group: NoneStatus: Fixed
Privacy: PublicAssigned to: Per Bothner <bothner>
Open/Closed: Closed

Add a New Comment(Rich Markup)
   

You are not logged in

Please log in, so followups can be emailed to you.

 

Fri 15 Apr 2016 01:05:40 AM UTC, comment #2:

I checked in a fix - see gnu/expr/ChangeLog.

I also checked in your testcase - see testsuite/sva47651.scm.
Thanks!

This change ended up both removing some redundant code, and causing slightly better code to be generated in at least one test-case.

Per Bothner <bothner>
Project AdministratorIn charge of this item.
Sun 10 Apr 2016 05:53:50 AM UTC, comment #1:

hanks for the clean test-case.
I have confirmed that it fails for me.
I'll try to figure out what is going on tomorrow.

Per Bothner <bothner>
Project AdministratorIn charge of this item.
Fri 08 Apr 2016 09:24:55 PM UTC, original submission:

That code results in an error: test.scm:6:5: internal error: missing LambdaExp:print/5/l:11

It seems like the lambda doesn't capture the object's scope correctly. If I create my own for-each in Scheme or change it to ((this):print n) then it works.

Tom Bousso <tbousso>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach File(s):
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by bothner (Posted a comment)
  • -unavailable- added by tbousso (Submitted the item)
  •  

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follow 4 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Sat 23 Apr 2016 05:15:30 AM UTCbothnerOpen/ClosedOpen=>Closed
    Fri 15 Apr 2016 01:05:40 AM UTCbothnerStatusConfirmed=>Fixed
    Sun 10 Apr 2016 05:53:50 AM UTCbothnerStatusNone=>Confirmed
      Assigned toNone=>bothner

    Back to the top


    Powered by Savane 3.1-cleanup1