Skip to content

Update OleFileIO.py#931

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

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

Conversation

@JHei

@JHei JHei commented Sep 30, 2014

Copy link
Copy Markdown

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
Copy Markdown

Coverage Status

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

@hugovk

hugovk commented Oct 7, 2014

Copy link
Copy Markdown
Member

@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
Copy Markdown
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

JHei commented Oct 8, 2014

Copy link
Copy Markdown
Author

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