Re: [PATCH 9/16] HID: logitech-hidpp: fix error return code

2015-04-05 Thread Jiri Kosina
On Sun, 5 Apr 2015, Julia Lawall wrote: > Return a negative error code on failure. > > A simplified version of the semantic match that finds this problem is as > follows: (http://coccinelle.lip6.fr/) > > // > @@ > identifier ret; expression e1,e2; > @@ > ( > if (\(ret < 0\|ret != 0\)) > { ...

Re: [PATCH 9/16] HID: logitech-hidpp: fix error return code

2015-04-05 Thread Benjamin Tissoires
On Sun, Apr 5, 2015 at 8:06 AM, Julia Lawall wrote: > Return a negative error code on failure. > > A simplified version of the semantic match that finds this problem is as > follows: (http://coccinelle.lip6.fr/) > > // > @@ > identifier ret; expression e1,e2; > @@ > ( > if (\(ret < 0\|ret != 0\))

[PATCH 9/16] HID: logitech-hidpp: fix error return code

2015-04-05 Thread Julia Lawall
Return a negative error code on failure. A simplified version of the semantic match that finds this problem is as follows: (http://coccinelle.lip6.fr/) // @@ identifier ret; expression e1,e2; @@ ( if (\(ret < 0\|ret != 0\)) { ... return ret; } | ret = 0 ) ... when != ret = e1 when != &ret *