#3577 closed defect (fixed)
Complete the implementation of copyClass API
| Reported by: | Adeel Asghar | Owned by: | Per Östlund |
|---|---|---|---|
| Priority: | blocker | Milestone: | 1.9.4 |
| Component: | Interactive Environment | Version: | |
| Keywords: | Cc: |
Description
There are two issues with the API,
- It only works for a single class. If a class contains nested classes then the source info for them is copied from the source class which is bad.
- If the class is copied to a different scope then the elements of the class might need to be updated accordingly. One idea is to make the paths full e.g Interfaces.Pin p; should become Modelica.Electrical.Analog.Interfaces.Pin p;
Change History (6)
comment:1 by , 9 years ago
comment:2 by , 9 years ago
| Priority: | high → blocker |
|---|
comment:3 by , 9 years ago
copyClass has now been changed in a64582d so that it tries to figure out the destination file of the copied class.
comment:4 by , 9 years ago
| Resolution: | → fixed |
|---|---|
| Status: | new → closed |
The problem that inspired this ticket (#3630) is solved after Per's commit.
Note:
See TracTickets
for help on using tickets.
The first issue has been fixed in b1d0c1d. We now change all source infos in the copied class to <interactive>, with the lines offset with the enclosing class' start line (i.e. an element on line 4 in a class will have a source info that says <interactive>:4:...).