Messages in this thread | | | From | Todd Poynor <> | Date | Tue, 17 Jul 2018 12:53:09 -0700 | Subject | Re: [PATCH 12/32] staging: gasket: gasket_wait_with_reschedule return when condition hit |
| |
On Tue, Jul 17, 2018 at 4:12 AM, Dan Carpenter <dan.carpenter@oracle.com> wrote: > On Tue, Jul 17, 2018 at 01:13:29PM +0300, Dan Carpenter wrote: >> On Mon, Jul 16, 2018 at 07:09:06PM -0700, Todd Poynor wrote: >> > From: Todd Poynor <toddpoynor@google.com> >> > >> > Return right away instead of break out of while and then return. >> > >> > > Btw, I wasn't going to complain about this but since Greg is being extra > critical about commit messages that don't explain *why* it just creates > an environment why I also feel free to complain... I couldn't > understand what this commit does without looking at the code in context. > > A possible commit message here is: > > [PATCH 12/32] staging: gasket: small cleanup in gasket_wait_with_reschedule() > > gasket_wait_with_reschedule() is a little more clear if we just return > directly.
I've updated the text for v3, thanks -- Todd
> > regards, > dan carpenter >
-- Todd
| |