OMNeT++/OMNEST Bug Tracker

View Issue Details Jump to Notes ] Issue History ] Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0000816OMNeT++simulation kernelpublic2015-03-06 16:542017-01-11 20:07
Reportertill 
Assigned Toandras 
PrioritynormalSeverityminorReproducibilityalways
StatusresolvedResolutionfixed 
PlatformOSOS Version
Product Version5.0b1 
Target VersionFixed in Version 
Summary0000816: cenvir.h misses csimulation.h
Descriptioncenvir.h misses csimulation.h include, though using ev macro defined in cenvir.h will result in error as long as csimulation.h is not included.

I guess this was not intended.
Additional Informationresolution:
add #include "csimulation.h" to cenvir.h
TagsNo tags attached.
Attached Files

- Relationships

-  Notes
(0000990)
andras (administrator)
2015-03-12 20:10

Why is this important? For simulation models, only <omnetpp.h> is public API, individual headers should be considered internal, and never be included directly.

BTW it's not so easy to fix, e.g. the suggested >>add #include "csimulation.h" to cenvir.h<< leads to compile error. Many headers contain inline methods (for performance reasons) that refer to classes in other headers, and this creates dependencies across headers that are difficult to resolve.
(0001191)
andras (administrator)
2016-03-16 10:45

The "ev" macro has been removed.

- Issue History
Date Modified Username Field Change
2015-03-06 16:54 till New Issue
2015-03-12 20:10 andras Note Added: 0000990
2015-10-12 09:28 ammmar1988 Issue cloned: 0000856
2016-03-16 10:45 andras Note Added: 0001191
2016-03-16 10:45 andras Status new => resolved
2016-03-16 10:45 andras Resolution open => fixed
2016-03-16 10:45 andras Assigned To => andras


Copyright © 2000 - 2019 MantisBT Team
Powered by Mantis Bugtracker