Re: [PATCH 22/49] xfs: minor cleanups

Dave Chinner
2013-07-28T01:07:53Z

On Sat, Jul 27, 2013 at 01:00:42PM -0500, Mark Tinguely wrote:
> On 07/19/13 01:24, Dave Chinner wrote:
> >From: Dave Chinner<dchinner@redhat.com>
> >
> >These come from syncing the shared userspace and kernel code. Small
> >whitespace and trivial cleanups.
> >
> >Signed-off-by: Dave Chinner<dchinner@redhat.com>
> >---
>
> okay, minor clean-ups
>
> Don't understand the change from min_t to min:
>
> >diff --git a/fs/xfs/xfs_attr_remote.c b/fs/xfs/xfs_attr_remote.c
> >index 13a0ed9..b42926a 100644
> >--- a/fs/xfs/xfs_attr_remote.c
> >+++ b/fs/xfs/xfs_attr_remote.c
> >@@ -251,7 +251,7 @@ xfs_attr_rmtval_copyout(
> > int hdr_size = 0;
> > int byte_cnt = XFS_ATTR3_RMT_BUF_SPACE(mp, XFS_LBSIZE(mp));
> >
> >- byte_cnt = min_t(int, *valuelen, byte_cnt);
> >+ byte_cnt = min(*valuelen, byte_cnt);

min_t is only needed when the types being compared are different.
I.e. it ensures the comparison is done with the appropriate casts on
the items being compared. However, both variables being compared
here are of the same type (int) and so the type casting of min_t is
not necessary.

Cheers,

Dave.
--
Dave Chinner
david@fromorbit.com