src/main/java/org/apache/commons/net/ftp/parser/EnterpriseUnixFTPEntryParser.java (2 lines): - line 55: + "\\s*" // TODO should the space be optional? - line 56: // TODO \\d* should be \\d? surely ? Otherwise 01111 is allowed src/main/java/org/apache/commons/net/ftp/parser/UnixFTPEntryParser.java (2 lines): - line 79: + "\\s*" // separator TODO why allow it to be omitted?? - line 219: type = FTPFile.FILE_TYPE; // TODO change this if DEVICE_TYPE implemented src/main/java/org/apache/commons/net/ftp/parser/MLSxEntryParser.java (2 lines): - line 163: // TODO these are mostly just guesses at present - line 282: // TODO should this cause failure, or can it be reported somehow? src/main/java/org/apache/commons/net/ftp/FTPListParseEngine.java (2 lines): - line 131: // TODO remove; not actually thrown - line 148: // TODO remove; not actually thrown src/main/java/org/apache/commons/net/telnet/TelnetInputStream.java (2 lines): - line 32: // TODO should these be private enums? - line 564: // TODO remove this src/main/java/org/apache/commons/net/ftp/FTPFile.java (2 lines): - line 93: /** TODO Consider changing internal representation to java.time. */ - line 360: // TODO: only allow permission setting if file is valid src/main/java/org/apache/commons/net/nntp/NntpThreadContainer.java (1 line): - line 48: // TODO convert this to an iterative function - this can blow the stack src/main/java/org/apache/commons/net/ftp/parser/VMSFTPEntryParser.java (1 line): - line 55: // TODO - perhaps restrict permissions to [RWED]* ? src/main/java/org/apache/commons/net/util/ListenerList.java (1 line): - line 68: * @since 2.0 TODO Check that this is a good defensive strategy src/main/java/org/apache/commons/net/ftp/FTPReply.java (1 line): - line 24: * TODO replace with an enum src/main/java/org/apache/commons/net/imap/IMAPSClient.java (1 line): - line 298: // TODO the following setup appears to duplicate that in the super class methods src/main/java/org/apache/commons/net/ftp/parser/FTPTimestampParserImpl.java (1 line): - line 292: // TODO this won't always work for systems that use short dates +0/-12months src/main/java/org/apache/commons/net/ftp/parser/MVSFTPEntryParser.java (1 line): - line 416: // TODO check this is ok src/main/java/org/apache/commons/net/telnet/TelnetOutputStream.java (1 line): - line 36: // TODO there does not appear to be any way to change this value - should it be a ctor parameter? src/main/java/org/apache/commons/net/nntp/Threader.java (1 line): - line 175: // TODO verify this will avoid rehashing src/main/java/org/apache/commons/net/io/FromNetASCIIInputStream.java (1 line): - line 65: // TODO - can we delete this override now? src/main/java/org/apache/commons/net/pop3/POP3SClient.java (1 line): - line 293: // TODO the following setup appears to duplicate that in the super class methods src/main/java/org/apache/commons/net/smtp/SMTPSClient.java (1 line): - line 302: // TODO the following setup appears to duplicate that in the super class methods src/main/java/org/apache/commons/net/util/SSLSocketUtils.java (1 line): - line 36: // TODO for 4.0: becomes a void method. src/main/java/org/apache/commons/net/imap/IMAPReply.java (1 line): - line 112: if (m.matches()) { // TODO would lookingAt() be more efficient? If so, then drop trailing .* from patterns src/main/java/org/apache/commons/net/ftp/parser/MacOsPeterFTPEntryParser.java (1 line): - line 140: type = FTPFile.FILE_TYPE; // TODO change this if DEVICE_TYPE implemented src/main/java/org/apache/commons/net/ftp/FTPSClient.java (1 line): - line 1125: // TODO the following setup appears to duplicate that in the super class methods src/main/java/org/apache/commons/net/ntp/TimeInfo.java (1 line): - line 168: // TODO: is offset still valid if rcvNtpTime=0 || xmitNtpTime=0 ???