dmytro-sylaiev commented on a change in pull request #95:
URL: https://github.com/apache/commons-net/pull/95#discussion_r800795665
##########
File path:
src/main/java/org/apache/commons/net/ftp/parser/OS400FTPEntryParser.java
##########
@@ -243,7 +243,7 @@
+ "(?:(\\d+)\\s+)?" // size, empty for members
+ "(?:(\\S+)\\s+(\\S+)\\s+)?" // date stuff, empty for members
+ "(\\*STMF|\\*DIR|\\*FILE|\\*MEM)\\s+" // *STMF/*DIR/*FILE/*MEM
- + "(?:(\\S+)\\s*)?"; // file name, missing, when CWD is a
*FILE
+ + "(((\\S+)\\s*)+)?"; // file name, missing, when CWD is a
*FILE
Review comment:
Hi @sebbASF
Could you please elaborate on the complicated parts of RegEX? I did just the
small fix and do not make it more complicated than it's already been
For file started with spaces - I can't find any strict documentation which
says it's possible (it's not a unix system), but when I tried to upload such
file or rename with RNTO command, the heading spaces just were trimmed in the
file name, so I don't think it's supported
--
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]
