| Ticket UUID: | 722192 | |||
| Title: | Use lsearch -exact in bwidget/tree.tcl | |||
| Type: | Bug | Version: | None | |
| Submitter: | jenglish | Created on: | 2003-04-16 00:24:09 | |
| Subsystem: | bwidget | Assigned To: | hobbs | |
| Priority: | 3 Low | Severity: | ||
| Status: | Closed | Last Modified: | 2003-05-15 06:54:20 | |
| Resolution: | Fixed | Closed By: | hobbs | |
| Closed on: | 2003-05-14 23:54:20 | |||
| Description: |
The attached patch is mechanical ly replaces all occurrences of [lsearch] in tree.tcl with [lsearch -exact]; this is to avoid problems in case node IDs contain "*" or other glob characters. I'm *pretty* sure that [lsearch -exact] is the right thing in all cases, since the Tree widget isn't documented to use glob-matching in any operation, but this ought to be double-checked before committing. | |||
| User Comments: |
hobbs added on 2003-05-15 06:54:20:
Logged In: YES user_id=72656 Commited in all uses of lsearch for bw1.6 jenglish added on 2003-04-22 03:11:50: Logged In: YES user_id=68433 > Have you checked that patterns are not allowed by the documentation? Yes. The only user-level commands where it _might_ make sense are [$w exists ...] and possibly [$w index ...], and the documentation doesn't mention glob patterns here. hobbs added on 2003-04-22 01:30:04: Logged In: YES user_id=72656 Have you checked that patterns are not allowed by the documentation? jenglish added on 2003-04-16 07:24:09: File Added - 47885: bwidget-tree-lsearch-exact.patch | |||
Attachments:
- bwidget-tree-lsearch-exact.patch [download] added by jenglish on 2003-04-16 07:24:09. [details]