https://defect.opensolaris.org/bz/show_bug.cgi?id=18602

--- Comment #21 from Trond Norbye <[email protected]> 2011-07-01 05:32:53 
UTC ---
(In reply to comment #20)
> Created attachment 4554 [details]
> patch to reduce mem footprint of JarAnalyzer

Since we don't have a decent code review tool:


-        byte buf[] = new byte[1024];
+        byte buf[] = new byte[1<<11]; // 2 K

WHy don't you just write 2048 ? It's one character less to type, and you don't
have to write a comment that it's what you mean..

-- 
Configure bugmail: https://defect.opensolaris.org/bz/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.
_______________________________________________
opengrok-dev mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/opengrok-dev

Reply via email to