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