Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
Message-ID: <1516267392.24576.246.camel@haakon3.daterainc.com>
Subject: Re: [PATCH] tcmu: Fix trailing semicolon
From:   "Nicholas A. Bellinger" <nab@linux-iscsi.org>
To:     mchristi@redhat.com
Cc:     Luis de Bethencourt <luisbg@kernel.org>,
        linux-kernel@vger.kernel.org, Joe Perches <joe@perches.com>,
        linux-scsi@vger.kernel.org, target-devel@vger.kernel.org
Date:   Thu, 18 Jan 2018 01:23:12 -0800
In-Reply-To: <f188423f-8d53-ffe8-7d85-48424e15411b@redhat.com>
References: <20180116153432.17927-1-luisbg@kernel.org>
         <f188423f-8d53-ffe8-7d85-48424e15411b@redhat.com>
Content-Type: text/plain; charset="UTF-8"
Mime-Version: 1.0
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

On Tue, 2018-01-16 at 10:25 -0600, Michael Christie wrote:
> On 01/16/2018 09:34 AM, Luis de Bethencourt wrote:
> > The trailing semicolon is an empty statement that does no operation.
> > It is completely stripped out by the compiler. Removing it since it doesn't do
> > anything.
> > 
> > Signed-off-by: Luis de Bethencourt <luisbg@kernel.org>
> > ---
> > 
> > Hi,
> > 
> > After fixing the same thing in drivers/staging/rtl8723bs/, Joe Perches
> > suggested I fix it treewide [0].
> > 
> > Best regards 
> > Luis
> > 
> > 
> > [0] http://driverdev.linuxdriverproject.org/pipermail/driverdev-devel/2018-January/115410.html
> > [1] http://driverdev.linuxdriverproject.org/pipermail/driverdev-devel/2018-January/115390.html
> > 
> >  drivers/target/target_core_user.c | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> > 

Applied.  Thanks Luis + MNC.