Skip to content

gh-105751: Remove dead code in test_ctypes - #105814

Closed
vstinner wants to merge 2 commits into
python:mainfrom
vstinner:test_ctypes_dead_code
Closed

gh-105751: Remove dead code in test_ctypes#105814
vstinner wants to merge 2 commits into
python:mainfrom
vstinner:test_ctypes_dead_code

Conversation

@vstinner

@vstinnervstinner commented Jun 15, 2023

Copy link
Copy Markdown
Member
  • Remove "except: print(tp); raise" debug code.
  • Remove unused NoNullHandle() function.
  • Remove commented code.

* Remove "except: print(tp); raise" debug code.
* Remove unused NoNullHandle() function.
* Remove commented code.
@bedevere-botbedevere-bot added awaiting core review tests Tests in the Lib/test dir labels Jun 15, 2023
@bedevere-botbedevere-bot mentioned this pull request Jun 14, 2023
Reenable 3 tests:
* test_overflow()
* test_basic_wstrings()
* test_toolong()
Changes:
* Replace sys.maxint with sys.maxsize
* Replace c_wstring() with create_unicode_buffer()
* Update issue number
@vstinner
vstinner deleted the test_ctypes_dead_code branch June 15, 2023 09:04
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting core reviewskip newstestsTests in the Lib/test dir

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@vstinner@bedevere-bot