mirror of
https://github.com/RIOT-OS/RIOT.git
synced 2024-12-28 23:09:46 +01:00
sys/tsrb: add peek functions
This commit is contained in:
parent
45add49342
commit
857f3c3c51
@ -129,6 +129,14 @@ static inline unsigned int tsrb_free(const tsrb_t *rb)
|
||||
*/
|
||||
int tsrb_get_one(tsrb_t *rb);
|
||||
|
||||
/**
|
||||
* @brief Get a byte from ringbuffer, without removing it
|
||||
* @param[in] rb Ringbuffer to operate on
|
||||
* @return >=0 byte that has been read
|
||||
* @return -1 if no byte available
|
||||
*/
|
||||
int tsrb_peek_one(tsrb_t *rb);
|
||||
|
||||
/**
|
||||
* @brief Get bytes from ringbuffer
|
||||
* @param[in] rb Ringbuffer to operate on
|
||||
@ -138,6 +146,15 @@ int tsrb_get_one(tsrb_t *rb);
|
||||
*/
|
||||
int tsrb_get(tsrb_t *rb, uint8_t *dst, size_t n);
|
||||
|
||||
/**
|
||||
* @brief Get bytes from ringbuffer, without removing them
|
||||
* @param[in] rb Ringbuffer to operate on
|
||||
* @param[out] dst buffer to write to
|
||||
* @param[in] n max number of bytes to write to @p dst
|
||||
* @return nr of bytes written to @p dst
|
||||
*/
|
||||
int tsrb_peek(tsrb_t *rb, uint8_t *dst, size_t n);
|
||||
|
||||
/**
|
||||
* @brief Drop bytes from ringbuffer
|
||||
* @param[in] rb Ringbuffer to operate on
|
||||
|
@ -30,6 +30,11 @@ static uint8_t _pop(tsrb_t *rb)
|
||||
return rb->buf[rb->reads++ & (rb->size - 1)];
|
||||
}
|
||||
|
||||
static uint8_t _peek(tsrb_t *rb, unsigned int idx)
|
||||
{
|
||||
return rb->buf[(rb->reads + idx) & (rb->size - 1)];
|
||||
}
|
||||
|
||||
int tsrb_get_one(tsrb_t *rb)
|
||||
{
|
||||
int retval = -1;
|
||||
@ -41,6 +46,17 @@ int tsrb_get_one(tsrb_t *rb)
|
||||
return retval;
|
||||
}
|
||||
|
||||
int tsrb_peek_one(tsrb_t *rb)
|
||||
{
|
||||
int retval = -1;
|
||||
unsigned irq_state = irq_disable();
|
||||
if (!tsrb_empty(rb)) {
|
||||
retval = _peek(rb, 0);
|
||||
}
|
||||
irq_restore(irq_state);
|
||||
return retval;
|
||||
}
|
||||
|
||||
int tsrb_get(tsrb_t *rb, uint8_t *dst, size_t n)
|
||||
{
|
||||
size_t tmp = n;
|
||||
@ -53,6 +69,18 @@ int tsrb_get(tsrb_t *rb, uint8_t *dst, size_t n)
|
||||
return (n - tmp);
|
||||
}
|
||||
|
||||
int tsrb_peek(tsrb_t *rb, uint8_t *dst, size_t n)
|
||||
{
|
||||
size_t idx = 0;
|
||||
unsigned irq_state = irq_disable();
|
||||
unsigned int avail = tsrb_avail(rb);
|
||||
while (idx < n && idx != avail) {
|
||||
*dst++ = _peek(rb, idx++);
|
||||
}
|
||||
irq_restore(irq_state);
|
||||
return idx;
|
||||
}
|
||||
|
||||
int tsrb_drop(tsrb_t *rb, size_t n)
|
||||
{
|
||||
size_t tmp = n;
|
||||
|
@ -115,6 +115,43 @@ static void test_get(void)
|
||||
}
|
||||
}
|
||||
|
||||
static void test_peek_one(void)
|
||||
{
|
||||
int res;
|
||||
|
||||
TEST_ASSERT_EQUAL_INT(-1, tsrb_peek_one(&_tsrb));
|
||||
TEST_ASSERT_EQUAL_INT(0, tsrb_add_one(&_tsrb, TEST_INPUT));
|
||||
TEST_ASSERT_EQUAL_INT(TEST_INPUT, tsrb_peek_one(&_tsrb));
|
||||
TEST_ASSERT_EQUAL_INT(TEST_INPUT, tsrb_peek_one(&_tsrb));
|
||||
TEST_ASSERT_EQUAL_INT(TEST_INPUT, tsrb_get_one(&_tsrb));
|
||||
TEST_ASSERT_EQUAL_INT(-1, tsrb_peek_one(&_tsrb));
|
||||
TEST_ASSERT_EQUAL_INT(0, tsrb_add_one(&_tsrb, 0xff));
|
||||
res = tsrb_peek_one(&_tsrb);
|
||||
TEST_ASSERT_EQUAL_INT(0xff, res);
|
||||
/* 0xff is -1 in signed int8_t */
|
||||
TEST_ASSERT(-1 != res);
|
||||
}
|
||||
|
||||
static void test_peek(void)
|
||||
{
|
||||
TEST_ASSERT(BUFFER_SIZE < sizeof(_io_buffer));
|
||||
TEST_ASSERT_EQUAL_INT(0, tsrb_peek(&_tsrb, _io_buffer,
|
||||
sizeof(_io_buffer)));
|
||||
|
||||
for (int i = 0; i < BUFFER_SIZE; i++) {
|
||||
TEST_ASSERT_EQUAL_INT(0, tsrb_add_one(&_tsrb, TEST_INPUT + i));
|
||||
}
|
||||
TEST_ASSERT_EQUAL_INT(BUFFER_SIZE, tsrb_peek(&_tsrb, _io_buffer,
|
||||
sizeof(_io_buffer)));
|
||||
for (int i = 0; i < BUFFER_SIZE; i++) {
|
||||
TEST_ASSERT_EQUAL_INT((TEST_INPUT + i), _io_buffer[i]);
|
||||
}
|
||||
for (int i = BUFFER_SIZE; i < (int)sizeof(_io_buffer); i++) {
|
||||
TEST_ASSERT_EQUAL_INT(IO_BUFFER_CANARY, _io_buffer[i]);
|
||||
}
|
||||
TEST_ASSERT_EQUAL_INT(BUFFER_SIZE, tsrb_avail(&_tsrb));
|
||||
}
|
||||
|
||||
static void test_drop(void)
|
||||
{
|
||||
TEST_ASSERT(BUFFER_SIZE < sizeof(_io_buffer));
|
||||
@ -170,7 +207,9 @@ static Test *tests_tsrb_tests(void)
|
||||
new_TestFixture(test_full),
|
||||
new_TestFixture(test_free),
|
||||
new_TestFixture(test_get_one),
|
||||
new_TestFixture(test_peek_one),
|
||||
new_TestFixture(test_get),
|
||||
new_TestFixture(test_peek),
|
||||
new_TestFixture(test_drop),
|
||||
new_TestFixture(test_add_one),
|
||||
new_TestFixture(test_add),
|
||||
|
Loading…
Reference in New Issue
Block a user