Skip to content

[3.7] Fix _PyTime_MIN/MAX values (GH-15384)#15425

Merged
miss-islington merged 1 commit into
python:3.7from
miss-islington:backport-8e76c45-3.7
Aug 23, 2019
Merged

[3.7] Fix _PyTime_MIN/MAX values (GH-15384)#15425
miss-islington merged 1 commit into
python:3.7from
miss-islington:backport-8e76c45-3.7

Conversation

@miss-islington

Copy link
Copy Markdown
Contributor

_PyTime_t type is defined as int64_t, and so min/max are INT64_MIN/INT64_MAX,
not PY_LLONG_MIN/PY_LLONG_MAX.
(cherry picked from commit 8e76c45)

Co-authored-by: Sergey Fedoseev fedoseev.sergey@gmail.com

_PyTime_t type is defined as int64_t, and so min/max are INT64_MIN/INT64_MAX,
not PY_LLONG_MIN/PY_LLONG_MAX.
(cherry picked from commit 8e76c45)

Co-authored-by: Sergey Fedoseev <fedoseev.sergey@gmail.com>

@vstinner vstinner left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, good bot.

@miss-islington

Copy link
Copy Markdown
Contributor Author

@sir-sigurd and @vstinner: Status check is done, and it's a success ✅ .

@miss-islington miss-islington merged commit 8fac472 into python:3.7 Aug 23, 2019
@miss-islington miss-islington deleted the backport-8e76c45-3.7 branch August 23, 2019 15:56
@miss-islington

Copy link
Copy Markdown
Contributor Author

@sir-sigurd and @vstinner: Status check is done, and it's a success ✅ .

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip issue skip news type-feature A feature request or enhancement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants