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 -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
