On Tue, Jul 18, 2017 at 12:28 PM, Dan Carpenter <dan.carpenter@xxxxxxxxxx> wrote: > Static checkers complain because: > > if (peaq_ignore_events_counter && --peaq_ignore_events_counter >= 0) > ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ > The second part of the condition is always true because > "peaq_ignore_events_counter" is unsigned. It doesn't cause a problem > because the first part of the condition prevents underflows, but it's > simple enough to make the static checker happy. Isn't it the same to if (peaq_ignore_events_counter && peaq_ignore_events_counter--) ? -- With Best Regards, Andy Shevchenko
