[Fontconfig] [PATCH] Please make the output of cache files reproducible
Akira TAGOH
akira at tagoh.org
Mon May 14 11:14:44 UTC 2018
not sure what it should be named but the curly brackets is supposed to
be started at a new line, need a space after a function name etc.
Aside from that:
+static int64_t
+FcDirChecksumNano (struct stat *statb) {
+ /* No nanosecond component to parse */
+ if (getenv("SOURCE_DATE_EPOCH"))
+ return 0;
+#ifdef HAVE_STRUCT_STAT_ST_MTIM
+ return statb->st_mtim.tv_nsec;
+#endif
+ return 0;
+}
good to move the last "return 0" line into #else statement for
HAVE_STRUCT_STAT_ST_MTIM. also if the platforms doesn't support
st_mtim in stat structure, no need to check the env var even. it can
be entered into #ifdef too.
HTH,
On Mon, May 14, 2018 at 7:55 PM, Chris Lamb <lamby at debian.org> wrote:
> Chris Lamb wrote:
>
>> > Please follow up on the coding styles and write a document for this behavior.
>>
>> Thank you for your quick reply. Can you clarify on the coding style
>> angle? I tried to match what was already in the surrounding code.
>
> Gentle ping on this? :)
>
>
> Regards,
>
> --
> ,''`.
> : :' : Chris Lamb
> `. `'` lamby at debian.org / chris-lamb.co.uk
> `-
--
Akira TAGOH
More information about the Fontconfig
mailing list