]> source.dussan.org Git - vaadin-framework.git/commitdiff
Fixed unused variable warnings
authorArtur Signell <artur.signell@itmill.com>
Thu, 7 Oct 2010 08:13:32 +0000 (08:13 +0000)
committerArtur Signell <artur.signell@itmill.com>
Thu, 7 Oct 2010 08:13:32 +0000 (08:13 +0000)
svn changeset:15439/svn branch:6.5

21 files changed:
src/com/vaadin/terminal/gwt/client/Util.java
src/com/vaadin/terminal/gwt/client/ui/VAbsoluteLayout.java
src/com/vaadin/terminal/gwt/client/ui/VTextualDate.java
src/com/vaadin/terminal/gwt/client/ui/layout/CellBasedLayout.java
src/com/vaadin/terminal/gwt/client/ui/layout/ChildComponentContainer.java
src/com/vaadin/terminal/gwt/server/ApplicationRunnerServlet.java
src/com/vaadin/terminal/gwt/server/DragAndDropService.java
tests/src/com/vaadin/tests/components/combobox/ComboBoxValueUpdate.java
tests/src/com/vaadin/tests/components/menubar/MenuBarNavigation.java
tests/src/com/vaadin/tests/components/table/TextFieldRelativeWidth.java
tests/src/com/vaadin/tests/components/tree/TreeScrolling.java
tests/src/com/vaadin/tests/layouts/layouttester/GridLayoutTests.java
tests/src/com/vaadin/tests/layouts/layouttester/HorizontalLayoutTests.java
tests/src/com/vaadin/tests/layouts/layouttester/VerticalLayoutTests.java
tests/src/com/vaadin/tests/tickets/Ticket2021.java
tests/src/com/vaadin/tests/tickets/Ticket2029.java
tests/src/com/vaadin/tests/tickets/Ticket2048.java
tests/src/com/vaadin/tests/tickets/Ticket2204.java
tests/src/com/vaadin/tests/tickets/Ticket2242.java
tests/src/com/vaadin/tests/tickets/Ticket2344.java
tests/src/com/vaadin/tests/tickets/Ticket2998.java

index da1d9ce43eb11aa3e002264e29a7224a73746572..5afbd2f15c3b3293c5eb3766466fb4f5aa7ca249 100644 (file)
@@ -324,7 +324,7 @@ public class Util {
 
     public static int measureHorizontalBorder(Element element) {
         int borders;
-        int cw = element.getClientWidth();
+        // element.getClientWidth();
         if (BrowserInfo.get().isIE()) {
             String width = element.getStyle().getProperty("width");
             String height = element.getStyle().getProperty("height");
index af856d047c4e7509f6d66d6c1829a6b52ae0e111..50e33c4c19ee3fb48209d83ec8b511c6c65a39ea 100644 (file)
@@ -45,9 +45,9 @@ public class VAbsoluteLayout extends ComplexPanel implements Container {
 
     protected final Element canvas = DOM.createDiv();
 
-    private int excessPixelsHorizontal;
-
-    private int excessPixelsVertical;
+    // private int excessPixelsHorizontal;
+    //
+    // private int excessPixelsVertical;
 
     private Object previousStyleName;
 
@@ -199,8 +199,8 @@ public class VAbsoluteLayout extends ComplexPanel implements Container {
     public void setStyleName(String style) {
         super.setStyleName(style);
         if (previousStyleName == null || !previousStyleName.equals(style)) {
-            excessPixelsHorizontal = -1;
-            excessPixelsVertical = -1;
+            // excessPixelsHorizontal = -1;
+            // excessPixelsVertical = -1;
         }
     }
 
index df5a0a1db65244cb6c1acf9abe2770a28d0dfbcb..892d2b0851e347c73bbf9dcf9190648a69d400f6 100644 (file)
@@ -135,8 +135,8 @@ public class VTextualDate extends VDateField implements Paintable, Field,
                     String frmString = LocaleService
                             .getDateFormat(currentLocale);
                     frmString = cleanFormat(frmString);
-                    String delim = LocaleService
-                            .getClockDelimiter(currentLocale);
+                    // String delim = LocaleService
+                    // .getClockDelimiter(currentLocale);
 
                     if (currentResolution >= RESOLUTION_HOUR) {
                         if (dts.isTwelveHourClock()) {
index f573e36e09ecb27cfcebf5ae78c90c72d7e0224f..64b1ea5f69b1410c429934c1119893392421f8ff 100644 (file)
@@ -342,8 +342,8 @@ public abstract class CellBasedLayout extends ComplexPanel implements Container
                     throw new NullPointerException();
                 }
             }
-            ChildComponentContainer remove = widgetToComponentContainer
-                    .remove(widget);
+            // ChildComponentContainer remove =
+            widgetToComponentContainer.remove(widget);
             remove(child);
             if (!relocated) {
                 Paintable p = (Paintable) widget;
index babb00c61b515ad25f626c09b594d6a7e7186acc..bcb972ca6a914e19514e224ebc95d0f8b5baa643 100644 (file)
@@ -9,7 +9,6 @@ import java.util.NoSuchElementException;
 import com.google.gwt.dom.client.DivElement;
 import com.google.gwt.dom.client.Document;
 import com.google.gwt.dom.client.Element;
-import com.google.gwt.dom.client.Style;
 import com.google.gwt.dom.client.TableElement;
 import com.google.gwt.user.client.ui.Panel;
 import com.google.gwt.user.client.ui.Widget;
@@ -51,7 +50,7 @@ public class ChildComponentContainer extends Panel {
 
     private double expandRatio;
 
-    private int containerMarginLeft = 0;
+    // private int containerMarginLeft = 0;
     private int containerMarginTop = 0;
 
     AlignmentInfo alignment = AlignmentInfo.TOP_LEFT;
@@ -80,7 +79,7 @@ public class ChildComponentContainer extends Panel {
 
         widgetDIV = Document.get().createDivElement();
         if (BrowserInfo.get().isFF2()) {
-            Style style = widgetDIV.getStyle();
+            // Style style = widgetDIV.getStyle();
             // FF2 chokes on some floats very easily. Measuring size escpecially
             // becomes terribly slow
             TableElement tableEl = Document.get().createTableElement();
@@ -174,7 +173,7 @@ public class ChildComponentContainer extends Panel {
         // setWidth("0px");
         contSize.setHeight(0);
         contSize.setWidth(0);
-        containerMarginLeft = 0;
+        // containerMarginLeft = 0;
         containerMarginTop = 0;
         containerDIV.getStyle().setProperty("paddingLeft", "0");
         containerDIV.getStyle().setProperty("paddingTop", "0");
@@ -234,7 +233,7 @@ public class ChildComponentContainer extends Panel {
     }
 
     public void setMarginLeft(int marginLeft) {
-        containerMarginLeft = marginLeft;
+        // containerMarginLeft = marginLeft;
         containerDIV.getStyle().setPropertyPx("paddingLeft", marginLeft);
     }
 
index 2b21f121d3ee1100f33503272a69c5b03235c2c2..b30866441c59e32c979ac3116d956ad9366d9072 100644 (file)
@@ -82,9 +82,9 @@ public class ApplicationRunnerServlet extends AbstractApplicationServlet {
 
     private static class URIS {
         String staticFilesPath;
-        String applicationURI;
-        String context;
-        String runner;
+        // String applicationURI;
+        // String context;
+        // String runner;
         String applicationClassname;
 
     }
@@ -108,38 +108,38 @@ public class ApplicationRunnerServlet extends AbstractApplicationServlet {
         final String[] urlParts = request.getRequestURI().toString()
                 .split("\\/");
         String context = null;
-        String runner = null;
+        // String runner = null;
         URIS uris = new URIS();
         String applicationClassname = null;
         String contextPath = request.getContextPath();
         if (urlParts[1].equals(contextPath.replaceAll("\\/", ""))) {
             // class name comes after web context and runner application
             context = urlParts[1];
-            runner = urlParts[2];
+            // runner = urlParts[2];
             if (urlParts.length == 3) {
                 throw new IllegalArgumentException("No application specified");
             }
             applicationClassname = urlParts[3];
 
             uris.staticFilesPath = "/" + context;
-            uris.applicationURI = "/" + context + "/" + runner + "/"
-                    + applicationClassname;
-            uris.context = context;
-            uris.runner = runner;
+            // uris.applicationURI = "/" + context + "/" + runner + "/"
+            // + applicationClassname;
+            // uris.context = context;
+            // uris.runner = runner;
             uris.applicationClassname = applicationClassname;
         } else {
             // no context
             context = "";
-            runner = urlParts[1];
+            // runner = urlParts[1];
             if (urlParts.length == 2) {
                 throw new IllegalArgumentException("No application specified");
             }
             applicationClassname = urlParts[2];
 
             uris.staticFilesPath = "/";
-            uris.applicationURI = "/" + runner + "/" + applicationClassname;
-            uris.context = context;
-            uris.runner = runner;
+            // uris.applicationURI = "/" + runner + "/" + applicationClassname;
+            // uris.context = context;
+            // uris.runner = runner;
             uris.applicationClassname = applicationClassname;
         }
         return uris;
index a741d6dfedc817dd7921df825114f67fd4444bdc..0043679c75729e6ac76b4d851be1217e65ad779f 100644 (file)
@@ -24,8 +24,6 @@ public class DragAndDropService implements VariableOwner {
 
     private int lastVisitId;
 
-    private int currentEventId;
-
     private boolean lastVisitAccepted = false;
 
     private DragAndDropEvent dragEvent;
@@ -159,7 +157,6 @@ public class DragAndDropService implements VariableOwner {
     private Transferable constructTransferable(DropTarget dropHandlerOwner,
             Map<String, Object> variables) {
         int eventId = (Integer) variables.get("eventId");
-        currentEventId = eventId;
 
         final Component sourceComponent = (Component) variables
                 .get("component");
index 75cefe93605df5bc56378f516cfb9c8e9a13144f..b8f7a9758dcb81fa8bc2ce1ae04bd0002de78f7a 100644 (file)
@@ -12,8 +12,6 @@ public class ComboBoxValueUpdate extends TestBase {
         return 2451;
     }
 
-    private Label selectedLabel;
-
     @Override
     protected String getDescription() {
         return "Testcase for ComboBox. Test especially edge values(of page changes) when selecting items with keyboard only.";
index f9f56ddbfed531913931552f428c080c492c7ea6..54b2f986a43da3d2e8cba6b51646912e2d999f43 100644 (file)
@@ -10,7 +10,6 @@ public class MenuBarNavigation extends TestBase implements Command {
 \r
     private MenuItem edit;\r
     private MenuItem file;\r
-    private MenuItem help;\r
     private Log log;\r
     private MenuItem export;\r
 \r
@@ -30,7 +29,7 @@ public class MenuBarNavigation extends TestBase implements Command {
         edit.addItem("Copy", this);\r
         edit.addItem("Cut", this);\r
         edit.addItem("Paste", this);\r
-        help = mb.addItem("Help", this);\r
+        mb.addItem("Help", this);\r
 \r
         addComponent(mb);\r
 \r
index a2c96d291a2bf0112c59c9317d09739a925eba3c..13ca9f8cf31cd352f1b92b342f0cf762450bba3a 100644 (file)
@@ -57,7 +57,8 @@ public class TextFieldRelativeWidth extends TestBase {
 \r
                 public void valueChange(\r
                         com.vaadin.data.Property.ValueChangeEvent event) {\r
-                    IndexedContainer idc = (IndexedContainer) getContainerDataSource();\r
+                    // IndexedContainer idc = (IndexedContainer)\r
+                    // getContainerDataSource();\r
 \r
                 }\r
 \r
index 4406f1c68da8b0257b386cdf0fb4056ac0ce5b57..38bf07ef59865d88d1fdcfdfa87181cd95ba22a8 100644 (file)
@@ -1,6 +1,5 @@
 package com.vaadin.tests.components.tree;\r
 \r
-import com.vaadin.data.Item;\r
 import com.vaadin.tests.components.AbstractTestCase;\r
 import com.vaadin.ui.RichTextArea;\r
 import com.vaadin.ui.TextField;\r
@@ -22,10 +21,12 @@ public class TreeScrolling extends AbstractTestCase {
         Tree tree = new Tree();\r
         for (int i = 0; i < 20; i++) {\r
             String parentId = "Item " + i;\r
-            Item parentItem = tree.addItem(parentId);\r
+            // Item parentItem =\r
+            tree.addItem(parentId);\r
             for (int j = 0; j < 20; j++) {\r
                 String subId = "Item " + i + " - " + j;\r
-                Item subItem = tree.addItem(subId);\r
+                // Item subItem =\r
+                tree.addItem(subId);\r
                 tree.setParent(subId, parentId);\r
             }\r
 \r
index d8ee2e46b97eae8ec7e6c7c32ea9469631f60554..8985f1cf973bf322966837840a572527784725a7 100644 (file)
@@ -30,12 +30,10 @@ import com.vaadin.ui.themes.Reindeer;
 \r
 public class GridLayoutTests extends AbstractLayoutTests {\r
 \r
-    private Application application;\r
     private AbstractComponent rc1, col1, col2, col3, row1, row2, row3, x3, x22;\r
 \r
     public GridLayoutTests(Application application) {\r
         super();\r
-        this.application = application;\r
     }\r
 \r
     @Override\r
index 711469e79e286ff101003e6bba9a8eb3c96d555b..9db1f7f44f115013240a15fbe476d71fd6984e88 100644 (file)
@@ -26,11 +26,8 @@ import com.vaadin.ui.themes.Reindeer;
 \r
 public class HorizontalLayoutTests extends AbstractLayoutTests {\r
 \r
-    private Application application;\r
-\r
     public HorizontalLayoutTests(Application application) {\r
         super();\r
-        this.application = application;\r
     }\r
 \r
     @Override\r
index 6edd121576656b1730f773ede8ae3722b43bf8b3..cb52ab10519914b6b3eec6b6433fce36bc8f0300 100644 (file)
@@ -25,11 +25,8 @@ import com.vaadin.ui.VerticalLayout;
 \r
 public class VerticalLayoutTests extends AbstractLayoutTests {\r
 \r
-    private Application application;\r
-\r
     public VerticalLayoutTests(Application application) {\r
         super();\r
-        this.application = application;\r
     }\r
 \r
     @Override\r
index 9a12387464595580da1b4eb077fd2be47b2124f9..7cc1e49f69c09ea744da0624e6f7a03ac1297848 100644 (file)
@@ -24,8 +24,6 @@ public class Ticket2021 extends Application {
             + "Nulla nisi. Sed blandit, ante vitae sagittis volutpat, arcu mauris vehicula risus, vitae posuere felis lectus sit amet purus. Donec nec magna et leo eleifend scelerisque. Suspendisse condimentum pharetra ligula. Curabitur lorem. Pellentesque a augue sit amet enim fermentum placerat. Phasellus ante risus, molestie at, iaculis at, pellentesque non, tellus. Class aptent taciti sociosqu ad litora torquent per conubia nostra, per inceptos himenaeos. Phasellus non urna eget risus tempus imperdiet. Integer est diam, sagittis sit amet, posuere sit amet, bibendum sed, lacus. Aenean adipiscing cursus ipsum. Quisque at elit. Vestibulum vitae nunc. Praesent placerat metus viverra lorem. Cras nec elit congue nisi faucibus feugiat. Nam eget mi. Vestibulum condimentum. Nunc nisl ante, cursus in, dictum ac, lobortis rutrum, mi. Nulla eu nisi. In ultricies vehicula magna."\r
             + "Nunc eros dui, elementum at, ullamcorper eget, varius at, velit. Ut dictum. Cras ullamcorper ante vel tortor. Quisque viverra mauris vulputate quam. Nulla dui. Suspendisse non eros at ipsum faucibus hendrerit. Morbi dignissim pharetra tortor. Etiam malesuada. Mauris lacinia elementum erat. Duis mollis placerat metus. Nunc risus felis, cursus ac, cursus vel, convallis vel, metus. Ut vehicula nibh et nulla. Vivamus id pede. Quisque egestas arcu a ligula. Maecenas vehicula. Quisque sed ligula quis tellus tempus rutrum. Curabitur vel augue sed orci egestas pharetra. Duis pharetra.";\r
 \r
-    private AbstractOrderedLayout orderedLayout;\r
-\r
     @Override\r
     public void init() {\r
         Window w = new Window(getClass().getSimpleName());\r
index 5e85d2c595491a6e7d4c463fe103e2af2d7d4cc7..dc7c3e5c850e12f5703e81e7eba86176d6ed5709 100644 (file)
@@ -65,6 +65,7 @@ public class Ticket2029 extends Application {
         return p;\r
     }\r
 \r
+    @SuppressWarnings("unused")\r
     private Panel createGLPanel() {\r
         Panel p = new Panel("" + DIM1 + "x" + DIM2 + " GridLayout");\r
         p.setWidth("" + DIM1 + "px");\r
index 54546403cd7f3cdf1a8e7165da8464e208b3fd5b..65b5f73b5325ca7b48ca8f28f104a783b508c871 100644 (file)
@@ -10,7 +10,6 @@ import com.vaadin.ui.GridLayout;
 import com.vaadin.ui.Label;\r
 import com.vaadin.ui.Layout;\r
 import com.vaadin.ui.Panel;\r
-import com.vaadin.ui.SplitPanel;\r
 import com.vaadin.ui.VerticalLayout;\r
 import com.vaadin.ui.Window;\r
 \r
@@ -18,8 +17,6 @@ public class Ticket2048 extends Application {
 \r
     private Embedded embedded;\r
     private Panel p;\r
-    private SplitPanel splitPanel;\r
-    private GridLayout gridLayout;\r
     private VerticalLayout orderedLayout;\r
 \r
     @Override\r
index 241907c0beeb4d14a1f08f30fdcf82acccf462e9..3fdf61ac47450978c62f83a003588bc99a5a5163 100644 (file)
@@ -6,9 +6,7 @@ import java.util.List;
 import java.util.Map;
 
 import com.vaadin.Application;
-import com.vaadin.data.Container;
 import com.vaadin.data.Item;
-import com.vaadin.data.Property;
 import com.vaadin.data.util.BeanItem;
 import com.vaadin.ui.AbstractOrderedLayout;
 import com.vaadin.ui.Accordion;
@@ -18,8 +16,8 @@ import com.vaadin.ui.Button.ClickListener;
 import com.vaadin.ui.Component;
 import com.vaadin.ui.ComponentContainer;
 import com.vaadin.ui.Field;
-import com.vaadin.ui.FieldFactory;
 import com.vaadin.ui.Form;
+import com.vaadin.ui.FormFieldFactory;
 import com.vaadin.ui.GridLayout;
 import com.vaadin.ui.Label;
 import com.vaadin.ui.Layout;
@@ -122,40 +120,25 @@ public class Ticket2204 extends Application {
 
         if (c == Form.class) {
             Form f = (Form) cc;
-            f.setFieldFactory(new FieldFactory() {
-
-                public Field createField(Class<?> type, Component uiContext) {
-                    return createField();
-                }
-
-                public Field createField(Property property, Component uiContext) {
-                    return createField();
-                }
+            f.setFormFieldFactory(new FormFieldFactory() {
 
                 public Field createField(Item item, Object propertyId,
                         Component uiContext) {
-                    return createField();
-                }
-
-                private Field createField() {
                     formTextArea = new RichTextArea();
                     formTextArea.setVisible(false);
                     return formTextArea;
                 }
 
-                public Field createField(Container container, Object itemId,
-                        Object propertyId, Component uiContext) {
-                    return createField();
-                }
-
             });
             f.setItemDataSource(new BeanItem<Object>(new Object() {
                 private int a;
 
+                @SuppressWarnings("unused")
                 public int getA() {
                     return a;
                 }
 
+                @SuppressWarnings("unused")
                 public void setA(int a) {
                     this.a = a;
                 }
index 1ddb74187f9cd622c12955c89a02e0f5295789f8..fa25ff55f103ac7d371744c1a98771dca29b20dd 100644 (file)
@@ -70,43 +70,6 @@ public class Ticket2242 extends Application implements ValueChangeListener {
         return ic;
     }
 
-    private static class TestObject {
-        public TestObject(int a, String b, Long c) {
-            super();
-            this.a = a;
-            this.b = b;
-            this.c = c;
-        }
-
-        private int a;
-        private String b;
-        private Long c;
-
-        public int getA() {
-            return a;
-        }
-
-        public void setA(int a) {
-            this.a = a;
-        }
-
-        public String getB() {
-            return b;
-        }
-
-        public void setB(String b) {
-            this.b = b;
-        }
-
-        public Long getC() {
-            return c;
-        }
-
-        public void setC(Long c) {
-            this.c = c;
-        }
-    }
-
     public void valueChange(ValueChangeEvent event) {
         System.out.println("Value change from " + tableValue + " to "
                 + t.getValue());
index b023ed2c142036a0f9b9dd1112a11d21fbae7418..26ef21694085d25f1c8f53cb56c539f941476ace 100644 (file)
@@ -4,7 +4,6 @@ import java.util.Random;
 
 import com.vaadin.Application;
 import com.vaadin.ui.Button;
-import com.vaadin.ui.Panel;
 import com.vaadin.ui.Table;
 import com.vaadin.ui.VerticalLayout;
 import com.vaadin.ui.Window;
@@ -12,8 +11,6 @@ import com.vaadin.ui.themes.BaseTheme;
 
 public class Ticket2344 extends Application {
 
-    private Panel p;
-
     @Override
     public void init() {
         Window main = new Window("Quick test");
index 092d9b8b414fccdeccd549fdecc2c71a9211608c..88ee6dcc55f200d7ef363b1ab9f6836a88cdb33d 100644 (file)
@@ -37,7 +37,6 @@ import com.vaadin.ui.themes.Reindeer;
  */\r
 public class Ticket2998 extends Application {\r
     private Table table;\r
-    private WorkoutEditor editor = new WorkoutEditor(this);\r
     private VerticalLayout mainLayout;\r
 \r
     public class Workout implements Serializable {\r
@@ -105,7 +104,6 @@ public class Ticket2998 extends Application {
         private TextField kilomiters = new TextField("Kilometers");\r
         private TextField title = new TextField("Title/note");\r
 \r
-        private Workout run;\r
         private Ticket2998 workoutLog;\r
 \r
         public WorkoutEditor(Ticket2998 app) {\r
@@ -137,17 +135,13 @@ public class Ticket2998 extends Application {
                     workoutLog.getMainWindow().addWindow(this);\r
                 }\r
                 kilomiters.focus();\r
-                this.run = run;\r
             }\r
         }\r
     }\r
 \r
     public class MyFieldFactory extends DefaultFieldFactory {\r
 \r
-        private BeanItemContainer<String> trainingTypes;\r
-\r
         public MyFieldFactory(Ticket2998 app) {\r
-            trainingTypes = new BeanItemContainer<String>(String.class);\r
         }\r
 \r
         @Override\r