Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

bpo-42658: Using LCMapStringEx in ntpath.normcase#32010

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
zooba merged 17 commits intopython:mainfromaisk:winapi
Jun 6, 2022
Merged
Changes from1 commit
Commits
Show all changes
17 commits
Select commitHold shift + click to select a range
060cd71
Add _winapi.LCMapStringEx
aiskMar 20, 2022
6ecbc48
Using _winapi.LCMapStringEx in ntpath.normcase
aiskMar 20, 2022
c1429b9
📜🤖 Added by blurb_it.
blurb-it[bot]Mar 20, 2022
81be381
Changes from review
aiskMar 21, 2022
6108d20
Update for comments
aiskMar 21, 2022
3b4a5cb
Update for review and fix tests
aiskMar 22, 2022
ef66941
Update by review comment
aiskMar 23, 2022
6c0366b
Update for review comments
aiskMar 25, 2022
6c6f2e2
fix news
aiskMar 25, 2022
15749b8
Remove constants which can not been used
aiskMar 26, 2022
7b2c62c
Fix ntpath_tests on non Windows systems
aiskMar 26, 2022
a89adc0
Update news
aiskMar 31, 2022
4119cc2
Skip test on block leaks on windows
aiskMar 31, 2022
8a63de2
Fix news
aiskMar 31, 2022
6789cd9
Revert the test modify
aiskApr 9, 2022
d430371
update news
aiskJun 4, 2022
7e0dcf8
update clinic for _winapi
aiskJun 4, 2022
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
PrevPrevious commit
NextNext commit
Skip test on block leaks on windows
  • Loading branch information
@aisk
aisk committedJun 4, 2022
commit4119cc2a86d0326a6b2c82a9358e5547fa6866db
6 changes: 5 additions & 1 deletionLib/test/test_embed.py
View file
Open in desktop
Original file line numberDiff line numberDiff line change
Expand Up@@ -1705,7 +1705,11 @@ def test_no_memleak(self):
blocks = int(match.group(2))
with self.subTest(frozen_modules=flag, stmt=stmt):
self.assertEqual(refs, 0, out)
self.assertEqual(blocks, 0, out)
if not MS_WINDOWS:
self.assertEqual(blocks, 0, out)
else:
# bpo-42658: on Windows, Python still leaks 1 memory block at exit
self.assertEqual(blocks, 1, out)
Copy link
ContributorAuthor

Choose a reason for hiding this comment

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

Found that there is a block leak here, but I think this is not related to this issue (and have no idea how to fix it 😥). Just skipped it for now, but don't know if it's appropriate to do so.

Copy link
Member

Choose a reason for hiding this comment

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

It's not appropriate. If it only occurs in this PR, then we need to count references carefully and find out where it's leaking. If it happens in others as well, then we should make sure someone's looking into it and can probably merge anyway.

Copy link
ContributorAuthor

Choose a reason for hiding this comment

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

Got it.



class StdPrinterTests(EmbeddingTestsMixin, unittest.TestCase):
Expand Down

[8]ページ先頭

©2009-2025 Movatter.jp