Skip to content

Update OleFileIO.py#931

Closed
JHei wants to merge 1 commit intopython-pillow:masterfrom
JHei:master
Closed

Update OleFileIO.py#931
JHei wants to merge 1 commit intopython-pillow:masterfrom
JHei:master

Conversation

@JHei
Copy link

@JHei JHei commented Sep 30, 2014

replaced latin-1 by iso-8859-1 (gettext doesn't know latin-1)

replaced latin-1 by iso-8859-1 (gettext doesn't know latin-1)
@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling 1863321 on JHei:master into 5a68276 on python-pillow:master.

@hugovk
Copy link
Member

hugovk commented Oct 7, 2014

@JHei, Is this PR still needed after the recent change to OleFileIO.py (#942)? If so, please could you updates this PR?

@wiredfool
Copy link
Member

I don't think it matters, as the file is entirely in the <128 ascii space, so there are no 8 bit characters to interpret.

@JHei
Copy link
Author

JHei commented Oct 8, 2014

no problems here without the coding-line

@JHei JHei closed this Oct 8, 2014
@hugovk hugovk added the olefile label May 10, 2015
Sign up for free to 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.

4 participants