| 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> |
|---|---|---|
| ||
| Previous by Date: | Re: [PATCH 10/47] xfs: refactor btree owner change into a separate visit-blocks function, Christoph Hellwig |
|---|---|
| Next by Date: | Re: [PATCH 11/47] xfs: move deferred operations into a separate file, Christoph Hellwig |
| Previous by Thread: | Re: [PATCH xfsprogs 2/2] linux.h: Define xfs_off_t as int64_t, Christoph Hellwig |
| Next by Thread: | Re: [PATCH xfsprogs 2/2] linux.h: Define xfs_off_t as int64_t, Dave Chinner |
| Indexes: | [Date] [Thread] [Top] [All Lists] |