-
-
Notifications
You must be signed in to change notification settings - Fork 2.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Nitpicks #942
Nitpicks #942
Conversation
OleFileIO.py has an Also, I'm not sure if we're taking changes to OleFileIO.py as it's also maintained elsewhere, but having said that we have made some changes recently which should probably be contributed back upstream, so I'll leave this to the other project members to decide. This also concerns PR #931. |
Oh yes I missed the |
@wiredfool @aclark4life Are we ok to take changes for OleFileIO.py? See also #931. |
+0, although if it's just nitpicks with no serious consequences then +1 for sure. |
I don't think the changes to OleFileIO are going to lead to messy merges with Philippe's future changes. So, fine with me. |
Fixes for things rpmlint complains about
Thanks! |
Hi, I am not sure why the encoding declaration in OleFileIO is an issue, but removing it should be fine. However, according to PEP 0263, the absence of encoding declaration means the encoding is "ascii" (7-bits), and not UTF-8 as mentioned in commit 0ca102f. If UTF-8 is preferred, I think it should be explicit with an encoding declaration. |
My bad, thanks for spotting. |
Thanks @decalage2! |
Just some trivial fixes for things
rpmlint
complains about:Jpeg2KEncode.c
should not be executableOleFileIO.py
should not have a shebang since it is not an executable python scriptWalImageFile.py
should be utf-8 encoded