RFR: 8006090 - Formatter asserts with -esa

Alan Bateman Alan.Bateman at oracle.com
Thu Jan 17 18:00:46 UTC 2013


This looks fine to me.

Sherman - I suspect the same assert in the print method added by JSR-310 
has the same issue.

-Alan.

On 17/01/2013 17:36, Brian Burkhalter wrote:
> The assert occurs in
>
> private Appendable print(StringBuilder sb, Calendar t, char c,
>                                   Locale l)
>
> due to
>
> assert(width == -1);
>
> not being satisfied. The proposed fix is to remove this assert statement which appears to be vestigial. Regression and compatibility testing did not reveal any failures due to removing this statement.
>
> --- old/src/share/classes/java/util/Formatter.java      2013-01-16 12:22:55.000000000 -0800
> +++ new/src/share/classes/java/util/Formatter.java      2013-01-16 12:22:55.000000000 -0800
> @@ -3827,7 +3827,6 @@
>                                    Locale l)
>               throws IOException
>           {
> -            assert(width == -1);
>               if (sb == null)
>                   sb = new StringBuilder();
>               switch © {
>
> Thanks,
>
> Brian




More information about the core-libs-dev mailing list