From: Linus Torvalds <torvalds@linux-foundation.org>
Date: Sun, 28 Oct 2007 17:09:10 GMT
Subject: Re: [RFH] gcc constant expression warning...
Message-ID: <alpine.LFD.0.999.0710281000260.30120@woody.linux-foundation.org>
In-Reply-To: <slrnfi8pj7.mb4.antti-juhani@kukkaseppele.kaijanaho.fi>



On Sun, 28 Oct 2007, Antti-Juhani Kaijanaho wrote:
>
> A correct fix would be to check for the size of off_t in some other (and
> defined) manner, but I don't know off_t well enough to suggest one.

In this case, it's trying to make sense that "off_t" can hold more than 32
bits. So I think that test can just be rewritten as

	if (sizeof(off_t) <= 4) {
		munmap(idx_map, idx_size);
		return error("pack too large for current definition of off_t in %s", path);
	}

instead.

			Linus