curl / Mailing Lists / curl-library / Single Mail

curl-library

Re: [PATCH v2] timeval.c: Use long long constant type for timeval assignment

From: Daniel Stenberg <daniel_at_haxx.se>
Date: Mon, 3 Jul 2017 13:31:23 +0200 (CEST)

On Mon, 3 Jul 2017, Martin Kepplinger wrote:

> Thanks Daniel, indeed we have to use signed values. sparse doesn't complain
> on my 32 bit machine, but I still think using long long is even more safe
> here.

Thanks. I made a pull-request out of it so that it gets a round of CI tests
done before we merge: https://github.com/curl/curl/pull/1636

If Windows use 64 bit time_t, I fear the LL suffixes will cause problems...

-- 
  / daniel.haxx.se
-------------------------------------------------------------------
Unsubscribe: https://cool.haxx.se/list/listinfo/curl-library
Etiquette:   https://curl.haxx.se/mail/etiquette.html
Received on 2017-07-03