Review needed for 7132879 to address the findbug errors in WebRowSetXmlWriter
Lance Andersen - Oracle
lance.andersen at oracle.com
Tue Jan 24 19:02:31 UTC 2012
Here is the change removing the java.util.Map
thanks for the feedback.
new-host-4:internal lanceandersen$ hg diff
diff -r b14e13237498 src/share/classes/com/sun/rowset/internal/WebRowSetXmlWriter.java
--- a/src/share/classes/com/sun/rowset/internal/WebRowSetXmlWriter.java Wed Jan 18 11:00:20 2012 -0800
+++ b/src/share/classes/com/sun/rowset/internal/WebRowSetXmlWriter.java Tue Jan 24 14:01:20 2012 -0500
@@ -1,5 +1,5 @@
/*
- * Copyright (c) 2003, 2010, Oracle and/or its affiliates. All rights reserved.
+ * Copyright (c) 2003, 2012, Oracle and/or its affiliates. All rights reserved.
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
*
* This code is free software; you can redistribute it and/or modify it
@@ -48,7 +48,7 @@
* for this field is set with the <code>java.io.Writer</code> object given
* as the second argument to the <code>writeXML</code> method.
*/
- private java.io.Writer writer;
+ private transient java.io.Writer writer;
/**
* The <code>java.util.Stack</code> object that this <code>WebRowSetXmlWriter</code>
@@ -205,17 +205,12 @@
//Changed to beginSection and endSection for maps for proper indentation
beginSection("map");
- java.util.Map<?,?> typeMap = caller.getTypeMap();
- if (typeMap != null) {
- Iterator<?> i = typeMap.keySet().iterator();
- Class<?> c;
- String type;
- while (i.hasNext()) {
- type = (String)i.next();
- c = (Class)typeMap.get(type);
- propString("type", type);
- propString("class", c.getName());
- }
+ Map<String, Class<?>> typeMap = caller.getTypeMap();
+ if(typeMap != null) {
+ for(Map.Entry<String, Class<?>> mm : typeMap.entrySet()) {
+ propString("type", mm.getKey());
+ propString("class", mm.getValue().getName());
+ }
}
endSection("map");
On Jan 24, 2012, at 1:58 PM, Rémi Forax wrote:
> On 01/24/2012 07:50 PM, Lance Andersen - Oracle wrote:
>> You are suggesting changing
>>
>> java.util.Map<?,?> typeMap = caller.getTypeMap();
>>
>> to
>>
>> Map<?,?> typeMap = caller.getTypeMap();
>>
>>
>> Which I can do. I probably did not think about it much as it was their previously but I agree that would be the way to go.
>>
>> Please let me know if I am on point with your suggestion
>
> Yes.
>
> Map<String, Class<?>> typeMap = ...
>
>
>>
>> Best
>> Lance
>
> rgds,
> Rémi
>
>> On Jan 24, 2012, at 1:29 PM, Remi Forax wrote:
>>
>>> Looks good.
>>> The only weird thing is the use of java.util.Map instead of Map. Because Map is used in when referencing Map.Entry just after.
>>>
>>> Rémi
>>>
>>>
>>> ----- Reply message -----
>>> From: "Lance Andersen - Oracle" <lance.andersen at oracle.com <mailto:lance.andersen at oracle.com>>
>>> To: "core-libs-dev Libs" <core-libs-dev at openjdk.java.net <mailto:core-libs-dev at openjdk.java.net>>
>>> Subject: Review needed for 7132879 to address the findbug errors in WebRowSetXmlWriter
>>> Date: Tue, Jan 24, 2012 19:18
>>>
>>>
>>> Hi,
>>>
>>> Looking for a reviewer for http://cr.openjdk.java.net/~lancea/7132879/webrev.00/ <http://cr.openjdk.java.net/%7Elancea/7132879/webrev.00/>
>>>
>>> this is a small change to address 2 findbug errors for this class. The findbug issues being addressed:
>>>
>>> Inefficient use of keySet iterator instead of entrySet iterator
>>>
>>> and
>>>
>>> non-serializable instance field in serializable class
>>>
>>>
>>> Best,
>>> Lance
>>>
>>> Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037
>>> Oracle Java Engineering
>>> 1 Network Drive
>>> Burlington, MA 01803
>>> Lance.Andersen at oracle.com <mailto:Lance.Andersen at oracle.com>
>>>
>>
>> <http://oracle.com/us/design/oracle-email-sig-198324.gif>
>> <http://oracle.com/us/design/oracle-email-sig-198324.gif>Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037
>> Oracle Java Engineering
>> 1 Network Drive
>> Burlington, MA 01803
>> Lance.Andersen at oracle.com <mailto:Lance.Andersen at oracle.com>
>>
>
Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037
Oracle Java Engineering
1 Network Drive
Burlington, MA 01803
Lance.Andersen at oracle.com
More information about the core-libs-dev
mailing list