Bug 1346 - Include cleanup: use C++ headers instead of C headers where possible, remove unused headers, remove using namespace std statements
Include cleanup: use C++ headers instead of C headers where possible, remove ...
Status: RESOLVED DUPLICATE of bug 1237
Product: ns-3
Classification: Unclassified
Component: core
pre-release
All All
: P5 normal
Assigned To: Mathieu Lacage
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2012-01-16 15:12 EST by Vedran Miletić
Modified: 2012-01-29 08:22 EST (History)
1 user (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Vedran Miletić 2012-01-16 15:12:44 EST
This was discussed long time ago and patch was quite big. Two points were discussed:
1) Portability of <cheader> headers; need to test with g++-3.4.6 and g++-4.0.
2) Sorting order of includes: it would be good if we can standardize an order, e.g. first local module headers, then ns3/ headers, then C++ headers and C headers, and finally Unix headers if any.

I will post a cleaned-up patch that applies against current codebase in a few days.

include-what-you-use[1] could also be helpful here, but in that case we should probably first make ns-3 build with clang since it's clang-based.

[1] http://code.google.com/p/include-what-you-use/
Comment 1 Vedran Miletić 2012-01-29 08:22:42 EST
Oops, I forgot about this.

*** This bug has been marked as a duplicate of bug 1237 ***