#3666 closed defect (fixed)
Undefined variable time is not detected within functions
| Reported by: | lochel | Owned by: | perost |
|---|---|---|---|
| Priority: | high | Milestone: | 1.9.4 |
| Component: | Frontend | Version: | |
| Keywords: | Cc: |
Description
Undefined variable time in functions is not detected. In some cases the function call get inlined and the simulation get compiled, in the other cases compilation of the generated code fails.
Change History (6)
comment:1 Changed 9 years ago by perost
- Owner changed from somebody to perost
- Status changed from new to accepted
comment:2 Changed 9 years ago by perost
- Resolution set to fixed
- Status changed from accepted to closed
comment:4 Changed 9 years ago by adrpo
Maybe we should add a small library coverage test with BuildModelRecursive.mos as your commit broke it. I now fixed the use of time in the script functions.
comment:5 Changed 8 years ago by sjoelund.se
- Milestone changed from 1.9.4 to 1.9.4-1.9.x
Milestone renamed
comment:6 Changed 8 years ago by sjoelund.se
- Milestone changed from 1.9.4-1.9.x to 1.9.4
Milestone renamed
Note: See
TracTickets for help on using
tickets.
Fixed in 47ae7d2. time is now only allowed in models and blocks as per the specification. Time is also treated more lika a built-in name, i.e. it's not allowed to declare components named time.