FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

bpo-34419: selectmodule.c does not compile on HP-UX due to bpo-31938 by michael-o · Pull Request #8796 · python/cpython · GitHub

/ cpython Public

bpo-34419: selectmodule.c does not compile on HP-UX due to bpo-31938 - #8796

Merged
taleinat merged 1 commit into
python:masterfrom
michael-o:bpo-34419
Aug 17, 2018
Merged

bpo-34419: selectmodule.c does not compile on HP-UX due to bpo-31938#8796
taleinat merged 1 commit into
python:masterfrom
michael-o:bpo-34419

Conversation

michael-o commented Aug 17, 2018
edited by bedevere-bot
Loading

Copy link
Copy Markdown
Contributor

Fix compile errors reported by HP aCC by changing oversights from 6dc57e2
which do not cause trouble on clang or GCC.

Patch by Michael Osipov.

Please see for a detailed description the bug report.

https://bugs.python.org/issue34419

Copy link
Copy Markdown
Contributor Author

If absolutely necessary, I can add #ifndef __hpux.

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.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

I think this is just a cleanup after bpo-31938, so we can skip adding a NEWS entry.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

@berkerpeksag Done, file dropped.

Fix compile errors reported by HP aCC by changing oversights from 6dc57e2
which do not cause trouble on clang or GCC.

Patch by Michael Osipov.

taleinat left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

This fixes two obvious bugs. Good catches in both cases!

LGTM

taleinat merged commit 0e6e7a1 into python:master Aug 17, 2018
michael-o deleted the bpo-34419 branch August 17, 2018 11:46
carljm added a commit to carljm/cpython that referenced this pull request Aug 19, 2018
* master: (107 commits)
  bpo-22057: Clarify eval() documentation (pythonGH-8812)
  bpo-34318: Convert deprecation warnings to errors in assertRaises() etc. (pythonGH-8623)
  bpo-22602: Raise an exception in the UTF-7 decoder for ill-formed sequences starting with "+". (pythonGH-8741)
  bpo-34415: Updated logging.Formatter docstring. (pythonGH-8811)
  bpo-34432: doc Mention complex and decimal.Decimal on str.format not about locales (pythonGH-8808)
  bpo-34381: refer to 'Running & Writing Tests' in README.rst (pythonGH-8797)
  Improve error message when mock.assert_has_calls fails (pythonGH-8205)
  Warn not to set SIGPIPE to SIG_DFL (python#6773)
  bpo-34419: selectmodule.c does not compile on HP-UX due to bpo-31938 (pythonGH-8796)
  bpo-34418: Fix HTTPErrorProcessor documentation (pythonGH-8793)
  bpo-34391: Fix ftplib test for TLS 1.3 (pythonGH-8787)
  bpo-34217: Use lowercase for windows headers (pythonGH-8472)
  bpo-34395: Fix memory leaks caused by incautious usage of PyMem_Resize(). (pythonGH-8756)
  bpo-34405: Updated to OpenSSL 1.1.0i for Windows builds. (pythonGH-8775)
  bpo-34384: Fix os.readlink() on Windows (pythonGH-8740)
  closes bpo-34400: Fix undefined behavior in parsetok(). (pythonGH-4439)
  bpo-34399: 2048 bits RSA keys and DH params (python#8762)
  Make regular expressions in test_tasks.py raw strings. (pythonGH-8759)
  smtplib documentation fixes (pythonGH-8708)
  Fix misindented yaml in logging how to example (pythonGH-8604)
  ...
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to 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.

5 participants


Back | FazBrowse Home | New Git URL