From mboxrd@z Thu Jan 1 00:00:00 1970 From: J. Bruce Fields Date: Tue, 4 Jun 2013 16:17:43 -0400 Subject: [Cluster-devel] [PATCH v1 04/11] locks: make "added" in __posix_lock_file a bool In-Reply-To: <1370056054-25449-5-git-send-email-jlayton@redhat.com> References: <1370056054-25449-1-git-send-email-jlayton@redhat.com> <1370056054-25449-5-git-send-email-jlayton@redhat.com> Message-ID: <20130604201743.GA15594@fieldses.org> List-Id: To: cluster-devel.redhat.com MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit On Fri, May 31, 2013 at 11:07:27PM -0400, Jeff Layton wrote: > ...save 3 bytes of stack space. > > Signed-off-by: Jeff Layton ACK.--b. > --- > fs/locks.c | 9 +++++---- > 1 files changed, 5 insertions(+), 4 deletions(-) > > diff --git a/fs/locks.c b/fs/locks.c > index a7d2253..cef0e04 100644 > --- a/fs/locks.c > +++ b/fs/locks.c > @@ -800,7 +800,8 @@ static int __posix_lock_file(struct inode *inode, struct file_lock *request, str > struct file_lock *left = NULL; > struct file_lock *right = NULL; > struct file_lock **before; > - int error, added = 0; > + int error; > + bool added = false; > > /* > * We may need two file_lock structures for this operation, > @@ -894,7 +895,7 @@ static int __posix_lock_file(struct inode *inode, struct file_lock *request, str > continue; > } > request = fl; > - added = 1; > + added = true; > } > else { > /* Processing for different lock types is a bit > @@ -905,7 +906,7 @@ static int __posix_lock_file(struct inode *inode, struct file_lock *request, str > if (fl->fl_start > request->fl_end) > break; > if (request->fl_type == F_UNLCK) > - added = 1; > + added = true; > if (fl->fl_start < request->fl_start) > left = fl; > /* If the next lock in the list has a higher end > @@ -935,7 +936,7 @@ static int __posix_lock_file(struct inode *inode, struct file_lock *request, str > locks_release_private(fl); > locks_copy_private(fl, request); > request = fl; > - added = 1; > + added = true; > } > } > /* Go on to next lock. > -- > 1.7.1 >