https://issues.apache.org/bugzilla/show_bug.cgi?id=50570
--- Comment #18 from Konstantin Kolinko <knst.koli...@gmail.com> 2011-07-05 21:57:06 UTC --- (In reply to comment #16) Regarding attachment 27225 (Java patch) I have two comments: 1) Code formatting. We usually have "{" on the same line as previous statement (like in Java coding style). 2) I go not get why there are two methods in SSL.java: fipsModeSet(Integer mode) and fipsModeSet(int mode). If you need class object of "int" to do lookup of the method, it is available as Integer.TYPE. -- Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email ------- 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