[Tile-serving] [osm2pgsql-dev/osm2pgsql] Replace fmt::localtime with localtime_r (PR #2457)
Jochen Topf
notifications at github.com
Sat Apr 4 15:09:19 UTC 2026
joto left a comment (osm2pgsql-dev/osm2pgsql#2457)
https://en.cppreference.com/w/c/chrono/localtime.html says `localtime_r` is C23, but the man page says its in POSIX.1-1196 and SUSv2, so that's plenty old enough for our use case. But [it doesn't look like MSVC supports it](https://learn.microsoft.com/en-us/cpp/c-runtime-library/reference/localtime-localtime32-localtime64). And, in fact, the code in `fmt` is much more complicated than in your commit. Looks like we need to use `localtime_s` for Windows.
--
Reply to this email directly or view it on GitHub:
https://github.com/osm2pgsql-dev/osm2pgsql/pull/2457#issuecomment-4187249866
You are receiving this because you are subscribed to this thread.
Message ID: <osm2pgsql-dev/osm2pgsql/pull/2457/c4187249866 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/tile-serving/attachments/20260404/dac563fa/attachment.htm>
More information about the Tile-serving
mailing list