[PATCHv2 2/2] mkfs.jffs2: fix casting of __off64_t
Mike Frysinger
vapier.adi at gmail.com
Mon Apr 18 17:33:14 EDT 2011
On Mon, Apr 18, 2011 at 11:20, Andy Shevchenko wrote:
> On Mon, 2011-04-18 at 10:57 -0400, ext Mike Frysinger wrote:
>> so let's go with an explicitly 64bit printf modifier.
>
> Regarding to above I couldn't see any objection against %jd. At least
> one for PRIu64 - it looks awful in code.
it doesnt make sense to me to pair a non-explicit sized format string
with an explicit sized type. off64_t is exactly 64bits, and PRIu64
prints exactly 64bits.
that said, %ju is at least an improvement over the current code as
well as the patch you've posted (doesnt seem to need casts on 32 or 64
bit systems). so if Artem is OK with that, i wont fight it. even
though it's still not as correct as my proposal :P.
-mike
More information about the linux-mtd
mailing list