RTT1.1.1 稳定版rt_exit_critical存在的Bug？

发布于 2013-10-14 08:04:29 浏览：3856 订阅该版

6 个回答

prife 2013-10-14

这家伙很懒，什么也没写！

>我使用的官网下载下来来RTT1.1.1 稳定版   地址：http://www.rt-thread.org/node/71，在scheduler.c文件中的rt_exit_critical方法发现一处不理解的地方
>我个人觉得    if (rt_thread_self() != RT_NULL);   这句判断语句后面直接多了一个分号，有点不合乎常理，显然没有起到判断的作用，请大家给分析下算不算Bug呢
>源代码如下：
>
>
>```
>/**
> * This function will unlock the thread scheduler.
> */
>void rt_exit_critical(void)
>{
>    register rt_base_t level;
>
>    /* disable interrupt */
>    level = rt_hw_interrupt_disable();
>
>    rt_scheduler_lock_nest --;
>
>    if (rt_scheduler_lock_nest <= 0)
>    {
>        rt_scheduler_lock_nest = 0;
>        /* enable interrupt */
>        rt_hw_interrupt_enable(level);
>
>                if (rt_thread_self() != RT_NULL);
>                rt_schedule();
>    }
>    else
>    {
>        /* enable interrupt */
>        rt_hw_interrupt_enable(level);
>    }
>}
>```

---

从缩进来看，似乎是你自己不小心复制了下面这段代码进去
                if (rt_thread_self() != RT_NULL);
                rt_schedule();

另外，在当前开发版的代码
[https://github.com/RT-Thread/rt-thread/blob/master/src/scheduler.c](https://github.com/RT-Thread/rt-thread/ ... cheduler.c)

```
void rt_exit_critical(void)
{
    register rt_base_t level;

/* disable interrupt */
    level = rt_hw_interrupt_disable();

rt_scheduler_lock_nest --;

if (rt_scheduler_lock_nest <= 0)
    {
        rt_scheduler_lock_nest = 0;
        /* enable interrupt */
        rt_hw_interrupt_enable(level);

rt_schedule();
    }
    else
    {
        /* enable interrupt */
        rt_hw_interrupt_enable(level);
    }
}
```