Re: [PATCH 4/4] HID: multitouch: remove unneeded else conditional cases

2018-08-09 Thread Joey Pabalinas
On Tue, Jul 03, 2018 at 10:17:58AM +0200, Benjamin Tissoires wrote: > This one looks good. > Reviewed-by: Benjamin Tissoires > > Just FYI, I sent out a big refactor of the hid-multitouch code. Jiri > should be still reviewing it, so I am not so sure who will have to > rebase which series on top

Re: [PATCH 4/4] HID: multitouch: remove unneeded else conditional cases

2018-08-09 Thread Joey Pabalinas
On Tue, Jul 03, 2018 at 10:17:58AM +0200, Benjamin Tissoires wrote: > This one looks good. > Reviewed-by: Benjamin Tissoires > > Just FYI, I sent out a big refactor of the hid-multitouch code. Jiri > should be still reviewing it, so I am not so sure who will have to > rebase which series on top

Re: [PATCH 4/4] HID: multitouch: remove unneeded else conditional cases

2018-07-03 Thread Benjamin Tissoires
On Sun, Jul 1, 2018 at 2:19 AM, Joey Pabalinas wrote: > Elide lone `else` cases and replace `else if` clauses > with plain `if` conditionals when they occur immediately > after return statements. > > Signed-off-by: Joey Pabalinas > This one looks good. Reviewed-by: Benjamin Tissoires Just

Re: [PATCH 4/4] HID: multitouch: remove unneeded else conditional cases

2018-07-03 Thread Benjamin Tissoires
On Sun, Jul 1, 2018 at 2:19 AM, Joey Pabalinas wrote: > Elide lone `else` cases and replace `else if` clauses > with plain `if` conditionals when they occur immediately > after return statements. > > Signed-off-by: Joey Pabalinas > This one looks good. Reviewed-by: Benjamin Tissoires Just