search

hroptatyr / dateutils

nifty command line date and time utilities; fast date calculations and conversion in the shell
http://www.fresse.org/dateutils/
Other
616 stars 42 forks source link

printf type mismatch #150

Closed dmacks closed 1 year ago

dmacks commented 1 year ago

Building 0.4.10 on OS X 10.13:

tzraw.c:705:42: warning: format specifies type 'long' but the argument has type 'stamp_t' (aka 'long long') [-Wformat]
                        printf("    tr[%zu]\t%ld\t%hhu\n", j, z->trs[j], z->tys[j]);
                                             ~~~              ^~~~~~~~~
                                             %lld

stamp_t is declared as int64_t

hroptatyr commented 1 year ago

Huh, I didn't close this. But I tried to fix it in 296777f.

dmacks commented 1 year ago

Works for me (obviously). The auto-closing of issues based on commit-messages is a neat feature. See https://docs.github.com/en/issues/tracking-your-work-with-issues/linking-a-pull-request-to-an-issue

hroptatyr commented 1 year ago

Oh, I see. Must be careful with the wording then. Thanks for the confirmation, and obviously thanks for the initial report.