Reduce code duplication in PDF parser classes

[Copilot]

Description of the new Feature/Bugfix

Eliminated duplicate code across PdfContentTextExtractor, PdfContentTextLocator, and PdfTextLocator by extracting shared implementations to the base class PdfContentStreamHandler.

Changes:

• Moved duplicate Do inner class (XObject form handler) from both subclasses to PdfContentStreamHandler as a protected inner class
• Extracted processContent() method to base class - handles PDF content stream parsing
• Created static utility getContentBytesFromPdfObjectStatic() for reading bytes from PdfObject (handles INDIRECT, STREAM, ARRAY types)
• Updated all three classes to use shared implementations

Impact:

• Removed 177 lines of duplicated code
• Added 114 lines of shared implementation
• Reduced duplication from 24.3%/23.2% to <3%
• No functional changes - behavior identical to previous implementation

Unit-Tests for the new Feature/Bugfix

• Existing unit tests cover the functionality
• All tests pass without modification (no behavioral changes)

Compatibilities Issues

No breaking changes. All methods remain package-protected or private. The Do class is now instantiated via this.new Do() in subclasses instead of new Do(), but this is an internal implementation detail.

Your real name

GitHub Copilot Workspace Agent

Testing details

Verified with full openpdf-core test suite. The refactoring is purely structural - no logic changes.

---

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

[sonarqubecloud]

Quality Gate passed

Issues
5 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud