diff options
author | Pau Espin Pedrol <pespin@sysmocom.de> | 2021-02-25 18:08:10 +0100 |
---|---|---|
committer | Pau Espin Pedrol <pespin@sysmocom.de> | 2021-02-26 11:50:21 +0100 |
commit | c85e093969d64ec99f1dec1ece3f2ff92d70dd51 (patch) | |
tree | 72a5a9f6c4ee89e64bb3f0acc6298056a848ad48 /tests | |
parent | 36177c6b584ebb6e301d55e5bc5d9eb76f80c368 (diff) |
Remove uneeded ms param from alloc_algorithm_func_t func
Since a while ago, the data architecture was changed so that TBF is
guaranteed to always have a MS object associated. Hence, it makes no
sense to pass the MS object as a separate param as we can take it from
tbf object and makes code less confusing.
Change-Id: Idc0c76cf6f007afa4236480cdad0d8e99dabec5f
Diffstat (limited to 'tests')
-rw-r--r-- | tests/alloc/AllocTest.cpp | 7 |
1 files changed, 2 insertions, 5 deletions
diff --git a/tests/alloc/AllocTest.cpp b/tests/alloc/AllocTest.cpp index d6266923..fe803e32 100644 --- a/tests/alloc/AllocTest.cpp +++ b/tests/alloc/AllocTest.cpp @@ -423,9 +423,6 @@ static void test_alloc_b() test_all_alloc_b(); } -typedef int (*algo_t)(struct gprs_rlcmac_bts *bts, struct GprsMs *ms, struct gprs_rlcmac_tbf *tbf, bool single, - int8_t use_trx); - static char get_dir_char(uint8_t mask, uint8_t tx, uint8_t rx, uint8_t busy) { int offs = busy ? 32 : 0; @@ -640,7 +637,7 @@ static unsigned alloc_many_tbfs(struct gprs_rlcmac_bts *bts, unsigned min_class, return counter; } -static void test_successive_allocation(algo_t algo, unsigned min_class, +static void test_successive_allocation(alloc_algorithm_func_t algo, unsigned min_class, unsigned max_class, enum test_mode mode, unsigned expect_num, const char *text) { @@ -674,7 +671,7 @@ static void test_successive_allocation(algo_t algo, unsigned min_class, talloc_free(bts); } -static void test_many_connections(algo_t algo, unsigned expect_num, +static void test_many_connections(alloc_algorithm_func_t algo, unsigned expect_num, const char *text) { struct gprs_rlcmac_bts *bts = bts_alloc(the_pcu, 0); |