On Mon, Sep 21, 2015 at 07:06:01PM +0200, Marek Polacek wrote:
> I realized that current patch has a minor deficiency: it will start
> a chain even in case the first condition has a side-effect thus the
> chain should be invalid.  I'll fix this problem soon.

I changed my mind, the above mean we'll warn for

int
fn3 (void)
{
  if (bar ()) 
    return 1;
  else if (a) 
    return 2;
  else if (a);
    return 3;
  return 0;
}

But I think that's ok to warn on.

So the v2 patch I posted is still the latest one.

        Marek

Reply via email to