0000816: cenvir.h misses csimulation.h - OMNeT++/OMNEST Bug Tracker

OMNeT++/OMNEST Bug Tracker

View Issue Details [ Jump to Notes ]

[ Issue History ] [ Print ]

ID

Project

Category

View Status

Date Submitted

Last Update

0000816

OMNeT++

simulation kernel

public

2015-03-06 16:54

2017-01-11 20:07

Reporter

till

Assigned To

andras

Priority

normal

Severity

minor

Reproducibility

always

Status

resolved

Resolution

fixed

Platform

OS

OS Version

Product Version

5.0b1

Target Version

Fixed in Version

Summary

0000816: cenvir.h misses csimulation.h

Description

cenvir.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 Information

resolution:
add #include "csimulation.h" to cenvir.h

Tags

No tags attached.

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 - 2021 MantisBT Team