diff options
author | Stefan Sperling <ssperling@sysmocom.de> | 2018-03-27 13:22:06 +0200 |
---|---|---|
committer | Stefan Sperling <ssperling@sysmocom.de> | 2018-03-27 12:25:19 +0000 |
commit | a8eafef966bfbd8046ee8319b889b2dee8391434 (patch) | |
tree | 201545c3a14c7525dbc5c71e7f5f220d3a9988f9 /tests | |
parent | ecb0308e74f3c9d0e228cddc79dc7794005b20f7 (diff) |
change return type of page_subscriber() to void
We deliberately ignore errors from page_subscriber() so there is
no point in having a non-void return value.
Provide more context in the error message logged if paging failed.
Add a comment in an implementation override of base_grace_paging_request()
in the test suite to make return value semantics more clear.
Change-Id: Ie18c2ba53d2055d3eaff8c9ed939eb844af6dd2e
Related: I48f5efbcddd98e15256edfca06ba0ae6acb5bab1
Diffstat (limited to 'tests')
-rw-r--r-- | tests/bssap/bssap_test.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/tests/bssap/bssap_test.c b/tests/bssap/bssap_test.c index 0cbf34a35..00d925d9f 100644 --- a/tests/bssap/bssap_test.c +++ b/tests/bssap/bssap_test.c @@ -50,7 +50,7 @@ int __wrap_bsc_grace_paging_request(enum signal_rf rf_policy, struct bsc_subscr else fprintf(stderr, "BSC paging started with LAC %u\n", subscr->lac); OSMO_ASSERT(gl_expect_lac == subscr->lac); - return 1; + return 1; /* pretend one BTS was paged */ } struct { |