ringbuff 组件的问题

发布于 2019-05-08 11:52:30 浏览：1447 订阅该版

* 本帖最后由 emlsyx 于 2019-5-8 11:57 编辑 *

**组件`ringbuff` 中有一个获取buff状态的函数，路径`components\drivers\src ingbuffer.c`**
函数内容: 
```rt_inline enum rt_ringbuffer_state rt_ringbuffer_status(struct rt_ringbuffer *rb)
{
    if (rb->read_index == rb->write_index)
    {
        if (rb->read_mirror == rb->write_mirror)
            return RT_RINGBUFFER_EMPTY;
        else
            return RT_RINGBUFFER_FULL;
    }
    return RT_RINGBUFFER_HALFFULL;
}```对这个函数有一个不成熟的建议，建议改写下面这样：
```rt_inline enum rt_ringbuffer_state rt_ringbuffer_status(struct rt_ringbuffer *rb)
{
    /* 先判断buff满不满*/
    if (rb->read_mirror == rb->write_mirror)
    {
        if (rb->read_index == rb->write_index)
            return RT_RINGBUFFER_EMPTY;
        else
            return RT_RINGBUFFER_HALFFULL;
    }
    return RT_RINGBUFFER_FULL;
}```

原因说起来有点麻烦，直接举个例子：
假如创建一个大小为12的环形buff，（ `rb->buffer_size` 等于12），第一次写入的数据为`value[15] = {1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13, 14, 15}`，按照逻辑，数据会溢出，`1,2,3`会被覆盖掉，如果用原来的判断，此时读数据的话，只能读出三个`13,14,15`，如果改成先判断`if (rb->read_mirror == rb->write_mirror)` 的话，就能读出`13,14,15,4,5,6,7,8,9,10,11,12`

还有一个问题，用md写的帖子，里边用```c  ```   格式时，代码中结构体指针符号"->"  会变成乱码