Steve Dower added the comment:

Any reason we can't make gmtoff a time_t instead of an int?

If we're going to truncate values to get rid of the warnings, I'd like to also 
see either proof that it will never exceed the size of an int (which may be a 
simple comment, but it's not obvious that this is the case from what appears in 
the patch), or an assertion when it does overflow.

But since we're presumably passing the value back into Python as an int, 
expanding the destination variable to fit all possible values is best.

----------

_______________________________________
Python tracker <rep...@bugs.python.org>
<http://bugs.python.org/issue29063>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

Reply via email to