BUG: catch the case where w[0] is an IndirectObject instead of an int#2154
Conversation
|
👍 |
Will do, would you want it inside an existing test file, or a separate one? And I assume the file goes to |
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## main #2154 +/- ##
=======================================
Coverage 94.34% 94.34%
=======================================
Files 43 43
Lines 7572 7572
Branches 1488 1488
=======================================
Hits 7144 7144
Misses 263 263
Partials 165 165
☔ View full report in Codecov by Sentry. |
|
It sounds good to add it to test_cmap.py |
You will find some examples of files being downloaded on-the-fly inside the test code, see Lines 1040 to 1044 in 05f2a65 |
The file I need is from arxiv, should be pretty easy to get a direct link. Thanks for the pointers. |
|
you should use https://github.com/py-pdf/pypdf/files/12489914/Morris.et.al.-.2020.-.TextAttack.A.Framework.for.Adversarial.Attacks.Data.Augmentation.and.Adversarial.Training.in.NLP.pdf which is already within discussion |
- a pdf file from arxiv is included
- URL too long - file name too long - variable declared but not used
The URL is too long to pass the code style check, I used the file directly from arxiv instead. But if file within github is preferred, I can shorten the file name and upload again. |
…e-interpreted-as-an-integer
|
Well done, @rraval ! I've just merged it and will release soon. If you want, I can add you to https://pypdf.readthedocs.io/en/latest/meta/CONTRIBUTORS.html |
## What's new ### Security (SEC) - Infinite recursion caused by IndirectObject clone (#2156) ### New Features (ENH) - Ease access to ViewerPreferences (#2144) ### Bug Fixes (BUG) - catch the case where w[0] is an IndirectObject instead of an int (#2154) - Cope with indirect objects in filters and remove deprecated code (#2177) - Cope with extra space (#2151) - Merge pages without resources (#2150) - getcontents() shall return None if contents is NullObject (#2161) - Fix conversion from 1 to LA (#2175) - Accept tabs in cmaps (#2174) ### Robustness (ROB) - Accept XYZ with no arguments (#2178) [Full Changelog](3.15.5...3.16.0)
Sure, that sounds nice. Thanks. |
Closes #2137