Ticket UUID: | 553773 | |||
Title: | calculus: define and doc scope of funcs | |||
Type: | Bug | Version: | None | |
Submitter: | dgp | Created on: | 2002-05-08 16:26:45 | |
Subsystem: | math :: calculus | Assigned To: | andreas_kupries | |
Priority: | 5 Medium | Severity: | ||
Status: | Closed | Last Modified: | 2002-06-04 03:02:00 | |
Resolution: | Fixed | Closed By: | andreas_kupries | |
Closed on: | 2002-06-03 20:02:00 | |||
Description: |
The commands math::calculus::* take "func" arguments, Tcl commands that accept and return numeric values. The scope for these commands needs to be defined. Currently, the command name passed in is just called within the procs, effectively seeking the commands in the ::math::calculus and :: namespaces. Reasonable alternatives would be to look in only ::, or in only the caller's context. I think I favor using the caller's context via [uplevel]. Whatever choice is made, it should be documented. | |||
User Comments: |
andreas_kupries added on 2002-06-04 03:02:00:
Logged In: YES user_id=75003 Took the chunk patching calculus.test and applied it. Checked for differences between the whole patch and my patch, applied some other changes in Don's patch. Committed to head. dgp added on 2002-06-01 04:01:51: File Added - 24162: calc.patch Logged In: YES user_id=80530 I took the tarball of updated files, made a few corrections and created the attached patch against the HEAD. andreas_kupries added on 2002-05-30 05:08:40: Logged In: YES user_id=75003 I am working on that. dgp added on 2002-05-29 23:43:02: Logged In: YES user_id=80530 I do not see the changes in CVS? arjenmarkus added on 2002-05-28 18:21:25: File Added - 23868: calculus.tar.gz Logged In: YES user_id=400048 I have updated the documentation, the code and the test script to get proper behaviour in different namespaces arjenmarkus added on 2002-05-15 13:54:04: Logged In: YES user_id=400048 I will fix the code to make sure it is looking at the caller's context via [uplevel]. And documentation will state this to be the case. This has, at least, been my intent from the start (just was not aware that this could be a problem). The test cases will need to be adjusted to make sure that the lookup occurs as intended. |