Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757180AbcLOERF (ORCPT ); Wed, 14 Dec 2016 23:17:05 -0500 Received: from sandeen.net ([63.231.237.45]:53626 "EHLO sandeen.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1757056AbcLOERD (ORCPT ); Wed, 14 Dec 2016 23:17:03 -0500 Subject: Re: [PATCH 1/1 linux-next] xfs: remove unnecessary return To: Fabian Frederick , Dave Chinner , Eric Sandeen References: <1481352729-7020-1-git-send-email-fabf@skynet.be> Cc: linux-xfs@vger.kernel.org, linux-kernel@vger.kernel.org From: Eric Sandeen Message-ID: <7ce30624-de82-f682-776c-7b6f1757026d@sandeen.net> Date: Wed, 14 Dec 2016 22:17:02 -0600 User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.11; rv:45.0) Gecko/20100101 Thunderbird/45.5.1 MIME-Version: 1.0 In-Reply-To: <1481352729-7020-1-git-send-email-fabf@skynet.be> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 903 Lines: 38 On 12/10/16 12:52 AM, Fabian Frederick wrote: > Commit f7a136aee3c1 > ("xfs: several xattr functions can be void") > > updated 2 end of function return 0 to return in void > functions. Remove it. > > Signed-off-by: Fabian Frederick Oh, sure. :) Reviewed-by: Eric Sandeen > --- > fs/xfs/xfs_attr_list.c | 2 -- > 1 file changed, 2 deletions(-) > > diff --git a/fs/xfs/xfs_attr_list.c b/fs/xfs/xfs_attr_list.c > index 97c45b6..5aa46c3 100644 > --- a/fs/xfs/xfs_attr_list.c > +++ b/fs/xfs/xfs_attr_list.c > @@ -433,7 +433,6 @@ xfs_attr3_leaf_list_int( > cursor->offset++; > } > trace_xfs_attr_list_leaf_end(context); > - return; > } > > /* > @@ -543,7 +542,6 @@ xfs_attr_put_listent( > alist->al_offset[context->count++] = context->firstu; > alist->al_count = context->count; > trace_xfs_attr_list_add(context); > - return; > } > > /* >