Copilot commented on code in PR #819:
URL: https://github.com/apache/fesod/pull/819#discussion_r2704411844
##########
fesod-sheet/src/main/java/org/apache/fesod/sheet/util/DateUtils.java:
##########
@@ -552,7 +555,7 @@ public static boolean isInternalDateFormat(short format) {
case 0x14:
case 0x15:
case 0x16:
- // 45-47
+ // 45-47
Review Comment:
The indentation of this comment has been changed unnecessarily, similar to
the comments above. This should be reverted to align with the original code
style.
##########
fesod-sheet/src/main/java/org/apache/fesod/sheet/util/DateUtils.java:
##########
@@ -540,9 +543,9 @@ public static boolean isADateFormatUncached(Short
formatIndex, String formatStri
*/
public static boolean isInternalDateFormat(short format) {
switch (format) {
- // Internal Date Formats as described on page 427 in
- // Microsoft Excel Dev's Kit...
- // 14-22
+ // Internal Date Formats as described on page 427 in
+ // Microsoft Excel Dev's Kit...
+ // 14-22
Review Comment:
The indentation of these comments has been changed unnecessarily. The
original alignment had comments flush with the case statements, but they've
been indented an additional 4 spaces. Since this PR is focused on removing
duplicate utility classes and updating imports, this formatting change appears
unintentional and should be reverted to maintain consistency with the original
code style.
--
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]
