bogdiuk commented on PR #189:
URL: https://github.com/apache/pdfbox/pull/189#issuecomment-2175740993
Reasoning behind using a separate class `LoadOnlyHeaders` instead of
extracting information directly from `TrueTypeFont`:
* easier to debug - single place to put breakpoints
* easier to refactor:
* for example, simple "Find All Usages" can reveal that `setSomething()`
is not used
* the alternative is having `boolean isLoadOnlyHeaders` in `TTFParser`,
`CFFParser` and `TrueTypeFont`: it would be harder to collect usages of all 3
* in the future, parsing some tables can be skipped completely, only setting
respective fields of `LoadOnlyHeaders`
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
