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

Reply via email to