bugText Extractor TXR - Bugs: bug #35010, Code-walking logic in @(repeat)...

 
 

bug #35010: Code-walking logic in @(repeat) that looks for vars is inadequate now.

Submitted by:  Kaz Kylheku <kkylheku>
Submitted on:  Thu 08 Dec 2011 12:48:02 AM UTC  
 
Category: NoneSeverity: 5 - Blocker
Item Group: CorrectnessStatus: Fixed
Privacy: PublicAssigned to: Kaz Kylheku <kkylheku>
Originator Name: Open/Closed: Closed
Component Version: NoneOperating System: None
Fixed Release: NonePlanned Release: None
Architecture: AllABI: All

Add a New Comment (Rich MarkupRich Markup):
   

You are not logged in

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

 

Thu 10 Oct 2013 05:27:04 AM UTC, comment #1:

I had forgotten about this old bug entry.

Recently I ran into this problem, and I put a language hack in to address it.

I didn't make @(repeat) smart enough to look into Lisp code and find free variables. There would likely be downsides to that.

Rather, @(repeat) now takes a :vars argument, which tells it that there exist variables it might not "see" which need to be iterated over anyway.

This approach works well enough that the bug can be closed.

Release 68 has the feature.

Kaz Kylheku <kkylheku>
Project AdministratorIn charge of this item.
Thu 08 Dec 2011 12:48:02 AM UTC, original submission:

Lisp forms can be evaluated in @(output), which breaks under @(repeat) and @(rep) because these directives walk the code looking for variables, and pass down bindings only for the variables that are found. But Lisp code can refer to pattern variables. The code walking does not see these references inside the Lisp code.

Kaz Kylheku <kkylheku>
Project AdministratorIn charge of this item.

 

(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 kkylheku (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 2 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Thu 10 Oct 2013 05:27:04 AM UTCkkylhekuStatusConfirmed=>Fixed
      Open/ClosedOpen=>Closed

    Back to the top


    Powered by Savane 3.1-cleanup