xfs
[Top] [All Lists]

Re: [PATCH xfsprogs 2/2] linux.h: Define xfs_off_t as int64_t

To: Christoph Hellwig <hch@xxxxxxxxxxxxx>
Subject: Re: [PATCH xfsprogs 2/2] linux.h: Define xfs_off_t as int64_t
From: Felix Janda <felix.janda@xxxxxxxxx>
Date: Mon, 1 Aug 2016 08:54:10 +0200
Cc: xfs@xxxxxxxxxxx
Delivered-to: xfs@xxxxxxxxxxx
In-reply-to: <20160801062449.GB596@xxxxxxxxxxxxx>
References: <20160730133737.GB5895@nyan> <20160801062449.GB596@xxxxxxxxxxxxx>
User-agent: Mutt/1.6.1 (2016-04-27)
Christoph Hellwig wrote:
> On Sat, Jul 30, 2016 at 03:37:37PM +0200, Felix Janda wrote:
> > int64_t is guaranteed to have the correct size and signedness and is
> > always avaible because linux.h has a <inttypes.h> include.
> > 
> > Fixes compilation error "unkown type name 'off64_t'" on linux when the
> > public header <xfs.h> is included without _LARGEFILE64_SOURCE or
> > _GNU_SOURCE defined. This bug was introduced in commit
> > cb898f157f8410a03cf5f3400baa1df9e5eecd33.
> 
> I would much prefer to just define _LARGEFILE64_SOURCE in linux.h..

Thanks for the suggestion, but that does not work if the system header
defining (or not) off64_t is included before the xfs headers.

---
Felix

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