关于rtgui中rtgui_server_post_event_sync函数的bug

发布于 2013-03-28 22:27:04 浏览：2226 订阅该版

在rtgui_server_post_event_sync（）函数的代码如下：
```rt_err_t rtgui_server_post_event_sync(struct rtgui_event *event, rt_size_t size)
{
    if (rtgui_server_app != RT_NULL)
        return rtgui_send_sync(rtgui_server_app, event, size);
    else
    {
        rt_kprintf("post when server is not running
");
        return -RT_ENOSYS;
    }
}```
通过判断rtgui_server_app 是否为空，还判断gui server是不是运行了。个人觉得这样判断是有问题的。因为rtgui_server_app这个变量的赋值是在名字为rtgui的线程里进行的。调用rtgui_system_server_init();这个函数会启动这个线程。如果在调用这个函数后马上现启动一个UI线程的话。就会调用到rtgui_server_post_event_sync（）函数，当这个函数去判断rtgui_server_app != RT_NULL的时候，可能rtgui_server_app并没有准备好。这个函数就会返回错误。造成UI线程失败。
　　如果延时几秒再去启动UI线程，这个时候rtgui_server_app就会已经准备好了。这样UI线程就不会失败了。所以个人觉得通过rtgui_server_app != RT_NULL来判断是存在bug的。

如果在这里先判断rtgui线程是否已经运行了。如果运行了再判断rtgui_server_app != RT_NULL，如是为空，则等待rtgui_server_app准备好会不会更好一些？我把这个函数改成了如下代码，不知可不可行。
```rt_err_t rtgui_server_post_event_sync(struct rtgui_event *event, rt_size_t size)
{
    if (rtgui_server_app != RT_NULL)
        return rtgui_send_sync(rtgui_server_app, event, size);
    else
    {
		if(rt_thread_find("rtgui") != RT_NULL)
		{	
			while(rtgui_server_app == RT_NULL)
				rt_thread_sleep(RT_TICK_PER_SECOND/10);
			
			return rtgui_send_sync(rtgui_server_app, event, size);
		}
		else
		{
                       rt_kprintf("post when server is not running
");
                       return -RT_ENOSYS;
		}
    }
}```