Fixes for bug #3418258. Cleans up all of the 'unchecked' and 'deprecated' linter warnings. Several of the redundant casts have been intentionally left for clarity. Set '-serial' and '-cast' linter flags to reduce the number of warnings (it should be safe to ignore serial because the end product is a jar rather than individual class files. AFAIK, there is no performance penalty for redundant casts in Java). If anyone objects to the linter flags they can be removed (the linter flags, that is).
git-svn-id: svn://svn.code.sf.net/p/tigervnc/code/trunk@4694 3789f03b-4d11-0410-bbf8-ca57d06f2519
diff --git a/java/src/com/tigervnc/vncviewer/CConn.java b/java/src/com/tigervnc/vncviewer/CConn.java
index 636bfac..8784750 100644
--- a/java/src/com/tigervnc/vncviewer/CConn.java
+++ b/java/src/com/tigervnc/vncviewer/CConn.java
@@ -942,8 +942,8 @@
}
clipboardDialog.setSendingEnabled(viewer.sendClipboard.getValue());
- menuKey = (int)(options.menuKey.getSelectedIndex()+0xFFBE);
- F8Menu.f8.setLabel("Send F"+(menuKey-Keysyms.F1+1));
+ menuKey = (options.menuKey.getSelectedIndex()+0xFFBE);
+ F8Menu.f8.setText("Send F"+(menuKey-Keysyms.F1+1));
shared = options.shared.isSelected();
setShared(shared);
@@ -1185,9 +1185,9 @@
if (cp.width != desktop.scaledWidth ||
cp.height != desktop.scaledHeight) {
int sx = (desktop.scaleWidthRatio == 1.00)
- ? ev.getX() : (int)Math.floor(ev.getX()/(float)desktop.scaleWidthRatio);
+ ? ev.getX() : (int)Math.floor(ev.getX()/desktop.scaleWidthRatio);
int sy = (desktop.scaleHeightRatio == 1.00)
- ? ev.getY() : (int)Math.floor(ev.getY()/(float)desktop.scaleHeightRatio);
+ ? ev.getY() : (int)Math.floor(ev.getY()/desktop.scaleHeightRatio);
ev.translatePoint(sx - ev.getX(), sy - ev.getY());
writer().writePointerEvent(new Point(ev.getX(),ev.getY()), buttonMask);
} else {
diff --git a/java/src/com/tigervnc/vncviewer/DesktopWindow.java b/java/src/com/tigervnc/vncviewer/DesktopWindow.java
index b1457c3..d6704e0 100644
--- a/java/src/com/tigervnc/vncviewer/DesktopWindow.java
+++ b/java/src/com/tigervnc/vncviewer/DesktopWindow.java
@@ -339,8 +339,8 @@
}
}
}
- scaleWidthRatio = (float)((float)scaledWidth / (float)cc.cp.width);
- scaleHeightRatio = (float)((float)scaledHeight / (float)cc.cp.height);
+ scaleWidthRatio = (float)scaledWidth / (float)cc.cp.width;
+ scaleHeightRatio = (float)scaledHeight / (float)cc.cp.height;
}
synchronized public void paintComponent(Graphics g) {
@@ -475,8 +475,8 @@
cursorVisible = true;
if (softCursor != null) return;
- int cursorLeft = (int)cursor.hotspot.x;
- int cursorTop = (int)cursor.hotspot.y;
+ int cursorLeft = cursor.hotspot.x;
+ int cursorTop = cursor.hotspot.y;
int cursorRight = cursorLeft + cursor.width();
int cursorBottom = cursorTop + cursor.height();
diff --git a/java/src/com/tigervnc/vncviewer/Makefile b/java/src/com/tigervnc/vncviewer/Makefile
index 136899b..614057b 100644
--- a/java/src/com/tigervnc/vncviewer/Makefile
+++ b/java/src/com/tigervnc/vncviewer/Makefile
@@ -4,7 +4,7 @@
CP = cp
JC = javac
-JCFLAGS = -target 1.5 -classpath ../../../
+JCFLAGS = -target 1.5 -Xlint:all,-serial,-cast -classpath ../../../
JAR = jar
ARCHIVE = VncViewer.jar
MANIFEST = MANIFEST.MF