Collin Funk wrote:
> The second patch adjusts the tests to use posixtime_z (NULL, ...),
> instead of setting the TZ environment variable.

That is problematic. It removes the only unit test of the posixtime
function, and thus also makes it a mystery how to call the posixtime
function correctly.

Instead of removing the posixtime's tests, can you instead arrange to
have two unit test functions: one that demonstrates and tests posixtime_z,
and one that demonstrates and tests posixtime?

Bruno




Reply via email to