Skip to content
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

ROB: Handle recursion error #804

Merged
merged 1 commit into from Apr 23, 2022
Merged

ROB: Handle recursion error #804

merged 1 commit into from Apr 23, 2022

Conversation

MartinThoma
Copy link
Member

@MartinThoma MartinThoma commented Apr 23, 2022

This doesn't solve the issue, but it might make it
less severe.

See #520
See #268
See virantha/pypdfocr#59

Taken from sfneal@3558a69

Co-authored-by: danniesim geemee@gmail.com

This doesn't solve the issue, but it might make it
less severe.

See #520
See #268
See virantha/pypdfocr#59

Co-authored-by: danniesim <geemee@gmail.com>
@codecov-commenter
Copy link

Codecov Report

Merging #804 (70b9733) into main (40df4d7) will not change coverage.
The diff coverage is 0.00%.

@@           Coverage Diff           @@
##             main     #804   +/-   ##
=======================================
  Coverage   75.22%   75.22%           
=======================================
  Files          11       11           
  Lines        3516     3516           
  Branches      810      810           
=======================================
  Hits         2645     2645           
  Misses        658      658           
  Partials      213      213           
Impacted Files Coverage Δ
PyPDF2/pdf.py 81.85% <0.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 40df4d7...70b9733. Read the comment docs.

@MartinThoma MartinThoma merged commit 3d65938 into main Apr 23, 2022
@MartinThoma MartinThoma deleted the recursion-error branch April 23, 2022 08:57
dsk7 pushed a commit to montaggroup/PyPDF2 that referenced this pull request Apr 23, 2022
This doesn't solve the issue, but it might make it less severe.

See py-pdf#520
See py-pdf#268
See virantha/pypdfocr#59

sfneal@3558a69

Co-authored-by: danniesim <geemee@gmail.com>
MartinThoma added a commit that referenced this pull request Apr 24, 2022
A change I would like to highlight is the performance improvement for
large PDF files (#808) 🎉

New Features (ENH):
-  Add papersizes (#800)
-  Allow setting permission flags when encrypting (#803)
-  Allow setting form field flags (#802)

Bug Fixes (BUG):
-  TypeError in xmp._converter_date (#813)
-  Improve spacing for text extraction (#806)
-  Fix PDFDocEncoding Character Set (#809)

Robustness (ROB):
-  Use null ID when encrypted but no ID given (#812)
-  Handle recursion error (#804)

Documentation (DOC):
-  CMaps (#811)
-  The PDF Format + commit prefixes (#810)
-  Add compression example (#792)

Developer Experience (DEV):
-  Add Benchmark for Performance Testing (#781)

Maintenance (MAINT):
-  Validate PDF magic byte in strict mode (#814)
-  Make PdfFileMerger.addBookmark() behave life PdfFileWriters\' (#339)
-  Quadratic runtime while parsing reduced to linear  (#808)

Testing (TST):
-  Newlines in text extraction (#807)

Full Changelog: 1.27.8...1.27.9
VictorCarlquist pushed a commit to VictorCarlquist/PyPDF2 that referenced this pull request Apr 29, 2022
This doesn't solve the issue, but it might make it less severe.

See py-pdf#520
See py-pdf#268
See virantha/pypdfocr#59

sfneal@3558a69

Co-authored-by: danniesim <geemee@gmail.com>
VictorCarlquist pushed a commit to VictorCarlquist/PyPDF2 that referenced this pull request Apr 29, 2022
A change I would like to highlight is the performance improvement for
large PDF files (py-pdf#808) 🎉

New Features (ENH):
-  Add papersizes (py-pdf#800)
-  Allow setting permission flags when encrypting (py-pdf#803)
-  Allow setting form field flags (py-pdf#802)

Bug Fixes (BUG):
-  TypeError in xmp._converter_date (py-pdf#813)
-  Improve spacing for text extraction (py-pdf#806)
-  Fix PDFDocEncoding Character Set (py-pdf#809)

Robustness (ROB):
-  Use null ID when encrypted but no ID given (py-pdf#812)
-  Handle recursion error (py-pdf#804)

Documentation (DOC):
-  CMaps (py-pdf#811)
-  The PDF Format + commit prefixes (py-pdf#810)
-  Add compression example (py-pdf#792)

Developer Experience (DEV):
-  Add Benchmark for Performance Testing (py-pdf#781)

Maintenance (MAINT):
-  Validate PDF magic byte in strict mode (py-pdf#814)
-  Make PdfFileMerger.addBookmark() behave life PdfFileWriters\' (py-pdf#339)
-  Quadratic runtime while parsing reduced to linear  (py-pdf#808)

Testing (TST):
-  Newlines in text extraction (py-pdf#807)

Full Changelog: py-pdf/pypdf@1.27.8...1.27.9
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants