Opened 8 years ago

Last modified 5 years ago

#2881 reopened defect

NFScodeDependency stack overflow

Reported by: sjoelund.se Owned by: perost
Priority: high Milestone: Future
Component: Frontend Version: trunk
Keywords: Cc:

Description

FCSys.Assemblies.Cells.Cell gives a stack overflow in the dependency analysis (so I cannot upload the total model). https://github.com/kdavies4/FCSys/archive/master.zip

loadModel(FCSys);getErrorString();
saveTotalModel("total.mo",FCSys.Assemblies.Cells.Cell);getErrorString();

Change History (15)

comment:1 Changed 8 years ago by sjoelund.se

  • Resolution set to fixed
  • Status changed from new to closed

Fixed in r22716 (the library has plenty of other issues though)

comment:2 Changed 8 years ago by perost

Not really fixed though, the fix has issues...

comment:3 Changed 8 years ago by sjoelund.se

  • Resolution fixed deleted
  • Status changed from closed to reopened

Stupid Hudson and new broken tests :) The fix solves this issue though... Fine, let's leave this open for a while.

comment:4 Changed 8 years ago by perost

Yes, I reverted the stack overflow fix, it really wasn't a good fix. I don't quite know how to fix this right now, but as you say the library has lots of other issues anyway.

comment:5 Changed 8 years ago by sjoelund.se

  • Milestone changed from 1.9.1 to 1.9.2

This ticket was not closed for 1.9.1, which has now been released. It was batch modified for milestone 1.9.2 (but maybe an empty milestone was more appropriate; feel free to change it).

comment:6 Changed 8 years ago by sjoelund.se

comment:7 Changed 8 years ago by sjoelund.se

See also #3037

comment:8 Changed 7 years ago by sjoelund.se

  • Milestone changed from 1.9.2 to 1.9.3

Milestone changed to 1.9.3 since 1.9.2 was released.

comment:9 Changed 7 years ago by sjoelund.se

  • Milestone changed from 1.9.3 to 1.9.4

Moved to new milestone 1.9.4

comment:10 Changed 6 years ago by sjoelund.se

  • Milestone changed from 1.9.4 to 1.9.5

Milestone pushed to 1.9.5

comment:11 Changed 6 years ago by sjoelund.se

  • Milestone changed from 1.9.5 to 1.10.0

Milestone renamed

comment:12 Changed 6 years ago by Christoph <buchner@…>

According to the docs, FCSys does not even support OM. In fact, I have never seen it pass a single test on Hudson. Maybe this should just be removed from the list of libraries, as it just takes up build time for no good reason?

comment:13 Changed 6 years ago by sjoelund.se

  • Milestone changed from 1.10.0 to 1.11.0

Ticket retargeted after milestone closed

comment:14 Changed 5 years ago by sjoelund.se

  • Milestone changed from 1.11.0 to 1.12.0

Milestone moved to 1.12.0 due to 1.11.0 already being released.

comment:15 Changed 5 years ago by casella

  • Milestone changed from 1.12.0 to Future

The milestone of this ticket has been reassigned to "Future".

If you think the issue is still valid and relevant for you, please select milestone 1.13.0 for back-end, code generation and run-time issues, or 2.0.0 for front-end issues.

If you are aware that the problem is no longer present, please select the milestone corresponding to the version of OMC you used to check that, and set the status to "worksforme".

In both cases, a short informative comment would be welcome.

Note: See TracTickets for help on using tickets.