ROB: Initialize states array with an empty value

[alexey-v-paramonov]

This is done to prevent dictionary key error below when states are not initialized

[stefan6419846]

We should probably have a corresponding test case for this as well to ensure that it indeed does not fail.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (5a2dd75) 94.45% compared to head (87d892f) 94.45%.

❗ Current head 87d892f differs from pull request most recent head 61bb49b. Consider uploading reports for the commit 61bb49b to get more accurate results

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #2280   +/-   ##
=======================================
  Coverage   94.45%   94.45%           
=======================================
  Files          43       43           
  Lines        7650     7651    +1     
  Branches     1511     1511           
=======================================
+ Hits         7226     7227    +1     
  Misses        262      262           
  Partials      162      162           

Files	Coverage Δ
pypdf/_reader.py	91.69% <100.00%> (+<0.01%)	⬆️

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[alexey-v-paramonov]

CI complains about type hints, but that is not related to my change, should I fix it?

states = []

Should be

states: List[Any] = []

I guess

[MartinThoma]

Thank you for your contribution @alexey-v-paramonov :-)

If you want, I can add you to https://pypdf.readthedocs.io/en/latest/meta/CONTRIBUTORS.html

[MartinThoma]

@pubpub-zz @stefan6419846 Thank you for reviewing the changes 🙏

[alexey-v-paramonov]

Thank you for your contribution @alexey-v-paramonov :-)

If you want, I can add you to https://pypdf.readthedocs.io/en/latest/meta/CONTRIBUTORS.html

That would be awesome! Thanks!