Skip to content

bpo-39573: Use Py_TYPE() macro in object.c - #18398

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

bpo-39573: Use Py_TYPE() macro in object.c#18398
vstinner merged 1 commit into
python:masterfrom
vstinner:py_type

Conversation

@vstinner

@vstinnervstinner commented Feb 7, 2020

Copy link
Copy Markdown
Member

Replace direct acccess to PyVarObject.ob_size with usage of the
Py_SIZE() macro.

https://bugs.python.org/issue39573

Replace direct acccess to PyVarObject.ob_size with usage of the
Py_SIZE() macro.
@codecov

codecovBot commented Feb 7, 2020

Copy link
Copy Markdown

Codecov Report

Merging #18398 into master will increase coverage by 1.08%.
The diff coverage is n/a.

Impacted file tree graph

@@ Coverage Diff @@## master #18398 +/- ##
===========================================
+ Coverage 82.11% 83.19% +1.08% 
===========================================
Files 1955 1570 -385 Lines 588906 414397 -174509 Branches 44429 44429 ===========================================
- Hits 483572 344754 -138818 + Misses 95679 59996 -35683 + Partials 9655 9647 -8 
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 435 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 bec4186...832b38e. 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