Re: [PATCH 07/12] e2fsck: track ea_inode references

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



> On Jun 26, 2017, at 7:25 PM, Tahsin Erdogan <tahsin@xxxxxxxxxx> wrote:
> 
>>> +#ifdef EXT2_NO_64_TYPE
>>> +             fprintf(f, "%*u", width, ctx->num2);
>>> +#else
>>> +             fprintf(f, "%*llu", width, (long long)ctx->num2);
>>> #endif
>> 
>> Rather than a series of "#ifdef EXT2_NO_64_TYPE" checks, it would be cleaner
>> to have a single #define to set the printf type, like:
>> 
>> #ifdef EXT2_NO_64_TYPE
>> #define EXT2_64U "%*u"
>> #define EXT2_64D "%*d"
>> #define EXT2_64X "%*x"
>> #else
>> #define EXT2_64U "%*llu"
>> #define EXT2_64D "%*lld"
>> #define EXT2_64X "%*llx"
>> #endif
> 
> I am trying to figure out the purpose of #ifdef EXT2_NO_64_TYPE checks
> now. Who defines it and what problem does it solve?

Sorry, I can't answer that.  Git history shows that it has been around since
2000 but is not set by the configure script.  I just thought it made sense to
clean up the code, but it might be even better to get rid of it completely.

Cheers, Andreas





Attachment: signature.asc
Description: Message signed with OpenPGP


[Index of Archives]     [Reiser Filesystem Development]     [Ceph FS]     [Kernel Newbies]     [Security]     [Netfilter]     [Bugtraq]     [Linux FS]     [Yosemite National Park]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Device Mapper]     [Linux Media]

  Powered by Linux