From 1187a7719c07476b9926c2fcafa64a6e003550d9 Mon Sep 17 00:00:00 2001
From: Max <msuraev@sysmocom.de>
Date: Fri, 26 Jan 2018 13:31:42 +0100
Subject: 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 [1] tool):

* add missing headers
* remove unused headers

[1] https://include-what-you-use.org/

Related: OS#1539
Change-Id: I8c9f488a43b099c72b2d30d3245e7ba50872fc00
---
 src/bts.cpp                  |  9 ++++++---
 src/bts.h                    |  4 ++--
 src/gprs_bssgp_pcu.cpp       |  2 ++
 src/gprs_bssgp_pcu.h         |  2 --
 src/gprs_ms.cpp              |  3 ++-
 src/gprs_ms.h                |  3 +++
 src/gprs_ms_storage.cpp      |  5 ++++-
 src/gprs_rlcmac.cpp          |  2 +-
 src/gprs_rlcmac_sched.cpp    |  9 ++++++++-
 src/gprs_rlcmac_ts_alloc.cpp |  3 +++
 src/llc.h                    |  1 -
 src/osmobts_sock.cpp         |  4 +++-
 src/pcu_l1_if.cpp            |  7 ++++++-
 src/pcu_main.cpp             | 18 +++++++++++++++---
 src/pcu_vty_functions.cpp    |  8 ++++++++
 src/poll_controller.cpp      |  7 +++++++
 src/poll_controller.h        |  2 --
 src/rlc.cpp                  |  8 +++++++-
 src/sba.cpp                  |  3 ++-
 src/sba.h                    |  1 -
 src/tbf.cpp                  | 10 ++++++++--
 src/tbf.h                    |  9 ++++++---
 src/tbf_dl.cpp               | 14 ++++++++++++--
 src/tbf_ul.cpp               | 11 ++++++++++-
 tests/edge/EdgeTest.cpp      |  2 ++
 tests/tbf/TbfTest.cpp        |  1 +
 26 files changed, 118 insertions(+), 30 deletions(-)

diff --git a/src/bts.cpp b/src/bts.cpp
index cbd3a1c0..b0d92f0d 100644
--- a/src/bts.cpp
+++ b/src/bts.cpp
@@ -25,9 +25,10 @@
 #include <decoding.h>
 #include <rlc.h>
 #include <pcu_l1_if.h>
-
+#include <gprs_ms.h>
 #include <gprs_rlcmac.h>
 #include <gprs_debug.h>
+#include <cxx_linuxlist.h>
 
 extern "C" {
 	#include <osmocom/core/talloc.h>
@@ -37,10 +38,12 @@ extern "C" {
 	#include <osmocom/gsm/gsm_utils.h>
 	#include <osmocom/core/gsmtap_util.h>
 	#include <osmocom/core/application.h>
+	#include <osmocom/core/bitvec.h>
+	#include <osmocom/core/gsmtap.h>
+	#include <osmocom/core/logging.h>
+	#include <osmocom/core/utils.h>
 }
 
-#include <arpa/inet.h>
-
 #include <errno.h>
 #include <string.h>
 
diff --git a/src/bts.h b/src/bts.h
index 25f8ccde..9b38ba94 100644
--- a/src/bts.h
+++ b/src/bts.h
@@ -26,17 +26,17 @@ extern "C" {
 #include <osmocom/core/linuxlist.h>
 #include <osmocom/core/rate_ctr.h>
 #include <osmocom/core/stat_item.h>
-#include <osmocom/core/timer.h>
-#include <osmocom/core/gsmtap.h>
 #include <osmocom/gsm/l1sap.h>
 #include <osmocom/gsm/protocol/gsm_04_08.h>
 }
 
+#include <gsm_rlcmac.h>
 #include "poll_controller.h"
 #include "sba.h"
 #include "tbf.h"
 #include "gprs_ms_storage.h"
 #include "gprs_coding_scheme.h"
+#include <cxx_linuxlist.h>
 #endif
 
 #include <stdint.h>
diff --git a/src/gprs_bssgp_pcu.cpp b/src/gprs_bssgp_pcu.cpp
index a86e09df..e0d756ae 100644
--- a/src/gprs_bssgp_pcu.cpp
+++ b/src/gprs_bssgp_pcu.cpp
@@ -21,8 +21,10 @@
 #include <gprs_rlcmac.h>
 #include <gprs_bssgp_pcu.h>
 #include <pcu_l1_if.h>
+#include <gprs_debug.h>
 #include <bts.h>
 #include <tbf.h>
+#include <gprs_coding_scheme.h>
 
 #define BSSGP_TIMER_T1	30	/* Guards the (un)blocking procedures */
 #define BSSGP_TIMER_T2	30	/* Guards the reset procedure */
diff --git a/src/gprs_bssgp_pcu.h b/src/gprs_bssgp_pcu.h
index bb449034..4eda57d6 100644
--- a/src/gprs_bssgp_pcu.h
+++ b/src/gprs_bssgp_pcu.h
@@ -42,8 +42,6 @@ struct bssgp_bvc_ctx *btsctx_alloc(uint16_t bvci, uint16_t nsei);
 #define NS_HDR_LEN 4
 #define IE_LLC_PDU 14
 
-struct gprs_rlcmac_bts;
-
 struct gprs_bssgp_pcu {
 	struct gprs_nsvc *nsvc;
 	struct bssgp_bvc_ctx *bctx;
diff --git a/src/gprs_ms.cpp b/src/gprs_ms.cpp
index 33d9cad5..740ef6b1 100644
--- a/src/gprs_ms.cpp
+++ b/src/gprs_ms.cpp
@@ -20,7 +20,7 @@
 
 
 #include "gprs_ms.h"
-
+#include <gprs_coding_scheme.h>
 #include "bts.h"
 #include "tbf.h"
 #include "gprs_debug.h"
@@ -33,6 +33,7 @@ extern "C" {
 	#include <osmocom/core/talloc.h>
 	#include <osmocom/core/utils.h>
 	#include <osmocom/gsm/protocol/gsm_04_08.h>
+	#include <osmocom/core/logging.h>
 }
 
 #define GPRS_CODEL_SLOW_INTERVAL_MS 4000
diff --git a/src/gprs_ms.h b/src/gprs_ms.h
index f094e966..857f0c91 100644
--- a/src/gprs_ms.h
+++ b/src/gprs_ms.h
@@ -22,13 +22,16 @@
 
 struct gprs_codel;
 
+#include <gprs_coding_scheme.h>
 #include "cxx_linuxlist.h"
 #include "llc.h"
 #include "tbf.h"
 #include "pcu_l1_if.h"
+#include <gprs_coding_scheme.h>
 
 extern "C" {
 	#include <osmocom/core/timer.h>
+	#include <osmocom/core/linuxlist.h>
 }
 
 #include <stdint.h>
diff --git a/src/gprs_ms_storage.cpp b/src/gprs_ms_storage.cpp
index 6a7f3360..c7de6d30 100644
--- a/src/gprs_ms_storage.cpp
+++ b/src/gprs_ms_storage.cpp
@@ -23,7 +23,10 @@
 
 #include "tbf.h"
 #include "bts.h"
-#include "gprs_debug.h"
+
+extern "C" {
+	#include <osmocom/core/linuxlist.h>
+}
 
 #define GPRS_UNDEFINED_IMSI "000"
 
diff --git a/src/gprs_rlcmac.cpp b/src/gprs_rlcmac.cpp
index 06c5479e..da5c5417 100644
--- a/src/gprs_rlcmac.cpp
+++ b/src/gprs_rlcmac.cpp
@@ -24,7 +24,7 @@
 #include <bts.h>
 #include <encoding.h>
 #include <tbf.h>
-
+#include <gprs_debug.h>
 
 extern void *tall_pcu_ctx;
 
diff --git a/src/gprs_rlcmac_sched.cpp b/src/gprs_rlcmac_sched.cpp
index ebf4714c..e4e40b1f 100644
--- a/src/gprs_rlcmac_sched.cpp
+++ b/src/gprs_rlcmac_sched.cpp
@@ -22,9 +22,16 @@
 #include <pcu_l1_if.h>
 #include <bts.h>
 #include <tbf.h>
-
+#include <gprs_debug.h>
+#include <gprs_ms.h>
+#include <rlc.h>
+#include <sba.h>
 #include "pcu_utils.h"
 
+extern "C" {
+	#include <osmocom/core/gsmtap.h>
+}
+
 static uint32_t sched_poll(BTS *bts,
 		    uint8_t trx, uint8_t ts, uint32_t fn, uint8_t block_nr,
 		    struct gprs_rlcmac_tbf **poll_tbf,
diff --git a/src/gprs_rlcmac_ts_alloc.cpp b/src/gprs_rlcmac_ts_alloc.cpp
index c45aa398..aa10e454 100644
--- a/src/gprs_rlcmac_ts_alloc.cpp
+++ b/src/gprs_rlcmac_ts_alloc.cpp
@@ -31,6 +31,9 @@
 
 extern "C" {
 #include "mslot_class.h"
+#include <osmocom/core/linuxlist.h>
+#include <osmocom/core/logging.h>
+#include <osmocom/core/utils.h>
 }
 
 /* Consider a PDCH as idle if has at most this number of TBFs assigned to it */
diff --git a/src/llc.h b/src/llc.h
index 9d402c38..2e7229ce 100644
--- a/src/llc.h
+++ b/src/llc.h
@@ -29,7 +29,6 @@ extern "C" {
 #define LLC_MAX_LEN 1543
 
 struct BTS;
-struct msgb;
 
 /**
  * I represent the LLC data to a MS
diff --git a/src/osmobts_sock.cpp b/src/osmobts_sock.cpp
index 9e4000a0..b9664b08 100644
--- a/src/osmobts_sock.cpp
+++ b/src/osmobts_sock.cpp
@@ -29,9 +29,11 @@ extern "C" {
 #include <osmocom/core/talloc.h>
 #include <osmocom/core/select.h>
 #include <osmocom/core/msgb.h>
+#include <osmocom/core/linuxlist.h>
+#include <osmocom/core/logging.h>
+#include <osmocom/core/timer.h>
 }
 
-#include <gprs_rlcmac.h>
 #include <pcu_l1_if.h>
 #include <gprs_debug.h>
 #include <gprs_bssgp_pcu.h>
diff --git a/src/pcu_l1_if.cpp b/src/pcu_l1_if.cpp
index 0a9803b1..805336a5 100644
--- a/src/pcu_l1_if.cpp
+++ b/src/pcu_l1_if.cpp
@@ -26,12 +26,18 @@
 #include <sys/socket.h>
 #include <sys/un.h>
 #include <arpa/inet.h>
+
 extern "C" {
 #include <osmocom/core/talloc.h>
 #include <osmocom/core/select.h>
 #include <osmocom/core/msgb.h>
 #include <osmocom/core/gsmtap_util.h>
 #include <osmocom/core/gsmtap.h>
+#include <osmocom/core/bitvec.h>
+#include <osmocom/core/logging.h>
+#include <osmocom/core/utils.h>
+#include <osmocom/gsm/l1sap.h>
+#include <osmocom/gsm/protocol/gsm_04_08.h>
 }
 
 #include <gprs_rlcmac.h>
@@ -40,7 +46,6 @@ extern "C" {
 #include <gprs_bssgp_pcu.h>
 #include <osmocom/pcu/pcuif_proto.h>
 #include <bts.h>
-#include <tbf.h>
 
 // FIXME: move this, when changed from c++ to c.
 extern "C" {
diff --git a/src/pcu_main.cpp b/src/pcu_main.cpp
index b7574f96..0c3a4142 100644
--- a/src/pcu_main.cpp
+++ b/src/pcu_main.cpp
@@ -17,22 +17,34 @@
  * Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA  02111-1307, USA.
  */
 
-#include <gprs_bssgp_pcu.h>
-#include <arpa/inet.h>
 #include <pcu_l1_if.h>
 #include <gprs_rlcmac.h>
 #include <gsm_timer.h>
 #include <gprs_debug.h>
 #include <unistd.h>
+#include <stdbool.h>
+#include <stdio.h>
+#include <stdlib.h>
+#include <string.h>
+#include <errno.h>
 #include <getopt.h>
 #include <signal.h>
 #include <sched.h>
 #include <bts.h>
+#include <gprs_coding_scheme.h>
+#include <osmocom/pcu/pcuif_proto.h>
 extern "C" {
 #include "pcu_vty.h"
+#include <osmocom/gprs/gprs_bssgp.h>
+#include <osmocom/gprs/gprs_ns.h>
 #include <osmocom/vty/telnet_interface.h>
-#include <osmocom/vty/logging.h>
+#include <osmocom/vty/command.h>
+#include <osmocom/vty/vty.h>
 #include <osmocom/vty/ports.h>
+#include <osmocom/core/rate_ctr.h>
+#include <osmocom/core/select.h>
+#include <osmocom/core/application.h>
+#include <osmocom/core/msgb.h>
 #include <osmocom/core/stats.h>
 #include <osmocom/core/gsmtap.h>
 #include <osmocom/core/gsmtap_util.h>
diff --git a/src/pcu_vty_functions.cpp b/src/pcu_vty_functions.cpp
index 41165f3c..e80b0f57 100644
--- a/src/pcu_vty_functions.cpp
+++ b/src/pcu_vty_functions.cpp
@@ -27,11 +27,19 @@
 #include "gprs_ms_storage.h"
 #include "gprs_ms.h"
 #include "cxx_linuxlist.h"
+#include <gprs_coding_scheme.h>
+#include <llc.h>
+#include <pcu_l1_if.h>
+#include <rlc.h>
+#include <tbf.h>
 
 extern "C" {
 #include <osmocom/vty/command.h>
 #include <osmocom/vty/logging.h>
 #include <osmocom/vty/misc.h>
+	#include <osmocom/core/linuxlist.h>
+	#include <osmocom/core/utils.h>
+	#include <osmocom/vty/vty.h>
 }
 
 static void tbf_print_vty_info(struct vty *vty, gprs_rlcmac_tbf *tbf)
diff --git a/src/poll_controller.cpp b/src/poll_controller.cpp
index 5c5a7177..744b612f 100644
--- a/src/poll_controller.cpp
+++ b/src/poll_controller.cpp
@@ -23,6 +23,13 @@
 #include <poll_controller.h>
 #include <bts.h>
 #include <tbf.h>
+#include <cxx_linuxlist.h>
+#include <sba.h>
+
+extern "C" {
+#include <osmocom/core/linuxlist.h>
+#include <osmocom/gsm/gsm_utils.h>
+}
 
 PollController::PollController(BTS& bts)
 	: m_bts(bts)
diff --git a/src/poll_controller.h b/src/poll_controller.h
index b9ae80e5..65d1fee9 100644
--- a/src/poll_controller.h
+++ b/src/poll_controller.h
@@ -21,8 +21,6 @@
 
 #pragma once
 
-struct gprs_rlcmac_bts;
-
 struct BTS;
 
 /**
diff --git a/src/rlc.cpp b/src/rlc.cpp
index d7f06090..a82f550a 100644
--- a/src/rlc.cpp
+++ b/src/rlc.cpp
@@ -18,11 +18,17 @@
 
 #include "bts.h"
 #include "gprs_debug.h"
+#include <gprs_coding_scheme.h>
+#include <rlc.h>
 
-#include <errno.h>
+#include <stdbool.h>
+#include <string.h>
+#include <sys/types.h>
 
 extern "C" {
 #include <osmocom/core/utils.h>
+#include <osmocom/core/bitvec.h>
+#include <osmocom/core/logging.h>
 }
 
 
diff --git a/src/sba.cpp b/src/sba.cpp
index 56a75432..5dfdddbb 100644
--- a/src/sba.cpp
+++ b/src/sba.cpp
@@ -20,14 +20,15 @@
  */
 
 #include <sba.h>
-#include <gprs_rlcmac.h>
 #include <gprs_debug.h>
 #include <bts.h>
 #include <pcu_utils.h>
 
 extern "C" {
+#include <osmocom/core/logging.h>
 #include <osmocom/core/talloc.h>
 #include <osmocom/gsm/protocol/gsm_04_08.h>
+#include <osmocom/gsm/gsm_utils.h>
 }
 
 #include <errno.h>
diff --git a/src/sba.h b/src/sba.h
index d2d3106e..27dae212 100644
--- a/src/sba.h
+++ b/src/sba.h
@@ -27,7 +27,6 @@ extern "C" {
 }
 
 struct BTS;
-class PollController;
 struct gprs_rlcmac_pdch;
 
 /*
diff --git a/src/tbf.cpp b/src/tbf.cpp
index 8cfca3a9..74f2cd86 100644
--- a/src/tbf.cpp
+++ b/src/tbf.cpp
@@ -25,10 +25,13 @@
 #include <encoding.h>
 #include <gprs_rlcmac.h>
 #include <gprs_debug.h>
-#include <gprs_bssgp_pcu.h>
 #include <gprs_ms.h>
-#include <decoding.h>
 #include <pcu_utils.h>
+#include <gprs_ms_storage.h>
+#include <gsm_rlcmac.h>
+#include <sba.h>
+#include <gprs_coding_scheme.h>
+#include <gsm_timer.h>
 
 extern "C" {
 #include <osmocom/core/msgb.h>
@@ -36,6 +39,9 @@ extern "C" {
 #include <osmocom/core/talloc.h>
 #include <osmocom/core/stats.h>
 #include <osmocom/core/logging.h>
+	#include <osmocom/core/bitvec.h>
+	#include <osmocom/core/rate_ctr.h>
+	#include <osmocom/gsm/protocol/gsm_04_08.h>
 }
 
 #include <errno.h>
diff --git a/src/tbf.h b/src/tbf.h
index 2ed63f97..239b8fdc 100644
--- a/src/tbf.h
+++ b/src/tbf.h
@@ -20,20 +20,23 @@
 
 #ifdef __cplusplus
 
-#include "gprs_rlcmac.h"
 #include "llc.h"
 #include "rlc.h"
 #include "cxx_linuxlist.h"
 #include <gprs_debug.h>
-
+#include <gprs_coding_scheme.h>
+#include <gsm_timer.h>
 #include <stdint.h>
 extern "C" {
 #include <osmocom/core/utils.h>
+	#include <osmocom/core/linuxlist.h>
+	#include <osmocom/core/logging.h>
+	#include <osmocom/core/timer.h>
 }
 
 struct bssgp_bvc_ctx;
-struct pcu_l1_meas;
 class GprsMs;
+struct gprs_rlcmac_bts;
 
 /*
  * TBF instance
diff --git a/src/tbf_dl.cpp b/src/tbf_dl.cpp
index dc4fa6ae..c92bfdfa 100644
--- a/src/tbf_dl.cpp
+++ b/src/tbf_dl.cpp
@@ -28,18 +28,28 @@
 #include <gprs_codel.h>
 #include <decoding.h>
 #include <encoding.h>
-
+#include <gprs_coding_scheme.h>
+#include <gprs_ms.h>
+#include <gprs_ms_storage.h>
+#include <llc.h>
 #include "pcu_utils.h"
 
 extern "C" {
 #include <osmocom/core/msgb.h>
 #include <osmocom/core/talloc.h>
 #include <osmocom/gprs/gprs_bssgp_bss.h>
+	#include <osmocom/core/bitvec.h>
+	#include <osmocom/core/linuxlist.h>
+	#include <osmocom/core/logging.h>
+	#include <osmocom/core/rate_ctr.h>
+	#include <osmocom/core/timer.h>
+	#include <osmocom/core/utils.h>
+	#include <osmocom/gsm/gsm_utils.h>
+	#include <osmocom/gsm/protocol/gsm_04_08.h>
 }
 
 #include <errno.h>
 #include <string.h>
-#include <math.h>
 
 /* After sending these frames, we poll for ack/nack. */
 #define POLL_ACK_AFTER_FRAMES 20
diff --git a/src/tbf_ul.cpp b/src/tbf_ul.cpp
index 46d4d12b..a4d3499e 100644
--- a/src/tbf_ul.cpp
+++ b/src/tbf_ul.cpp
@@ -28,12 +28,21 @@
 #include <gprs_bssgp_pcu.h>
 #include <decoding.h>
 #include <pcu_l1_if.h>
-
+#include <gprs_coding_scheme.h>
+#include <gprs_ms.h>
+#include <llc.h>
 #include "pcu_utils.h"
 
 extern "C" {
 #include <osmocom/core/msgb.h>
 #include <osmocom/core/talloc.h>
+	#include <osmocom/core/bitvec.h>
+	#include <osmocom/core/logging.h>
+	#include <osmocom/core/rate_ctr.h>
+	#include <osmocom/core/utils.h>
+	#include <osmocom/gprs/gprs_bssgp_bss.h>
+	#include <osmocom/gprs/protocol/gsm_08_18.h>
+	#include <osmocom/gsm/tlv.h>
 }
 
 #include <errno.h>
diff --git a/tests/edge/EdgeTest.cpp b/tests/edge/EdgeTest.cpp
index 1abdcfd1..eb9ef42a 100644
--- a/tests/edge/EdgeTest.cpp
+++ b/tests/edge/EdgeTest.cpp
@@ -27,6 +27,8 @@
 #include "rlc.h"
 #include "llc.h"
 #include "bts.h"
+#include <gprs_rlcmac.h>
+
 extern "C" {
 #include "pcu_vty.h"
 
diff --git a/tests/tbf/TbfTest.cpp b/tests/tbf/TbfTest.cpp
index d4702f26..fb9158f9 100644
--- a/tests/tbf/TbfTest.cpp
+++ b/tests/tbf/TbfTest.cpp
@@ -27,6 +27,7 @@
 #include "gprs_bssgp_pcu.h"
 #include "pcu_l1_if.h"
 #include "decoding.h"
+#include <gprs_rlcmac.h>
 
 extern "C" {
 #include "pcu_vty.h"
-- 
cgit v1.2.3