Skip to content

bpo-39573: Use Py_TYPE() macro in Objects directory - #18392

Merged
vstinner merged 1 commit into
python:masterfrom
vstinner:py_type_objects
Feb 7, 2020
Merged

bpo-39573: Use Py_TYPE() macro in Objects directory#18392
vstinner merged 1 commit into
python:masterfrom
vstinner:py_type_objects

Conversation

@vstinner

@vstinnervstinner commented Feb 7, 2020

Copy link
Copy Markdown
Member

Replace direct access to PyObject.ob_type with Py_TYPE().

https://bugs.python.org/issue39573

Replace direct access to PyObject.ob_type with Py_TYPE().
@codecov

codecovBot commented Feb 7, 2020

Copy link
Copy Markdown

Codecov Report

Merging #18392 into master will decrease coverage by 0.05%.
The diff coverage is n/a.

Impacted file tree graph

@@ Coverage Diff @@## master #18392 +/- ##
==========================================
- Coverage 83.24% 83.19% -0.06% 
==========================================
Files 1570 1570 Lines 414397 414397 Branches 44429 44429 ==========================================
- Hits 344979 344755 -224 - Misses 59767 59995 +228 + Partials 9651 9647 -4 
Impacted FilesCoverage Δ
Lib/distutils/tests/test_bdist_rpm.py30.00% <0.00%> (-65.00%)⬇️
Lib/distutils/command/bdist_rpm.py7.63% <0.00%> (-56.88%)⬇️
Lib/test/test_urllib2net.py76.92% <0.00%> (-13.85%)⬇️
Lib/test/test_smtpnet.py78.57% <0.00%> (-7.15%)⬇️
Lib/ftplib.py63.85% <0.00%> (-6.06%)⬇️
Lib/test/test_ftplib.py87.11% <0.00%> (-4.72%)⬇️
Tools/scripts/db2pickle.py17.82% <0.00%> (-3.97%)⬇️
Tools/scripts/pickle2db.py16.98% <0.00%> (-3.78%)⬇️
Lib/test/test_socket.py71.94% <0.00%> (-3.77%)⬇️
Lib/test/test_asyncio/test_base_events.py91.84% <0.00%> (-3.30%)⬇️
... and 51 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a102ed7...2fdd4cc. Read the comment docs.

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@vstinner@the-knights-who-say-ni@bedevere-bot