Skip to content

Fix "invalid escape sequence" SyntaxError#522

Merged
MartinThoma merged 1 commit intopy-pdf:masterfrom
niklasb:patch-1
Apr 6, 2022
Merged

Fix "invalid escape sequence" SyntaxError#522
MartinThoma merged 1 commit intopy-pdf:masterfrom
niklasb:patch-1

Conversation

@niklasb
Copy link
Copy Markdown
Contributor

@niklasb niklasb commented Oct 22, 2019

This happens when the library is used with -Werror, since \c is not a valid Python escape sequence.

This happens when the library is used with `-Werror`, since `\c` is not a valid Python escape sequence.
@MartinThoma MartinThoma added the is-bug From a users perspective, this is a bug - a violation of the expected behavior with a compliant PDF label Apr 6, 2022
@MartinThoma MartinThoma merged commit dacc6b7 into py-pdf:master Apr 6, 2022
@MartinThoma
Copy link
Copy Markdown
Member

Thank you for the contribution @niklasb :-) I just became the maintainer of this library and thus can get those fixes in :-)

@MartinThoma
Copy link
Copy Markdown
Member

MartinThoma commented Apr 6, 2022

Were there other things you've fixed in your fork?

MartinThoma added a commit that referenced this pull request Apr 7, 2022
Features:

 - Add alpha channel support for png files in Script (#614)

Bug fixes (BUG):

 - Fix formatWarning for filename without slash (#612)
 - Add whitespace between words for extractText() (#569, #334)
 - "invalid escape sequence" SyntaxError (#522)
 - Avoid error when printing warning in pythonw (#486)
 - Stream operations can be List or Dict (#665)

Documentation (DOC):

 - Added Scripts/pdf-image-extractor.py
 - Documentation improvements (#550, #538, #324, #426, #394)

Tests and Test setup (TST):

 - Add Github Action which automatically run unit tests via pytest and
   static code analysis with Flake8 (#660)
 - Add several unit tests (#661, #663)
 - Add .coveragerc to create coverage reports

Developer Experience Improvements (DEV):

 - Pre commit: Developers can now `pre-commit install` to avoid tiny issues
               like trailing whitespaces

Miscallenious:

 - Add the LICENSE file to the distributed packages (#288)
 - Use setuptools instead of distutils (#599)
 - Improvements for the PyPI page (#644)
 - Python 3 changes (#504, #366)

You can see the full changelog at: 1.26.0...1.27.0
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

is-bug From a users perspective, this is a bug - a violation of the expected behavior with a compliant PDF

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants