On Thu, Sep 15, 2016 at 8:53 AM, Mark Wielaard <m...@redhat.com> wrote:
> GCC6 will warn about:
>
> lib/rpmplugins.c: In function ‘rpmpluginsCallInit’:
> lib/rpmplugins.c:217:5: warning: this ‘if’ clause does not guard...
>      if (hookFunc)
>      ^~
> lib/rpmplugins.c:219:9: note: ...this statement, but the latter is 
> misleadingly indented as if it is guarded by the ‘if’
>          if (rc != RPMRC_OK && rc != RPMRC_NOTFOUND)
>          ^~
>
> GCC is right, this is misleading and causes the if statement to be executed
> even when it isn't necessary. It doesn't actually cause wrong results
> because rc is initialized to RPMRC_OK earlier and can only change when
> hookFunc != NULL.
>

Looks good to me, and should be okay to apply!



-- 
真実はいつも一つ!/ Always, there's only one truth!
_______________________________________________
Rpm-maint mailing list
Rpm-maint@lists.rpm.org
http://lists.rpm.org/mailman/listinfo/rpm-maint

Reply via email to