<Swing Dev> Supplemental code review request for JDK-8043550: Fix raw and unchecked lint warnings in javax.swing.*
Mandy Chung
mandy.chung at oracle.com
Thu Jul 3 20:20:46 UTC 2014
I reviewed the jconsole change that looks okay to me.
Mandy
On 7/3/14 12:44 PM, Joe Darcy wrote:
> Hello,
>
> In the course of fixing
>
> JDK-8043550: Fix raw and unchecked lint warnings in javax.swing.*
> http://cr.openjdk.java.net/~darcy/8043550.1/
>
> I encountered a build failure in jconsole due to a conflict between a
> recent generification in jconsole
>
> 8044865: Fix raw and unchecked lint warnings in management-related
> code
> http://hg.openjdk.java.net/jdk9/dev/jdk/rev/81e22723ba57
>
> and the newly generified swing. Fortunately, the adjustment in
> jconsole to be reviewed is simple:
>
> --- a/src/share/classes/sun/tools/jconsole/inspector/TableSorter.java
> Tue Jul 01 23:09:52 2014 +0000
> +++ b/src/share/classes/sun/tools/jconsole/inspector/TableSorter.java
> Thu Jul 03 12:09:44 2014 -0700
> @@ -217,17 +217,17 @@
> }
> }
>
> - private Vector<?> getRow(int row) {
> - return (Vector) dataVector.elementAt(row);
> + private Vector<Object> getRow(int row) {
> + return dataVector.elementAt(row);
> }
>
> @SuppressWarnings("unchecked")
> - private void setRow(Vector<?> data, int row) {
> + private void setRow(Vector<Object> data, int row) {
> dataVector.setElementAt(data,row);
> }
>
> private void swap(int i, int j, int column) {
> - Vector<?> data = getRow(i);
> + Vector<Object> data = getRow(i);
> setRow(getRow(j),i);
> setRow(data,j);
>
>
> The protected dataVector field is now declared to be a
> Vector<Vector<Object>> so the more precise type can be used in these
> private methods.
>
> Thanks,
>
> -Joe
More information about the swing-dev
mailing list