xfs
[Top] [All Lists]

Re: [PATCH v2] xfs: test attr_list_by_handle cursor iteration

To: Christoph Hellwig <hch@xxxxxxxxxxxxx>
Subject: Re: [PATCH v2] xfs: test attr_list_by_handle cursor iteration
From: Eryu Guan <eguan@xxxxxxxxxx>
Date: Fri, 12 Aug 2016 11:14:00 +0800
Cc: "Darrick J. Wong" <darrick.wong@xxxxxxxxxx>, david@xxxxxxxxxxxxx, linux-btrfs@xxxxxxxxxxxxxxx, fstests@xxxxxxxxxxxxxxx, xfs@xxxxxxxxxxx
Delivered-to: xfs@xxxxxxxxxxx
In-reply-to: <20160811174855.GD31540@xxxxxxxxxxxxx>
References: <20160802235251.GB8586@xxxxxxxxxxxxxxxx> <20160803225150.GC25565@xxxxxxxxxxxxxxxx> <20160804042448.GR27776@xxxxxxxxxxxxxxxxxxxxxxxx> <20160811174855.GD31540@xxxxxxxxxxxxx>
User-agent: Mutt/1.6.1 (2016-04-27)
On Thu, Aug 11, 2016 at 10:48:55AM -0700, Christoph Hellwig wrote:
> On Thu, Aug 04, 2016 at 12:24:48PM +0800, Eryu Guan wrote:
> > I have to replace false/true with 0/1 to build it on RHEL6. I can fix it
> > at commit time if there's no other major updates.
> 
> Simply including <stdbool.h> would fix that on RHEL6, wouldn't it?

Yes, including <stdbool.h> fixes the build failure on RHEL6, another
option is to replace all bool with int, as mentioned in my first review
email. And Darric chose the "int" way in v2 but only updated the
function definition and forgot to update the callers (still passing
true/false, not 1/0).

Thanks for the review!

Eryu

<Prev in Thread] Current Thread [Next in Thread>