diff options
| author | Philipp Maier <pmaier@sysmocom.de> | 2018-08-29 16:41:09 +0200 |
|---|---|---|
| committer | Philipp Maier <pmaier@sysmocom.de> | 2018-08-29 17:34:49 +0200 |
| commit | 9b41b36e97383ce64db6732a7146ff25c126dc5a (patch) | |
| tree | 011dc3a3012e0687654ae6337c5fbf41874cde53 /src | |
| parent | 76c292ea5da1949407f1572321994537b21280c9 (diff) | |
measurement: add unit tests for ts45008_83_is_sub()
The function ts45008_83_is_sub() is an integral part of the measurement
calculation as it automatically tags incoming measurements as SUB
measurements. This is important in the context of DTX. Unfortunately
there is no unit test for this function yet.
- Add unit test for ts45008_83_is_sub()
Change-Id: Ia26774859f4bf31baee075896905079368bddd42
Related: OS#3502
Diffstat (limited to 'src')
| -rw-r--r-- | src/common/measurement.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/src/common/measurement.c b/src/common/measurement.c index b0ef8694..bc27287c 100644 --- a/src/common/measurement.c +++ b/src/common/measurement.c @@ -27,8 +27,9 @@ static bool array_contains(const uint8_t *arr, unsigned int len, uint8_t val) { return false; } -/* Decide if a given frame number is part of the "-SUB" measurements (true) or not (false) */ -static bool ts45008_83_is_sub(struct gsm_lchan *lchan, uint32_t fn, bool is_amr_sid_update) +/* Decide if a given frame number is part of the "-SUB" measurements (true) or not (false) + * (this function is only used internally, it is public to call it from unit-tests) */ +bool ts45008_83_is_sub(struct gsm_lchan *lchan, uint32_t fn, bool is_amr_sid_update) { uint32_t fn104 = fn % 104; |