Skip to content

bpo-28667: Fix a compile warning on FreeBSD when compare with FD_SETS…#501

Merged
serhiy-storchaka merged 1 commit into
python:masterfrom
serhiy-storchaka:bpo-28667-FD_SETSIZE
Mar 12, 2017
Merged

bpo-28667: Fix a compile warning on FreeBSD when compare with FD_SETS…#501
serhiy-storchaka merged 1 commit into
python:masterfrom
serhiy-storchaka:bpo-28667-FD_SETSIZE

Conversation

@serhiy-storchaka

Copy link
Copy Markdown
Member

…IZE.

FreeBSD is the only platforms with unsigned FD_SETSIZE.

…IZE.

FreeBSD is the only platforms with unsigned FD_SETSIZE.
@mention-bot

Copy link
Copy Markdown

@serhiy-storchaka, thanks for your PR! By analyzing the history of the files in this pull request, we identified @warsaw, @theller, @tiran, @Haypo and @akuchling to be potential reviewers.

@skrah

skrah commented Mar 12, 2017

Copy link
Copy Markdown
Contributor

I don't have a FreeBSD VM at the moment, so I've removed myself from the reviewers.

@serhiy-storchaka
serhiy-storchaka merged commit 783d0c1 into python:master Mar 12, 2017
@serhiy-storchaka
serhiy-storchaka deleted the bpo-28667-FD_SETSIZE branch March 12, 2017 12:43
@vstinner

Copy link
Copy Markdown
Member

@serhiy-storchaka : Why did you close and then immediately reopen the PR two times? You also closed and then reopened PR #569 for example.

@serhiy-storchaka

Copy link
Copy Markdown
Member Author

This is an easy way to rerun tests.

vstinner added a commit that referenced this pull request Aug 22, 2017
…IZE. (#501) (#3190)

FreeBSD is the only platforms with unsigned FD_SETSIZE.

(cherry picked from commit 783d0c1)
jaraco pushed a commit that referenced this pull request Dec 2, 2022
Co-authored-by: Mariatta Wijaya <Mariatta@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants