Skip to content

Conversation

@hauntsaninja
Copy link
Contributor

@hauntsaninja hauntsaninja commented Feb 22, 2020

@codecov
Copy link

codecov bot commented Feb 22, 2020

Codecov Report

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

Impacted file tree graph

@@            Coverage Diff             @@
##           master   #18602      +/-   ##
==========================================
- Coverage   82.20%   82.06%   -0.14%     
==========================================
  Files        1958     1955       -3     
  Lines      589743   584072    -5671     
  Branches    44457    44458       +1     
==========================================
- Hits       484772   479324    -5448     
+ Misses      95308    95117     -191     
+ Partials     9663     9631      -32     
Impacted Files Coverage Δ
Lib/distutils/tests/test_bdist_rpm.py 30.00% <0.00%> (-65.00%) ⬇️
Lib/distutils/command/bdist_rpm.py 7.63% <0.00%> (-56.88%) ⬇️
Modules/_decimal/libmpdec/umodarith.h 80.76% <0.00%> (-19.24%) ⬇️
Lib/test/test_urllib2net.py 76.92% <0.00%> (-13.85%) ⬇️
Lib/test/test_smtpnet.py 78.57% <0.00%> (-12.86%) ⬇️
Lib/ftplib.py 63.85% <0.00%> (-6.06%) ⬇️
Lib/test/test_ftplib.py 87.11% <0.00%> (-4.72%) ⬇️
Tools/scripts/db2pickle.py 17.82% <0.00%> (-3.97%) ⬇️
Tools/scripts/pickle2db.py 16.98% <0.00%> (-3.78%) ⬇️
Lib/test/test_socket.py 71.94% <0.00%> (-3.87%) ⬇️
... and 336 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 c33bdbb...9a9f2c3. Read the comment docs.

/*[clinic end generated code: output=2fe4ecbd6f3d6844 input=9121d12e66f85b25]*/
{
return PyFile_OpenCodeObject(path);
return PyFile_OpenCodeObject(PyOS_FSPath(path));
Copy link
Contributor

@lazka lazka Feb 22, 2020

Choose a reason for hiding this comment

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

Where does the new reference returned by PyOS_FSPath() get freed?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks, hopefully in this next commit :-)

@tirkarthi
Copy link
Member

io.open_code shouldn't be accepting path-like objects as noted in https://bugs.python.org/msg362522 . I would propose closing this PR and opening another one for documentation update as @zooba suggested.

@hauntsaninja
Copy link
Contributor Author

Sounds good, opened #19824 for the documentation update.

@hauntsaninja hauntsaninja deleted the bpo39723 branch April 30, 2020 23:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants