lurongjiang commented on PR #1044: URL: https://github.com/apache/poi/pull/1044#issuecomment-4214070579
Thank you for the review and feedback. I understand your concerns. My original intention was that since POI aims to support various Office document formats, I thought it should be able to handle WPS-generated files as well. When I encountered parsing failures with certain WPS documents, I attempted to add a tolerance mechanism as a workaround. However, I now understand your point. This is not really a WPS bug, but rather a limitation of POI's current implementation - it cannot handle certain non-standard file formats generated by WPS. Adding workarounds in POI's core components (like `ByteArrayBackedDataSource` and `FileBackedDataSource`) would compromise the integrity of the codebase. I'll close this PR as suggested. Users who need to handle such files can implement workarounds at the application level rather than modifying the library itself. -- 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]
