On 2020/9/22 19:23, Andy Shevchenko wrote:



+          if ((!vmsg) || (vmsg != &vi->vmsg)) {
 
This is a bit difficult to read (esp. taking into account the above comment)
besides the fact of redundant parentheses. Why not simply:
 
         if (!(vmsg && vmsg == &vi->vmsg)) {
 
?

Thanks Andy. It looks better.

 

 
-- 
2.7.4