* [PATCH 1/1] Staging: comedi: cd_pcidas64.c: Compression of
@ 2016-08-02 18:16 Nadim almas
2016-08-21 14:57 ` Greg KH
0 siblings, 1 reply; 2+ messages in thread
From: Nadim almas @ 2016-08-02 18:16 UTC (permalink / raw)
To: abbotti; +Cc: gregkh, linux-kernel, Nadim almas
This patch compresses two lines in to a single line in file
cb_pcidas64.c
if immediate return statement is found.
It is done using script Coccinelle. And coccinelle uses following
semantic patch for this compression function:
@@
expression e, ret;
@@
-ret =
+return
e;
-return ret
Signed-off-by: Nadim Almas <nadim.902@gmail.com>
---
drivers/staging/comedi/drivers/cb_pcidas64.c | 4 +---
1 files changed, 1 insertions(+), 3 deletions(-)
diff --git a/drivers/staging/comedi/drivers/cb_pcidas64.c b/drivers/staging/comedi/drivers/cb_pcidas64.c
index 1f9c08a..d30cf3d 100644
--- a/drivers/staging/comedi/drivers/cb_pcidas64.c
+++ b/drivers/staging/comedi/drivers/cb_pcidas64.c
@@ -1408,9 +1408,7 @@ static int set_ai_fifo_size(struct comedi_device *dev, unsigned int num_samples)
if (retval < 0)
return retval;
- num_samples = retval * fifo->num_segments * fifo->sample_packing_ratio;
-
- return num_samples;
+ return retval * fifo->num_segments * fifo->sample_packing_ratio;
}
/* query length of fifo */
--
2.7.4
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH 1/1] Staging: comedi: cd_pcidas64.c: Compression of
2016-08-02 18:16 [PATCH 1/1] Staging: comedi: cd_pcidas64.c: Compression of Nadim almas
@ 2016-08-21 14:57 ` Greg KH
0 siblings, 0 replies; 2+ messages in thread
From: Greg KH @ 2016-08-21 14:57 UTC (permalink / raw)
To: Nadim almas; +Cc: abbotti, linux-kernel
On Tue, Aug 02, 2016 at 11:16:04AM -0700, Nadim almas wrote:
> This patch compresses two lines in to a single line in file
> cb_pcidas64.c
> if immediate return statement is found.
>
> It is done using script Coccinelle. And coccinelle uses following
> semantic patch for this compression function:
>
> @@
> expression e, ret;
> @@
>
> -ret =
> +return
> e;
> -return ret
>
> Signed-off-by: Nadim Almas <nadim.902@gmail.com>
> ---
>
> drivers/staging/comedi/drivers/cb_pcidas64.c | 4 +---
> 1 files changed, 1 insertions(+), 3 deletions(-)
>
Subject is still quite odd :(
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2016-08-21 21:15 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-08-02 18:16 [PATCH 1/1] Staging: comedi: cd_pcidas64.c: Compression of Nadim almas
2016-08-21 14:57 ` Greg KH
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).