public inbox for java-patches@gcc.gnu.org
 help / color / mirror / Atom feed
* FYI: Another javax.management fix
@ 2007-02-09 17:17 Gary Benson
  0 siblings, 0 replies; only message in thread
From: Gary Benson @ 2007-02-09 17:17 UTC (permalink / raw)
  To: java-patches

[-- Attachment #1: Type: text/plain, Size: 229 bytes --]

Hi all,

The attached patch fixes two of the three exceptions I posted earlier
today.  It's committed to redhat/gcc-4_1-branch-java-merge-20070117
and to trunk (and I even remembered to commit the class this time!)

Cheers,
Gary

[-- Attachment #2: patch --]
[-- Type: text/plain, Size: 909 bytes --]

Index: ChangeLog.gcj
===================================================================
--- ChangeLog.gcj	(revision 121739)
+++ ChangeLog.gcj	(working copy)
@@ -1,3 +1,8 @@
+2007-02-09  Gary Benson  <gbenson@redhat.com>
+
+	* javax/management/ObjectName.java
+	(quote): Initialize StringBuilder correctly.
+
 2007-02-08  Gary Benson  <gbenson@redhat.com>
 
 	* javax/management/MBeanServerFactory.java
Index: javax/management/ObjectName.java
===================================================================
--- javax/management/ObjectName.java	(revision 121739)
+++ javax/management/ObjectName.java	(working copy)
@@ -674,7 +674,8 @@
    */
   public static String quote(String string)
   {
-    StringBuilder builder = new StringBuilder('"');
+    StringBuilder builder = new StringBuilder();
+    builder.append('"');
     for (int a = 0; a < string.length(); ++a)
       {
 	char s = string.charAt(a);

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2007-02-09 17:17 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2007-02-09 17:17 FYI: Another javax.management fix Gary Benson

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).