Harald Welte has submitted this change and it was merged.
Change subject: Update header includes
..
Update header includes
Many files include unnecessary headers and don't include headers which
are actually used. Because of
Patch Set 4: Code-Review+2
--
To view, visit https://gerrit.osmocom.org/6100
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I8c9f488a43b099c72b2d30d3245e7ba50872fc00
Gerrit-PatchSet: 4
Gerrit-Project: osmo-pcu
Gerrit-Branch: master
Hello Jenkins Builder,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/6100
to look at the new patch set (#4).
Update header includes
Many files include unnecessary headers and don't include headers which
are actually used. Because of that combined with the
Hello Jenkins Builder,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/6100
to look at the new patch set (#3).
Update header includes
Many files include unnecessary headers and don't include headers which
are actually used. Because of that combined with the
Patch Set 1:
(1 comment)
https://gerrit.osmocom.org/#/c/6100/1/src/bts.h
File src/bts.h:
PS1, Line 38: #include
: #include
Cosmetic: usually we place the <..> headers first.
--
To view, visit https://gerrit.osmocom.org/6100
To unsubscribe, visit
Review at https://gerrit.osmocom.org/6100
Update header includes
Many files include unnecessary headers and don't include headers which
are actually used. Because of that combined with the fact that OsmoPCU
is a mixture of C and C++, it makes it hard to modularize code. Fix
this (using iwyu