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

Reply via email to