Re: [PATCH 4/4] usbip: remove unneeded structure

From: Valentina Manea
Date: Mon Dec 01 2014 - 03:39:27 EST


On Sun, Nov 30, 2014 at 8:14 PM, Julia Lawall <Julia.Lawall@xxxxxxx> wrote:
> From: Julia Lawall <Julia.Lawall@xxxxxxx>
>
> Delete a local structure that is only used to be initialized by memset.
>
> A semantic patch that makes this change is as follows:
> (http://coccinelle.lip6.fr/)
>
> // <smpl>
> @@
> identifier x,i;
> @@
>
> {
> ... when any
> -struct i x;
> <+... when != x
> - memset(&x,...);
> ...+>
> }
> // </smpl>
>
> Signed-off-by: Julia Lawall <Julia.Lawall@xxxxxxx>
>
> ---
> tools/usb/usbip/src/usbipd.c | 2 --
> 1 file changed, 2 deletions(-)
>
> diff --git a/tools/usb/usbip/src/usbipd.c b/tools/usb/usbip/src/usbipd.c
> index 2f87f2d..2a7cd2b 100644
> --- a/tools/usb/usbip/src/usbipd.c
> +++ b/tools/usb/usbip/src/usbipd.c
> @@ -91,7 +91,6 @@ static void usbipd_help(void)
> static int recv_request_import(int sockfd)
> {
> struct op_import_request req;
> - struct op_common reply;
> struct usbip_exported_device *edev;
> struct usbip_usb_device pdu_udev;
> struct list_head *i;
> @@ -100,7 +99,6 @@ static int recv_request_import(int sockfd)
> int rc;
>
> memset(&req, 0, sizeof(req));
> - memset(&reply, 0, sizeof(reply));
>
> rc = usbip_net_recv(sockfd, &req, sizeof(req));
> if (rc < 0) {
>

Acked-by: Valentina Manea <valentina.manea.m@xxxxxxxxx>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/