liugddx commented on code in PR #745:
URL: https://github.com/apache/fesod/pull/745#discussion_r2645377912
##########
fesod-sheet/src/main/java/org/apache/fesod/sheet/support/ExcelTypeEnum.java:
##########
@@ -50,7 +50,12 @@ public enum ExcelTypeEnum {
/**
* xlsx
*/
- XLSX(".xlsx", new byte[] {80, 75, 3, 4});
+ XLSX(".xlsx", new byte[] {80, 75, 3, 4}),
+
+ /**
+ * ods (OpenDocument Spreadsheet)
+ */
+ ODS(".ods", new byte[] {80, 75, 3, 4});
Review Comment:
> as ODS has same magic number with XLSX, so any potential issue during the
magic usage in project level?
Check the internal structure of ZIP to distinguish (e.g., XLSX has
xl/workbook.xml, ODS has content.xml).
--
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]
