Bugs for Sorcery are reported here

[bugzilla-daemon AT bugs.sourcemage.org]

http://bugs.sourcemage.org/show_bug.cgi?id=13458

           Summary: select_list should also mention variables from
                    select_list_sub or the latter should use more obscure
                    ones
           Product: Sorcery
           Version: 1.14.x
          Platform: Other
        OS/Version: other
            Status: NEW
          Severity: minor
          Priority: P4
         Component: Documentation
        AssignedTo: sm-sorcery-bugs AT lists.ibiblio.org
        ReportedBy: jakakranjc AT email.si


select_list has a nice warning in the bashdoc header:
@param return_var (must not be i, foo, temp, returnvar, stuff or default)

This is nice, but it forgot to mention the local variables that 
select_list_sub 
uses, which can't be used either. Those are: hashnum, msgstr, result and 
answer.
The last two are pretty common words, so I suggest that all the 
select_list_sub 
variables are prefixed with an underscore. I think that would be better than 
just updating the comments (they would need to be updated either way).

select_provider doesn't give any warnings at all, maybe it should.


navaden@lynxlynx wormux-svn $ . /etc/sorcery/config
navaden@lynxlynx wormux-svn $ select_list ANS 2 1 2 3 4 5
        (0)  1
        (1)  2
        (2)  3
        (3)  4
        (4)  5

Which one do you want? [1] 4
navaden@lynxlynx wormux-svn $ echo $ANS
5
navaden@lynxlynx wormux-svn $ echo $answer

navaden@lynxlynx wormux-svn $ select_list answer 2 1 2 3 4 5
        (0)  1
        (1)  2
        (2)  3
        (3)  4
        (4)  5

Which one do you want? [1] 4
navaden@lynxlynx wormux-svn $ echo $answer

navaden@lynxlynx wormux-svn $

-- 
Configure bugmail: http://bugs.sourcemage.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.