Zsh Mailing List Archive
Messages sorted by: Reverse Date, Date, Thread, Author

List of patches to review (mainly for named reference)



In my opinion, the highest priority should be to submit the patch that simplifies resolve_nameref and setscope (workers/53740) as this will ease other changes.

The following patches are 4 prerequisites, where each depends on the previous one:

Fix loading of autoload variables accessed via references. - workers/53781
Don't follow namerefs when computing base scope of nameref (part 1). - workers/53782
Don't follow namerefs when computing base scope of nameref (part 2). - workers/53782
Only ever compute a base scope for non-upper references. - workers/53782

Another orthogonal prerequisite (it eliminates the "BUG: local parameter is not unset" debug messages):

- Fix type conversions via assignments to references. - workers/53784
  (The code is ready but I should still add tests.)

Two other likely prerequisites (I'm not 100% sure they are real prerequisites):

Remove bogus self reference warning. - workers/53739
- Fix imprecise invalid reference warning. - workers/53739

One last likely prerequisite:

Remove invalid upscope from createparam. - workers/53676
  You had concerns about the scope in which assignments to not-yet-defined references should create a new variable, which currently is the global scope. A related issue is how "typeset <options but no -n> ref=..." statements behave when "ref" is a reference and no "-n" option is included. I noticed that such statements exhibit inconsistencies and outright bugs. I will eventually share something about that. However, all these issues are orthogonal to what this patch fixes. So I think it could be submitted and we can resolve these issues separately.

Something completely unrelated but that I think you already reviewed and only needs to be submitted:

Don't allow exec in always blocks. - workers/53732
  
Philippe



Messages sorted by: Reverse Date, Date, Thread, Author