https://issues.apache.org/bugzilla/show_bug.cgi?id=57540
--- Comment #7 from Mark Thomas <ma...@apache.org> --- Reviewing the proposed patch: 1. The changes to tomcat/util/net/jsse/openssl/Protocol.java need to be removed 2. There looks to be the right hooks in tc-native to obtain the protocol so do that rather than throw an IOE for APR. 3. Comment lines should wrap at 80 chars. 4. The Javadoc comments starting "as per..." should not refer to this bug report but fully describe the elements they are documenting. I'd expect the description for the attribute name to be different to that for the method. -- You are receiving this mail because: You are the assignee for the bug. --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org For additional commands, e-mail: dev-h...@tomcat.apache.org
