Darrell Ball <[email protected]> writes: > This warning could be considered a bit confusing bcoz this patch does not > modify the file being reported.
Agreed. I need to fix up the auto-detection. It looks like the series had already been applied. The robot no longer warns when the patch has already been applied but it looks like it won't stop processing. The mechanism is to `git am` and then `checkpatch -q -1` Since these patches were already applied, the checkpatch -q -1 was picking up the last patch in the series twice. I'll see what I can do about getting it fixed :) Thanks, Darrell! > On Mon, Feb 25, 2019 at 4:48 PM 0-day Robot <[email protected]> wrote: > > Bleep bloop. Greetings Darrell Ball, I am a robot and I have tried out your > patch. > Thanks for your contribution. > > I encountered some error that I wasn't expecting. See the details below. > > checkpatch: > WARNING: Empty return followed by brace, consider omitting > #78 FILE: lib/conntrack.c:3101: > } > > Lines checked: 83, Warnings: 1, Errors: 0 > > Please check this out. If you feel there has been an error, please email > [email protected] > > Thanks, > 0-day Robot _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
