Type | Bug | Status | tested | Date | 21-Oct-2010 07:51 |
---|---|---|---|---|---|
Version | alpha 107 | Category | Native | Submitted by | BrianH |
Platform | All | Severity | minor | Priority | normal |
Summary | False positive for FIND of a specific module in a block |
---|---|
Description |
If you attempt to FIND a specific module in a block, it will find the first module in the block, not the specific module you are looking for. This is almost always a false positive. It should return the same module it is looking for (not an "equal" one, the same one). This currently breaks IMPORT of premade module! values, and DO of modules. I can add a workaround to the source for now. |
Example code |
>> find system/modules module [] [] == [make module! [ lib-base: handle! xtest: make function! [[][ foreach blk [ [xarg0] [xarg1 111] [xarg1 1.1] [xarg1 "test"] [xarg1 [1 2 3]] [xarg1 10-Sep-2010] [xarg2 111 222] [xword0] [xword1 "system"] [xobj1 system 'version] [calls lib 'negate] [calls lib 'sine] [... ; Should return none, since a freshly made module! would not be there yet. |
Assigned to | n/a | Fixed in | alpha 110 | Last Update | 2-Nov-2010 06:47 |
---|
Comments | |
---|---|
(0002656)
BrianH 24-Oct-2010 20:53 |
If this ticket is dismissed (semantics of FIND changing and all that), at least the workaround code works. I'm not pushing for this either way. |
(0002666)
BrianH 26-Oct-2010 23:09 |
See #1720 for an alternative suggestion. |
(0002670)
Carl 26-Oct-2010 23:53 |
That's bizarre. Trying some tests on it. For now, assume it is fixed and don't work around it. |
(0002675)
BrianH 27-Oct-2010 00:31 |
Note: It doesn't even test equivalence - any module will do. I will comment out the workaround code when FIND is verified to work (wrote it already days ago). |
Date | User | Field | Action | Change |
---|---|---|---|---|
2-Nov-2010 06:47 | BrianH | Status | Modified | built => tested |
27-Oct-2010 00:35 | BrianH | Comment : 0002675 | Modified | - |
27-Oct-2010 00:35 | BrianH | Comment : 0002675 | Modified | - |
27-Oct-2010 00:31 | BrianH | Comment : 0002675 | Added | - |
27-Oct-2010 00:04 | carl | Fixedin | Modified | => alpha 110 |
27-Oct-2010 00:04 | carl | Status | Modified | reviewed => built |
26-Oct-2010 23:54 | carl | Comment : 0002670 | Modified | - |
26-Oct-2010 23:53 | carl | Comment : 0002670 | Added | - |
26-Oct-2010 23:09 | BrianH | Comment : 0002666 | Added | - |
24-Oct-2010 20:53 | BrianH | Comment : 0002656 | Added | - |
23-Oct-2010 22:50 | BrianH | Priority | Modified | high => normal |
21-Oct-2010 08:54 | BrianH | Description | Modified | - |
21-Oct-2010 08:54 | BrianH | Code | Modified | - |
21-Oct-2010 08:54 | BrianH | Status | Modified | submitted => reviewed |
21-Oct-2010 08:54 | BrianH | Priority | Modified | urgent => high |
21-Oct-2010 07:51 | BrianH | Ticket | Added | - |