X Tutup
Skip to content

Fix leak of filedescriptor if fontsize cannot be set.#7923

Merged
NelleV merged 1 commit intomatplotlib:masterfrom
Carreau:free-all-the-fonts
Jan 23, 2017
Merged

Fix leak of filedescriptor if fontsize cannot be set.#7923
NelleV merged 1 commit intomatplotlib:masterfrom
Carreau:free-all-the-fonts

Conversation

@Carreau
Copy link
Copy Markdown
Contributor

@Carreau Carreau commented Jan 22, 2017

Closes #7922

Should prevent resources warnings from poping up in scipts of the type:

import warnings
warnings.simplefilter('default')
from matplotlib.ft2font import FT2Font
FT2Font('/System/Library/Fonts/Apple Color Emoji.ttf')

Closes matplotlib#7922

Should prevent resources warnings from poping up in scipts of the type:

    import warnings
    warnings.simplefilter('default')
    from matplotlib.ft2font import FT2Font
    FT2Font('/System/Library/Fonts/Apple Color Emoji.ttf')
@codecov-io
Copy link
Copy Markdown

Current coverage is 62.18% (diff: 100%)

Merging #7923 into master will decrease coverage by <.01%

@@             master      #7923   diff @@
==========================================
  Files           175        175          
  Lines         56125      56125          
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
- Hits          34901      34899     -2   
- Misses        21224      21226     +2   
  Partials          0          0          

Powered by Codecov. Last update 60cd171...79218df

@QuLogic QuLogic changed the title Fix leak of filedescriptor if fontsize cannot be set. [MRG+1] Fix leak of filedescriptor if fontsize cannot be set. Jan 23, 2017
@NelleV NelleV merged commit c93142f into matplotlib:master Jan 23, 2017
@QuLogic QuLogic added this to the 2.1 (next point release) milestone Jan 23, 2017
@QuLogic QuLogic changed the title [MRG+1] Fix leak of filedescriptor if fontsize cannot be set. Fix leak of filedescriptor if fontsize cannot be set. Jan 23, 2017
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

X Tutup