Opened 9 years ago

Closed 9 years ago

Last modified 9 years ago

#3666 closed defect (fixed)

Undefined variable time is not detected within functions

Reported by: Lennart Ochel Owned by: Per Östlund
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 by Per Östlund, 9 years ago

Owner: changed from somebody to Per Östlund
Status: newaccepted

comment:2 by Per Östlund, 9 years ago

Resolution: fixed
Status: acceptedclosed

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.

comment:3 by Lennart Ochel, 9 years ago

Milestone: Future1.9.4

Thanks!

comment:4 by Adrian Pop, 9 years ago

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 by Martin Sjölund, 9 years ago

Milestone: 1.9.41.9.4-1.9.x

Milestone renamed

comment:6 by Martin Sjölund, 9 years ago

Milestone: 1.9.4-1.9.x1.9.4

Milestone renamed

Note: See TracTickets for help on using tickets.