Review request for #6783209

Brandon Passanisi brandon.passanisi at oracle.com
Tue Jan 3 18:37:22 UTC 2012


Hello core-libs-dev.  Can someone please do a code review of the 
following proposed fix for 6783209 : (fmt)Formatter doesn't support 
width for %%.

    Webrev : http://cr.openjdk.java.net/~bpassani/6783209/0/webrev/
    Bug URL: http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=6783209

The fix is to use justify() within Formatter.java to do the proper 
padding when using the '%' conversion.  I have included a new test 
program to test instances of format specifiers that use the '%' 
conversion with a width value.  The test also includes the testing of 
'%' without a width to ensure there aren't regressions.

Thanks.
-- 
Oracle <http://www.oracle.com>
Brandon Passanisi | Principle Member of Technical Staff


Green Oracle <http://www.oracle.com/commitment> Oracle is committed to 
developing practices and products that help protect the environment



More information about the core-libs-dev mailing list