Skip to content

Fix GH-20603 issue on windows 32 bits. - #20634

Closed
devnexen wants to merge 2 commits into
php:PHP-8.3from
devnexen:gh20601-fix
Closed

Fix GH-20603 issue on windows 32 bits.#20634
devnexen wants to merge 2 commits into
php:PHP-8.3from
devnexen:gh20601-fix

Conversation

@devnexen

Copy link
Copy Markdown
Member

the timeout needed to be unsigned.

the timeout needed to be unsigned.

@ndosschendossche 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.

This ain't enough I think.

#definePHP_TIMEOUT_ULL_MAX UINT64_MAX

This is defined as a 64-bit number. You'll have to either use a uint64_t or a double to compare. Probably the former. (which btw means that the check won't do anything on 32-bits)

@devnexen

Copy link
Copy Markdown
MemberAuthor

ah yes zend_ulong is uint32_t in 32 bits case..

@ndosschendossche 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.

This looks right, although I wonder why you even need the double case and double division instead of using integer division?

edorian pushed a commit that referenced this pull request Dec 9, 2025
the timeout needed to be unsigned.
closeGH-20634
bukka pushed a commit that referenced this pull request Dec 12, 2025
the timeout needed to be unsigned.
closeGH-20634
(cherry picked from commit ff51ac1)
bukka pushed a commit that referenced this pull request Dec 12, 2025
the timeout needed to be unsigned.
closeGH-20634
(cherry picked from commit ff51ac1)
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@devnexen@ndossche