Skip to content

Replace cdll with CDLL and windll with WinDLL#36

Merged
moi15moi merged 1 commit intomainfrom
replace-cdll-with-CDLL-and-windll-with-WinDLL
Oct 15, 2025
Merged

Replace cdll with CDLL and windll with WinDLL#36
moi15moi merged 1 commit intomainfrom
replace-cdll-with-CDLL-and-windll-with-WinDLL

Conversation

@moi15moi
Copy link
Owner

As noted in this comment, it is preferable to use ctypes.WinDLL instead of ctypes.windll. This approach avoids modifying shared DLL objects and ensures better isolation.

As noted in [this comment](python/cpython#125788 (comment)), it is preferable to use `ctypes.WinDLL` instead of `ctypes.windll`. This approach avoids modifying shared DLL objects and ensures better isolation.
@codecov-commenter
Copy link

codecov-commenter commented Oct 15, 2025

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 95.10%. Comparing base (1676644) to head (ecdd82e).
❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #36   +/-   ##
=======================================
  Coverage   95.10%   95.10%           
=======================================
  Files          23       23           
  Lines        1001     1001           
=======================================
  Hits          952      952           
  Misses         49       49           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@moi15moi moi15moi merged commit 781147e into main Oct 15, 2025
40 checks passed
@moi15moi moi15moi deleted the replace-cdll-with-CDLL-and-windll-with-WinDLL branch October 15, 2025 00:50
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.

2 participants