Skip to content

bpo-39646: Remove compile warnings in unicodeobject.c#18519

Merged
vstinner merged 1 commit intopython:masterfrom
shihai1991:bpo_39646
Feb 17, 2020
Merged

bpo-39646: Remove compile warnings in unicodeobject.c#18519
vstinner merged 1 commit intopython:masterfrom
shihai1991:bpo_39646

Conversation

@shihai1991
Copy link
Copy Markdown
Member

@shihai1991 shihai1991 commented Feb 16, 2020

@codecov
Copy link
Copy Markdown

codecov Bot commented Feb 16, 2020

Codecov Report

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

Impacted file tree graph

@@             Coverage Diff             @@
##           master   #18519       +/-   ##
===========================================
+ Coverage   79.46%   83.20%    +3.73%     
===========================================
  Files         384     1571     +1187     
  Lines      169290   414730   +245440     
  Branches        0    44456    +44456     
===========================================
+ Hits       134529   345062   +210533     
- Misses      34761    60021    +25260     
- Partials        0     9647     +9647     
Impacted Files Coverage Δ
Objects/stringlib/transmogrify.h
Parser/acceler.c
Objects/stringlib/localeutil.h
Objects/stringlib/unicode_format.h
Modules/_curses_panel.c
Modules/_decimal/libmpdec/transpose.h
Python/clinic/_warnings.c.h
Modules/_heapqmodule.c
Modules/_testmultiphase.c
Modules/_threadmodule.c
... and 1944 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 f632736...30bab85. Read the comment docs.

@shihai1991
Copy link
Copy Markdown
Member Author

@vstinner vstinner merged commit 3d235f5 into python:master Feb 17, 2020
@vstinner
Copy link
Copy Markdown
Member

I changed the bpo number in the commit message to: https://bugs.python.org/issue39500

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.

4 participants