]> source.dussan.org Git - vaadin-framework.git/commitdiff
Update modifier order to follow Java Specification suggestions (#10050)
authorAhmed Ashour <asashour@yahoo.com>
Thu, 21 Sep 2017 12:25:46 +0000 (14:25 +0200)
committerHenri Sara <henri.sara@gmail.com>
Thu, 21 Sep 2017 12:25:46 +0000 (15:25 +0300)
Also includes some other minor cleanup.

116 files changed:
client/src/main/java/com/vaadin/client/ApplicationConfiguration.java
client/src/main/java/com/vaadin/client/ApplicationConnection.java
client/src/main/java/com/vaadin/client/BrowserInfo.java
client/src/main/java/com/vaadin/client/ConnectorMap.java
client/src/main/java/com/vaadin/client/MeasuredSize.java
client/src/main/java/com/vaadin/client/Profiler.java
client/src/main/java/com/vaadin/client/SuperDevMode.java
client/src/main/java/com/vaadin/client/UIDL.java
client/src/main/java/com/vaadin/client/Util.java
client/src/main/java/com/vaadin/client/VCaption.java
client/src/main/java/com/vaadin/client/WidgetUtil.java
client/src/main/java/com/vaadin/client/communication/AtmospherePushConnection.java
client/src/main/java/com/vaadin/client/communication/MessageHandler.java
client/src/main/java/com/vaadin/client/communication/StateChangeEvent.java
client/src/main/java/com/vaadin/client/componentlocator/LegacyLocatorStrategy.java
client/src/main/java/com/vaadin/client/connectors/grid/ColumnConnector.java
client/src/main/java/com/vaadin/client/data/AbstractRemoteDataSource.java
client/src/main/java/com/vaadin/client/data/CacheStrategy.java
client/src/main/java/com/vaadin/client/event/PointerEvent.java
client/src/main/java/com/vaadin/client/renderers/ClickableRenderer.java
client/src/main/java/com/vaadin/client/renderers/HierarchyRenderer.java
client/src/main/java/com/vaadin/client/ui/VAbstractCalendarPanel.java
client/src/main/java/com/vaadin/client/ui/VAbstractSplitPanel.java
client/src/main/java/com/vaadin/client/ui/VDragAndDropWrapper.java
client/src/main/java/com/vaadin/client/ui/VFormLayout.java
client/src/main/java/com/vaadin/client/ui/VPopupView.java
client/src/main/java/com/vaadin/client/ui/VUI.java
client/src/main/java/com/vaadin/client/ui/csslayout/CssLayoutConnector.java
client/src/main/java/com/vaadin/client/ui/dd/VAbstractDropHandler.java
client/src/main/java/com/vaadin/client/ui/dd/VAcceptAll.java
client/src/main/java/com/vaadin/client/ui/dd/VAnd.java
client/src/main/java/com/vaadin/client/ui/dd/VContainsDataFlavor.java
client/src/main/java/com/vaadin/client/ui/dd/VDragSourceIs.java
client/src/main/java/com/vaadin/client/ui/dd/VHtml5File.java
client/src/main/java/com/vaadin/client/ui/dd/VOverTreeNode.java
client/src/main/java/com/vaadin/client/ui/menubar/MenuBarConnector.java
client/src/main/java/com/vaadin/client/widget/escalator/ScrollbarBundle.java
client/src/main/java/com/vaadin/client/widget/escalator/events/RowHeightChangedEvent.java
client/src/main/java/com/vaadin/client/widget/grid/events/ColumnReorderEvent.java
client/src/main/java/com/vaadin/client/widget/grid/events/ColumnResizeEvent.java
client/src/main/java/com/vaadin/client/widget/grid/events/ColumnVisibilityChangeEvent.java
client/src/main/java/com/vaadin/client/widgets/Escalator.java
client/src/main/java/com/vaadin/client/widgets/Grid.java
client/src/main/java/com/vaadin/client/widgets/Overlay.java
compatibility-client/src/main/java/com/vaadin/v7/client/renderers/ClickableRenderer.java
compatibility-client/src/main/java/com/vaadin/v7/client/ui/VCalendar.java
compatibility-client/src/main/java/com/vaadin/v7/client/ui/VCalendarPanel.java
compatibility-client/src/main/java/com/vaadin/v7/client/ui/dd/VIsOverId.java
compatibility-client/src/main/java/com/vaadin/v7/client/ui/dd/VItemIdIs.java
compatibility-client/src/main/java/com/vaadin/v7/client/ui/tree/VTargetInSubtree.java
compatibility-client/src/main/java/com/vaadin/v7/client/widget/escalator/ScrollbarBundle.java
compatibility-client/src/main/java/com/vaadin/v7/client/widget/escalator/events/RowHeightChangedEvent.java
compatibility-client/src/main/java/com/vaadin/v7/client/widget/grid/events/ColumnReorderEvent.java
compatibility-client/src/main/java/com/vaadin/v7/client/widget/grid/events/ColumnResizeEvent.java
compatibility-client/src/main/java/com/vaadin/v7/client/widget/grid/events/ColumnVisibilityChangeEvent.java
compatibility-client/src/main/java/com/vaadin/v7/client/widget/grid/events/SelectAllEvent.java
compatibility-client/src/main/java/com/vaadin/v7/client/widgets/Escalator.java
compatibility-client/src/main/java/com/vaadin/v7/client/widgets/Grid.java
compatibility-server/src/main/java/com/vaadin/v7/data/util/AbstractInMemoryContainer.java
compatibility-server/src/main/java/com/vaadin/v7/data/util/FilesystemContainer.java
compatibility-server/src/main/java/com/vaadin/v7/data/util/PropertyFormatter.java
compatibility-server/src/main/java/com/vaadin/v7/data/util/TextFileProperty.java
compatibility-server/src/main/java/com/vaadin/v7/data/util/converter/DefaultConverterFactory.java
compatibility-server/src/main/java/com/vaadin/v7/data/util/sqlcontainer/ColumnProperty.java
compatibility-server/src/main/java/com/vaadin/v7/data/util/sqlcontainer/SQLContainer.java
compatibility-server/src/main/java/com/vaadin/v7/data/util/sqlcontainer/query/generator/filter/QueryBuilder.java
compatibility-server/src/main/java/com/vaadin/v7/event/FieldEvents.java
compatibility-server/src/main/java/com/vaadin/v7/ui/AbstractColorPicker.java
compatibility-server/src/main/java/com/vaadin/v7/ui/AbstractSelect.java
compatibility-server/src/main/java/com/vaadin/v7/ui/Grid.java
compatibility-server/src/main/java/com/vaadin/v7/ui/Table.java
compatibility-server/src/main/java/com/vaadin/v7/ui/Tree.java
compatibility-server/src/test/java/com/vaadin/v7/data/util/BeanItemContainerGenerator.java
compatibility-server/src/test/java/com/vaadin/v7/data/util/PerformanceTestIndexedContainerTest.java
compatibility-server/src/test/java/com/vaadin/v7/tests/server/ContextClickListenerTest.java
server/src/main/java/com/vaadin/data/provider/DataCommunicator.java
server/src/main/java/com/vaadin/event/FieldEvents.java
server/src/main/java/com/vaadin/event/ShortcutListener.java
server/src/main/java/com/vaadin/event/dd/acceptcriteria/Or.java
server/src/main/java/com/vaadin/server/ComponentSizeValidator.java
server/src/main/java/com/vaadin/server/JsonPaintTarget.java
server/src/main/java/com/vaadin/server/VaadinPortlet.java
server/src/main/java/com/vaadin/server/VaadinServlet.java
server/src/main/java/com/vaadin/server/communication/FileUploadHandler.java
server/src/main/java/com/vaadin/server/communication/PushHandler.java
server/src/main/java/com/vaadin/server/widgetsetutils/ClassPathExplorer.java
server/src/main/java/com/vaadin/ui/AbstractLocalDateField.java
server/src/main/java/com/vaadin/ui/Composite.java
server/src/main/java/com/vaadin/ui/MenuBar.java
server/src/main/java/com/vaadin/ui/Tree.java
server/src/main/java/com/vaadin/ui/declarative/DesignAttributeHandler.java
server/src/main/java/com/vaadin/util/FileTypeResolver.java
server/src/test/java/com/vaadin/tests/server/ClassesSerializableTest.java
server/src/test/java/com/vaadin/tests/server/ClasspathHelper.java
uitest/src/main/java/com/vaadin/launcher/ApplicationRunnerServlet.java
uitest/src/main/java/com/vaadin/launcher/DevelopmentServerLauncher.java
uitest/src/main/java/com/vaadin/tests/components/AbstractComponentTestCase.java
uitest/src/main/java/com/vaadin/tests/components/button/ButtonClick.java
uitest/src/main/java/com/vaadin/tests/components/menubar/MenuBarsWithNesting.java
uitest/src/main/java/com/vaadin/tests/components/table/TableNavigationPageDown.java
uitest/src/main/java/com/vaadin/tests/components/tabsheet/TabKeyboardNavigation.java
uitest/src/main/java/com/vaadin/tests/dd/DDTest2.java
uitest/src/main/java/com/vaadin/tests/dd/DDTest6.java
uitest/src/main/java/com/vaadin/tests/elements/notification/NotificationGetTypeAndDescription.java
uitest/src/main/java/com/vaadin/tests/layouts/layouttester/BaseAlignment.java
uitest/src/main/java/com/vaadin/tests/util/PortableRandom.java
uitest/src/main/java/com/vaadin/tests/widgetset/client/TestWidgetConnector.java
uitest/src/main/java/com/vaadin/tests/widgetset/client/csrf/CsrfButtonConnector.java
uitest/src/main/java/com/vaadin/tests/widgetset/client/v7/grid/GridBasicClientFeaturesWidget.java
uitest/src/main/java/com/vaadin/tests/widgetset/server/gwtrpc/GwtRpc.java
uitest/src/test/java/com/vaadin/tests/components/grid/basicfeatures/escalator/EscalatorColumnFreezingTest.java
uitest/src/test/java/com/vaadin/tests/components/grid/basicfeatures/escalator/EscalatorSpacerTest.java
uitest/src/test/java/com/vaadin/tests/components/tabsheet/TabKeyboardNavigationTest.java
uitest/src/test/java/com/vaadin/tests/components/tabsheet/TabSelectionRevertedByServerTest.java
uitest/src/test/java/com/vaadin/tests/elements/abstracttextfield/AbstractTextElementSetValueTest.java
uitest/src/test/java/com/vaadin/tests/elements/abstracttextfield/CompatibilityAbstractTextElementSetValueTest.java

index 0ee4a4f7081fec18f062962fe1d05b888f22525e..41ce18e8ee28c9ca54f5ce8ea5d9e24d2d1cd7db 100644 (file)
@@ -495,7 +495,7 @@ public class ApplicationConfiguration implements EntryPoint {
      *            the id of the application to get configuration data for
      * @return a native javascript object containing the configuration data
      */
-    private native static JsoConfiguration getJsoConfiguration(String appId)
+    private static native JsoConfiguration getJsoConfiguration(String appId)
     /*-{
         return $wnd.vaadin.getApp(appId);
      }-*/;
@@ -838,7 +838,7 @@ public class ApplicationConfiguration implements EntryPoint {
      * @param widgetsetName
      *            the name of this widgetset
      */
-    public native static void registerCallback(String widgetsetName)
+    public static native void registerCallback(String widgetsetName)
     /*-{
         var callbackHandler = $entry(@com.vaadin.client.ApplicationConfiguration::startApplication(Ljava/lang/String;));
         $wnd.vaadin.registerWidgetset(widgetsetName, callbackHandler);
@@ -872,7 +872,7 @@ public class ApplicationConfiguration implements EntryPoint {
         return !isDebugAvailable();
     }
 
-    private native static boolean isDebugAvailable()
+    private static native boolean isDebugAvailable()
     /*-{
         if($wnd.vaadin.debug) {
             return true;
index 5a67d41a541018db447a13fca306227d10f7c088..58d5259f1beff3da6414ca3eb6d0631ca1828398 100644 (file)
@@ -195,7 +195,7 @@ public class ApplicationConnection implements HasHandlers {
         }
     }
 
-    public static abstract class ApplicationConnectionEvent
+    public abstract static class ApplicationConnectionEvent
             extends GwtEvent<CommunicationHandler> {
 
         private ApplicationConnection connection;
index 6e0b8589909add7c235d22bc4e0eeb445babe5d3..0197c4f3f860d1d42df886cb76c67cd2ae508dbd 100644 (file)
@@ -351,7 +351,7 @@ public class BrowserInfo {
         return browserDetails.isOpera() && getBrowserMajorVersion() == 11;
     }
 
-    public native static String getBrowserString()
+    public static native String getBrowserString()
     /*-{
        return $wnd.navigator.userAgent;
     }-*/;
index f79aa0250f91fb3be78b19bb4fd3faa97410d82e..fbd5b40e3f66a8d31a132ce599fbde225ee7f12f 100644 (file)
@@ -139,7 +139,7 @@ public class ConnectorMap {
      *            element of the connector whose id is desired
      * @return the id of the element's connector, if it's a connector
      */
-    native static final String getConnectorId(Element el)
+    static final native String getConnectorId(Element el)
     /*-{
         return el.tkPid;
     }-*/;
index 0f02b46e5ce2be5e38523febca12266909a1d435..2029fd9e4d31f11fd69e84ccdfed4b3f4e5208db 100644 (file)
@@ -21,7 +21,7 @@ import com.google.gwt.core.client.JsArrayString;
 import com.google.gwt.dom.client.Element;
 
 public class MeasuredSize {
-    private final static boolean debugSizeChanges = false;
+    private static final boolean debugSizeChanges = false;
 
     public static class MeasureResult {
         private final boolean widthChanged;
index 3986bdddbb12ce26ab0dbd124f9cc62a8b290f9c..bc07836b89c941dd296ae04234a3b156d053c649 100644 (file)
@@ -384,7 +384,7 @@ public class Profiler {
         return RELATIVE_TIME_SUPPLIER.getRelativeTime();
     }
 
-    private static native final void logGwtEvent(String name, String type)
+    private static final native void logGwtEvent(String name, String type)
     /*-{
         $wnd.__gwtStatsEvent({
             evtGroup: @com.vaadin.client.Profiler::evtGroup,
index e8eab082d1e2c76aaf73ca56af98296afeab02b0..fa565c4ce857c1beb9a08ffb02ee243ea6ffab9c 100644 (file)
@@ -122,7 +122,7 @@ public class SuperDevMode {
                 + ".nocache.js";
     }
 
-    private native static String getRecompileParameters(String moduleName)
+    private static native String getRecompileParameters(String moduleName)
     /*-{
         var prop_map = $wnd.__gwt_activeModules[moduleName].bindings();
     
@@ -217,7 +217,7 @@ public class SuperDevMode {
         return isSuperDevModeEnabledInModule(moduleName);
     }
 
-    protected native static boolean isSuperDevModeEnabledInModule(
+    protected static native boolean isSuperDevModeEnabledInModule(
             String moduleName)
     /*-{
         if (!$wnd.__gwt_activeModules)
index 165f0c6b0574a321c6aa62cd0131d1a45f9bd9cc..05a2e5f5ef1789c5acdbabf0533b1e8ac55d063b 100644 (file)
@@ -476,7 +476,7 @@ public final class UIDL extends JavaScriptObject {
      * @deprecated should not be used anymore
      */
     @Deprecated
-    public final static class XML extends JavaScriptObject {
+    public static final class XML extends JavaScriptObject {
         protected XML() {
         }
 
index 553911b7b87d6b9b614ec78a3e45df4d2bf295e8..b849d0c0b353e487a6e366db20ea4d13fe926cd4 100644 (file)
@@ -1149,7 +1149,7 @@ public class Util {
      *            a JavaScript object to be converted to a string
      * @return JSON in string representation
      */
-    private native static String stringify(JavaScriptObject json)
+    private static native String stringify(JavaScriptObject json)
     /*-{
         return JSON.stringify(json);
     }-*/;
@@ -1162,7 +1162,7 @@ public class Util {
      * @param jsonAsString
      * @return a JavaScript object constructed from the parse
      */
-    public native static <T extends JavaScriptObject> T parse(
+    public static native <T extends JavaScriptObject> T parse(
             String jsonAsString)
     /*-{
         return JSON.parse(jsonAsString);
index 8da36becfa82b4928d06a14904b0e8bf83cb48ce..1bff175d7e713e5bec5531628a4fe416c29f6ec9 100644 (file)
@@ -692,12 +692,12 @@ public class VCaption extends HTML {
         return getOwnerPid(e);
     }
 
-    private native static void setOwnerPid(Element el, String pid)
+    private static native void setOwnerPid(Element el, String pid)
     /*-{
         el.vOwnerPid = pid;
     }-*/;
 
-    public native static String getOwnerPid(Element el)
+    public static native String getOwnerPid(Element el)
     /*-{
         return el.vOwnerPid;
     }-*/;
index d24a1d53a59be5e6546024f599659a6307b44fe4..7fc027d55de4fb5f216dcc9be1a5be4def6961e5 100644 (file)
@@ -1167,7 +1167,7 @@ public class WidgetUtil {
      *
      * @return The active element or null if no active element could be found.
      */
-    public native static Element getFocusedElement()
+    public static native Element getFocusedElement()
     /*-{
        if ($wnd.document.activeElement) {
            return $wnd.document.activeElement;
@@ -1315,7 +1315,7 @@ public class WidgetUtil {
      *
      * @since 7.3
      */
-    public native static void setSelectionRange(Element elem, int pos,
+    public static native void setSelectionRange(Element elem, int pos,
             int length, String direction)
     /*-{
        try {
@@ -1335,7 +1335,7 @@ public class WidgetUtil {
      *            <code>true</code> if selection is enabled; <code>false</code>
      *            if not
      */
-    public native static void setTextSelectionEnabled(Element e, boolean enable)
+    public static native void setTextSelectionEnabled(Element e, boolean enable)
     /*-{
         if (!enable) {
             e.ondrag = function () { return false; };
@@ -1353,7 +1353,7 @@ public class WidgetUtil {
      *
      * @since 7.6
      */
-    public native static void clearTextSelection()
+    public static native void clearTextSelection()
     /*-{
         if ($wnd.getSelection) {
             $wnd.getSelection().removeAllRanges();
index 511a37b7e6c38991cfd39e333e341a6fb24d5efd..ee0aab7b2e0029922391f96d8946d6605890e719 100644 (file)
@@ -417,9 +417,8 @@ public class AtmospherePushConnection implements PushConnection {
         getConnectionStateHandler().pushReconnectPending(this);
     }
 
-    public static abstract class AbstractJSO extends JavaScriptObject {
+    public abstract static class AbstractJSO extends JavaScriptObject {
         protected AbstractJSO() {
-
         }
 
         protected final native String getStringValue(String key)
index 6e13eb07eb9174e4308dce48aa0e8006a1911c7a..90e822649b6bc202af6be5386f83ab6e8fa87cc4 100644 (file)
@@ -1613,7 +1613,7 @@ public class MessageHandler {
         }
     }
 
-    private static native final int calculateBootstrapTime()
+    private static final native int calculateBootstrapTime()
     /*-{
         if ($wnd.performance && $wnd.performance.timing) {
             return (new Date).getTime() - $wnd.performance.timing.responseStart;
index 0012138ad9b0846c8bdce6401f3a3e3467d7dc16..80e5a32c8bfff1273d723cab1f2d017ed786a30b 100644 (file)
@@ -234,7 +234,7 @@ public class StateChangeEvent
      *            the JavaScript object to check
      * @return true if the property is defined
      */
-    private static native final boolean isInJson(String property,
+    private static final native boolean isInJson(String property,
             JavaScriptObject target)
     /*-{
         var segments = property.split('.');
index dca4ad7b3a6a702f102543255e267e6fdb7646cb..76360eb40d04d39c11c5c8c8cfdd68619b6647ec 100644 (file)
@@ -206,7 +206,7 @@ public class LegacyLocatorStrategy implements LocatorStrategy {
          * Path is of type "targetWidgetPath#componentPart" or
          * "targetWidgetPath".
          */
-        String parts[] = path.split(LegacyLocatorStrategy.SUBPART_SEPARATOR, 2);
+        String[] parts = path.split(LegacyLocatorStrategy.SUBPART_SEPARATOR, 2);
         String widgetPath = parts[0];
 
         // Note that this only works if baseElement can be mapped to a
@@ -328,7 +328,7 @@ public class LegacyLocatorStrategy implements LocatorStrategy {
      *         if the element could not be found.
      */
     private Element getElementByDOMPath(Element baseElement, String path) {
-        String parts[] = path.split(PARENTCHILD_SEPARATOR);
+        String[] parts = path.split(PARENTCHILD_SEPARATOR);
         Element element = baseElement;
 
         for (int i = 0, l = parts.length; i < l; ++i) {
@@ -501,7 +501,7 @@ public class LegacyLocatorStrategy implements LocatorStrategy {
     @SuppressWarnings("unchecked")
     private Widget getWidgetFromPath(String path, Widget baseWidget) {
         Widget w = baseWidget;
-        String parts[] = path.split(PARENTCHILD_SEPARATOR);
+        String[] parts = path.split(PARENTCHILD_SEPARATOR);
 
         for (int i = 0; i < parts.length; i++) {
             String part = parts[i];
index 7ac6c518adfe79121c04a16fcdc88029786f3ab5..b9d14dc91c1309c343982edbbfbd174e2444b784 100644 (file)
@@ -37,7 +37,7 @@ import elemental.json.JsonValue;
 @Connect(com.vaadin.ui.Grid.Column.class)
 public class ColumnConnector extends AbstractExtensionConnector {
 
-    public static abstract class CustomColumn
+    public abstract static class CustomColumn
             extends Column<Object, JsonObject> {
 
         private final String connectorId;
index c03d3b2936393b5b43fd87e352d5294b2ea96784..ce1acef64107b9c635c33a747fc69dba774cbbdc 100644 (file)
@@ -882,7 +882,7 @@ public abstract class AbstractRemoteDataSource<T> implements DataSource<T> {
      * @return a non-null object that uniquely and consistently represents the
      *         row object
      */
-    abstract public Object getRowKey(T row);
+    public abstract Object getRowKey(T row);
 
     @Override
     public int size() {
index acfadcbcf796ecf3b2e4819e23115a61923c8f58..15aaf531e6eb2422671f440271fe6ca720546f3e 100644 (file)
@@ -33,7 +33,7 @@ public interface CacheStrategy {
      * This simple approach rules out more advanced heuristics that would take
      * the current scrolling direction or past scrolling behavior into account.
      */
-    public static abstract class AbstractBasicSymmetricalCacheStrategy
+    public abstract static class AbstractBasicSymmetricalCacheStrategy
             implements CacheStrategy {
 
         @Override
index 2007ad66242d0a572932972f26cf8f362d91d6fb..49e2382a0ed00ce5cd03b356c074052a8de5dd03 100644 (file)
@@ -126,37 +126,37 @@ public abstract class PointerEvent<H extends EventHandler>
         return isPrimary(getNativeEvent());
     }
 
-    private static native final int getPointerId(NativeEvent e)
+    private static final native int getPointerId(NativeEvent e)
     /*-{
       return e.pointerId;
     }-*/;
 
-    private static native final int getWidth(NativeEvent e)
+    private static final native int getWidth(NativeEvent e)
     /*-{
       return e.width;
     }-*/;
 
-    private static native final int getHeight(NativeEvent e)
+    private static final native int getHeight(NativeEvent e)
     /*-{
       return e.height;
     }-*/;
 
-    private static native final double getPressure(NativeEvent e)
+    private static final native double getPressure(NativeEvent e)
     /*-{
       return e.pressure;
     }-*/;
 
-    private static native final double getTiltX(NativeEvent e)
+    private static final native double getTiltX(NativeEvent e)
     /*-{
       return e.tiltX;
     }-*/;
 
-    private static native final double getTiltY(NativeEvent e)
+    private static final native double getTiltY(NativeEvent e)
     /*-{
       return e.tiltY;
     }-*/;
 
-    private static native final String getPointerType(NativeEvent e)
+    private static final native String getPointerType(NativeEvent e)
     /*-{
       var pointerType = e.pointerType;
       if (typeof pointerType === "number") {
@@ -165,7 +165,7 @@ public abstract class PointerEvent<H extends EventHandler>
       return pointerType || "";
     }-*/;
 
-    private static native final boolean isPrimary(NativeEvent e)
+    private static final native boolean isPrimary(NativeEvent e)
     /*-{
       return e.isPrimary;
     }-*/;
index 34a2df4f183d9af12f89fe586428e99e232b49bd..45f36561f46b03169b486468d7d88b5bfb082700 100644 (file)
@@ -160,7 +160,7 @@ public abstract class ClickableRenderer<T, W extends Widget>
             return cellReference;
         }
 
-        private native static Escalator getEscalator(Grid<?> grid)
+        private static native Escalator getEscalator(Grid<?> grid)
         /*-{
           return grid.@com.vaadin.client.widgets.Grid::escalator;
         }-*/;
index 03f6525c1c3f89eec2d969c9e10513d915c97f33..9bedf65b706f58855fb93d79251bc45e8d09f151 100644 (file)
@@ -71,12 +71,12 @@ public class HierarchyRenderer extends ClickableRenderer<Object, Widget> {
             return (TableCellElement) element;
         }
 
-        private native static RowReference<Object> getRowReference(
+        private static native RowReference<Object> getRowReference(
                 RendererCellReference cell) /*-{
             return cell.@com.vaadin.client.widget.grid.CellReference::getRowReference()();
         }-*/;
 
-        private native static FlyweightCell getFlyweightCell(
+        private static native FlyweightCell getFlyweightCell(
                 RendererCellReference cell) /*-{
             return cell.@com.vaadin.client.widget.grid.RendererCellReference::cell;
         }-*/;
index 6d6f396147e89ddd6bf0e75241d423f94991ba80..f0a23b5f43c72a784c480c35b1ed4a8cc977381b 100644 (file)
@@ -2010,7 +2010,7 @@ public abstract class VAbstractCalendarPanel<R extends Enum<R>>
 
         private void setLabel() {
             if (getDateField() instanceof VAbstractPopupCalendar) {
-                ((VAbstractPopupCalendar) getDateField()).setFocusedDate(this);
+                ((VAbstractPopupCalendar<?, ?>) getDateField()).setFocusedDate(this);
             }
         }
     }
index 5f75869ba4fd9ed4ead2ffae572a76d6df2aa896..b12adecc625c6e0b586f8bb69a23f44bd01c48d5 100644 (file)
@@ -606,14 +606,14 @@ public abstract class VAbstractSplitPanel extends ComplexPanel {
      *
      * @since 7.5.1
      */
-    abstract protected void startResize();
+    protected abstract void startResize();
 
     /**
      * Called when stopping drag resize
      *
      * @since 7.5.1
      */
-    abstract protected void stopResize();
+    protected abstract void stopResize();
 
     /**
      * Gets the first container
index 0595d65069e7b48fab56acf0f111c07ec98ce1cf..d12a233e73b7f0110500e9f21a3a3f3e596a99da 100644 (file)
@@ -170,11 +170,11 @@ public class VDragAndDropWrapper extends VCustomComponent
         return false;
     }
 
-    protected final static int NONE = 0;
-    protected final static int COMPONENT = 1;
-    protected final static int WRAPPER = 2;
-    protected final static int HTML5 = 3;
-    protected final static int COMPONENT_OTHER = 4;
+    protected static final int NONE = 0;
+    protected static final int COMPONENT = 1;
+    protected static final int WRAPPER = 2;
+    protected static final int HTML5 = 3;
+    protected static final int COMPONENT_OTHER = 4;
 
     /** For internal use only. May be removed or replaced in the future. */
     public int dragStartMode;
index 18d7d76d74a737ab6f3fc1c52ddf9b168bcb5c77..a8a43a2d8d1566d7ffc88e918c6eca8b36ec9904 100644 (file)
@@ -45,7 +45,7 @@ import com.vaadin.shared.ui.MarginInfo;
  */
 public class VFormLayout extends SimplePanel {
 
-    private final static String CLASSNAME = "v-formlayout";
+    private static final String CLASSNAME = "v-formlayout";
 
     /** For internal use only. May be removed or replaced in the future. */
     public VFormLayoutTable table;
index 296cde28bcaeb5feae3285735913084d3c5f50ad..43da119c5a3a078f58689d0ca6b0b8d260755a68 100644 (file)
@@ -416,7 +416,7 @@ public class VPopupView extends HTML
             }
             return handler;
         }
-    }// class CustomPopup
+    }
 
     public HandlerRegistration addVisibilityChangeHandler(
             final VisibilityChangeHandler visibilityChangeHandler) {
@@ -442,4 +442,4 @@ public class VPopupView extends HTML
         return popupShowInProgress;
     }
 
-}// class VPopupView
+}
index 8a159a1f1ef7826ae8ef606a79f37db979d02200..a372b746b02117efc9d68008c3a8e549dd20e3d1 100644 (file)
@@ -343,7 +343,7 @@ public class VUI extends SimplePanel implements ResizeHandler,
         connection.flushActiveConnector();
     }
 
-    private native static void loadAppIdListFromDOM(List<String> list)
+    private static native void loadAppIdListFromDOM(List<String> list)
     /*-{
          var j;
          for(j in $wnd.vaadin.vaadinConfigurations) {
index 8b90fffeb6e01e7ab9eb8f7c012888fe17cac6cb..89bd7923cd16c6c6cf903395d62184d64eaa1068 100644 (file)
@@ -93,7 +93,7 @@ public class CssLayoutConnector extends AbstractLayoutConnector {
             // as it is added directly to the child component?
             String[] cssRules = css.split(";");
             for (String cssRule : cssRules) {
-                String parts[] = cssRule.split(":", 2);
+                String[] parts = cssRule.split(":", 2);
                 if (parts.length == 2) {
                     style.setProperty(makeCamelCase(parts[0].trim()),
                             parts[1].trim());
index 883b5e51644a6ecb1b0941eaf756e179a67160d0..c88a3079fb5d2917b76e9826cc717f17486d41dc 100644 (file)
@@ -113,7 +113,7 @@ public abstract class VAbstractDropHandler implements VDropHandler {
      *
      * @param drag
      */
-    abstract protected void dragAccepted(VDragEvent drag);
+    protected abstract void dragAccepted(VDragEvent drag);
 
     protected void validate(final VAcceptCallback cb, final VDragEvent event) {
         Command checkCriteria = new Command() {
index f7c75744765c11cf89b6d5dbd5bbb9f853470aa5..aa904bf52fdab0b07f19391919bd6890a6571803 100644 (file)
@@ -31,7 +31,7 @@ import com.vaadin.ui.dnd.DropTargetExtension;
  */
 @Deprecated
 @AcceptCriterion(AcceptAll.class)
-final public class VAcceptAll extends VAcceptCriterion {
+public final class VAcceptAll extends VAcceptCriterion {
 
     @Override
     protected boolean accept(VDragEvent drag, UIDL configuration) {
index 1aebfa2e27665dbe31b667c9b11a6e534538ae50..47deb06632ace6fc66b3b1cd05799ba023dc6edf 100644 (file)
@@ -28,7 +28,7 @@ import com.vaadin.ui.dnd.DropTargetExtension;
  */
 @Deprecated
 @AcceptCriterion(And.class)
-final public class VAnd extends VAcceptCriterion implements VAcceptCallback {
+public final class VAnd extends VAcceptCriterion implements VAcceptCallback {
     private boolean b1;
 
     static VAcceptCriterion getCriteria(VDragEvent drag, UIDL configuration,
index 98565713b2976ae99b9fef29748bb46e2f7da9e3..22c25705988a3d020d4d5071962e90ad44148b38 100644 (file)
@@ -28,7 +28,7 @@ import com.vaadin.ui.dnd.DropTargetExtension;
  */
 @Deprecated
 @AcceptCriterion(ContainsDataFlavor.class)
-final public class VContainsDataFlavor extends VAcceptCriterion {
+public final class VContainsDataFlavor extends VAcceptCriterion {
 
     @Override
     protected boolean accept(VDragEvent drag, UIDL configuration) {
index 5c1c9b75a02f0b51ee0d7621bf9ab41c88cec949..e5b4790cdac9fc7760061b81a263080191935dd8 100644 (file)
@@ -31,7 +31,7 @@ import com.vaadin.ui.dnd.DropTargetExtension;
  */
 @Deprecated
 @AcceptCriterion(SourceIs.class)
-final public class VDragSourceIs extends VAcceptCriterion {
+public final class VDragSourceIs extends VAcceptCriterion {
 
     @Override
     protected boolean accept(VDragEvent drag, UIDL configuration) {
index 5dcf0ce630a3742fcf6fefa1f2dd9b42963142bf..0698b958613a16a654e09fa2685d0187c77809b2 100644 (file)
@@ -31,12 +31,12 @@ public class VHtml5File extends JavaScriptObject {
     protected VHtml5File() {
     }
 
-    public native final String getName()
+    public final native String getName()
     /*-{
         return this.name;
      }-*/;
 
-    public native final String getType()
+    public final native String getType()
     /*-{
         return this.type;
      }-*/;
@@ -47,7 +47,7 @@ public class VHtml5File extends JavaScriptObject {
      * and then cast back to a long value.
      * www.gwtproject.org/doc/latest/DevGuideCodingBasicsJSNI.html#important
      */
-    public native final double getSize()
+    public final native double getSize()
     /*-{
         return this.size ? this.size : 0;
     }-*/;
index e310fd479d677f663d2fc62fd788057609b13d8a..e9e646bf96077f54f65e3a7f52dd3ac77bd84566 100644 (file)
@@ -25,7 +25,7 @@ import com.vaadin.ui.dnd.DropTargetExtension;
  *             and extending classes
  */
 @Deprecated
-final public class VOverTreeNode extends VAcceptCriterion {
+public final class VOverTreeNode extends VAcceptCriterion {
 
     @Override
     protected boolean accept(VDragEvent drag, UIDL configuration) {
index 79df33e9a758dd96daec914f8c129ba685b74411..72d0784c3efbb7add20a70629d737dc49330ce53 100644 (file)
@@ -167,11 +167,10 @@ public class MenuBarConnector extends AbstractComponentConnector
                 itr = iteratorStack.pop();
                 currentMenu = menuStack.pop();
             }
-        } // while
+        }
 
         getLayoutManager().setNeedsHorizontalLayout(this);
-
-    }// updateFromUIDL
+    }
 
     @Override
     public VMenuBar getWidget() {
index b5aaa5d894cbe676e2e1cd9f5c8e62bca9b466e6..41bc1b595218331e9d2a6a2d2920040171aeecdc 100644 (file)
@@ -205,7 +205,7 @@ public abstract class ScrollbarBundle implements DeferredWorker {
      *
      * @see VerticalScrollbarBundle#getElement()
      */
-    public final static class VerticalScrollbarBundle extends ScrollbarBundle {
+    public static final class VerticalScrollbarBundle extends ScrollbarBundle {
 
         @Override
         public void setStylePrimaryName(String primaryStyleName) {
@@ -275,7 +275,7 @@ public abstract class ScrollbarBundle implements DeferredWorker {
      *
      * @see HorizontalScrollbarBundle#getElement()
      */
-    public final static class HorizontalScrollbarBundle
+    public static final class HorizontalScrollbarBundle
             extends ScrollbarBundle {
 
         @Override
index 846b0ce7230f8ed3b4829f8fb22d23c6fe84e7c4..5ede9bb90e3cf3483dc4fe245458a5dff052ea19 100644 (file)
@@ -29,7 +29,7 @@ public class RowHeightChangedEvent extends GwtEvent<RowHeightChangedHandler> {
     /**
      * Handler type.
      */
-    public final static Type<RowHeightChangedHandler> TYPE = new Type<>();
+    public static final Type<RowHeightChangedHandler> TYPE = new Type<>();
 
     public static final Type<RowHeightChangedHandler> getType() {
         return TYPE;
index 0063c20b2859976464dfe038404839bc09efb2b9..56c1207439ad7f1d31507b508f956a3af21ca091 100644 (file)
@@ -34,7 +34,7 @@ public class ColumnReorderEvent<T> extends GwtEvent<ColumnReorderHandler<T>> {
     /**
      * Handler type.
      */
-    private final static Type<ColumnReorderHandler<?>> TYPE = new Type<>();
+    private static final Type<ColumnReorderHandler<?>> TYPE = new Type<>();
 
     public static final Type<ColumnReorderHandler<?>> getType() {
         return TYPE;
index abd369b175c1fd9f4a13e939afa2b07bd88504ed..3c049ea709b3b629161dd76aae3d9e850491cad0 100644 (file)
@@ -32,7 +32,7 @@ public class ColumnResizeEvent<T> extends GwtEvent<ColumnResizeHandler<T>> {
     /**
      * Handler type.
      */
-    private final static Type<ColumnResizeHandler<?>> TYPE = new Type<>();
+    private static final Type<ColumnResizeHandler<?>> TYPE = new Type<>();
 
     private Column<?, T> column;
 
index 5c7b88cc9195f1928a604faedd154065d183dc7f..7e01e91e8d2665dc59cb7d5984755aa64bcbf9bc 100644 (file)
@@ -31,7 +31,7 @@ import com.vaadin.client.widgets.Grid.Column;
 public class ColumnVisibilityChangeEvent<T>
         extends GwtEvent<ColumnVisibilityChangeHandler<T>> {
 
-    private final static Type<ColumnVisibilityChangeHandler<?>> TYPE = new Type<>();
+    private static final Type<ColumnVisibilityChangeHandler<?>> TYPE = new Type<>();
 
     public static final Type<ColumnVisibilityChangeHandler<?>> getType() {
         return TYPE;
index 178f3a422212ebf0a8a481a78ee248ab68bb75bc..092253bbc55e20fd964fb102a0968c393f8a3166 100644 (file)
@@ -324,7 +324,7 @@ public class Escalator extends Widget
              * {@link com.google.gwt.dom.client.NativeEvent NativeEvent} isn't
              * properly populated with the correct values.
              */
-            private final static class CustomTouchEvent
+            private static final class CustomTouchEvent
                     extends JavaScriptObject {
                 protected CustomTouchEvent() {
                 }
@@ -1496,7 +1496,7 @@ public class Escalator extends Widget
             return elem;
         }
 
-        abstract protected void recalculateSectionHeight();
+        protected abstract void recalculateSectionHeight();
 
         /**
          * Returns the height of all rows in the row container.
@@ -1746,7 +1746,7 @@ public class Escalator extends Widget
          * @return <code>true</code> iff this the given element, or any of its
          *         descendants, can be frozen
          */
-        abstract protected boolean rowCanBeFrozen(TableRowElement tr);
+        protected abstract boolean rowCanBeFrozen(TableRowElement tr);
 
         /**
          * Iterates through all the cells in a column and returns the width of
index 578f260a1e71bd5157c7ee11f23bd40be7fb2830..537d8648368dc738e649d86bb7024d117de8b841 100755 (executable)
@@ -2292,7 +2292,7 @@ public class Grid<T> extends ResizeComposite implements HasSelectionHandlers<T>,
         }
     }
 
-    public static abstract class AbstractGridKeyEvent<HANDLER extends AbstractGridKeyEventHandler>
+    public abstract static class AbstractGridKeyEvent<HANDLER extends AbstractGridKeyEventHandler>
             extends KeyEvent<HANDLER> {
 
         /**
@@ -2358,7 +2358,7 @@ public class Grid<T> extends ResizeComposite implements HasSelectionHandlers<T>,
         protected abstract void doDispatch(HANDLER handler, Section section);
     }
 
-    public static abstract class AbstractGridMouseEvent<HANDLER extends AbstractGridMouseEventHandler>
+    public abstract static class AbstractGridMouseEvent<HANDLER extends AbstractGridMouseEventHandler>
             extends MouseEvent<HANDLER> {
 
         /**
@@ -8972,17 +8972,17 @@ public class Grid<T> extends ResizeComposite implements HasSelectionHandlers<T>,
      * @param parent
      *            The parent to set
      */
-    private static native final void setParent(Widget widget, Grid<?> parent)
+    private static final native void setParent(Widget widget, Grid<?> parent)
     /*-{
         widget.@com.google.gwt.user.client.ui.Widget::setParent(Lcom/google/gwt/user/client/ui/Widget;)(parent);
     }-*/;
 
-    private static native final void onAttach(Widget widget)
+    private static final native void onAttach(Widget widget)
     /*-{
         widget.@Widget::onAttach()();
     }-*/;
 
-    private static native final void onDetach(Widget widget)
+    private static final native void onDetach(Widget widget)
     /*-{
         widget.@Widget::onDetach()();
     }-*/;
index e6ce34b8a7e78a7ca978bb8fcdb8939a1a5683c4..86758e317391fd652ca333e877022467cb0159c8 100644 (file)
@@ -352,7 +352,7 @@ public class Overlay extends PopupPanel {
         return topFix;
     }
 
-    private native static int detectRelativeBodyFixes(String axis)
+    private static native int detectRelativeBodyFixes(String axis)
     /*-{
         try {
             var b = $wnd.document.body;
index e76c3e49342f4a9e78939345b3c83cc8865ba6fd..204ec064b69ee4a982cfd55b59f5d1bfa8702cd6 100644 (file)
@@ -160,7 +160,7 @@ public abstract class ClickableRenderer<T, W extends Widget>
             return cellReference;
         }
 
-        private native static Escalator getEscalator(Grid<?> grid)
+        private static native Escalator getEscalator(Grid<?> grid)
         /*-{
           return grid.@com.vaadin.v7.client.widgets.Grid::escalator;
         }-*/;
index 77e5cbeab198bf1787f6abffa73d6ad70db476cd..864b48595dce8b9b4e50e310878fc0156b4bc943 100644 (file)
@@ -247,7 +247,7 @@ public class VCalendar extends Composite implements VHasDropHandler {
         void contextMenu(ContextMenuEvent event, Widget widget);
     }
 
-    private static abstract class AbstractEventComparator
+    private abstract static class AbstractEventComparator
             implements Comparator<CalendarEvent> {
 
         @Override
index 8780c980ab420980a39967a0d3498293209faa0b..afc0a7340e342eb662577f0bfa99c83fa8d432d0 100644 (file)
@@ -1183,38 +1183,27 @@ public class VCalendarPanel extends FocusableFlexTable implements
         // Ctrl and Shift selection not supported
         if (ctrl || shift) {
             return false;
-        }
-
-        else if (keycode == getPreviousKey()) {
+        } else if (keycode == getPreviousKey()) {
             focusNextYear(10); // Add 10 years
             return true;
-        }
-
-        else if (keycode == getForwardKey()) {
+        } else if (keycode == getForwardKey()) {
             focusNextYear(1); // Add 1 year
             return true;
-        }
-
-        else if (keycode == getNextKey()) {
+        } else if (keycode == getNextKey()) {
             focusPreviousYear(10); // Subtract 10 years
             return true;
-        }
-
-        else if (keycode == getBackwardKey()) {
+        } else if (keycode == getBackwardKey()) {
             focusPreviousYear(1); // Subtract 1 year
             return true;
-
         } else if (keycode == getSelectKey()) {
             value = (Date) focusedDate.clone();
             onSubmit();
             return true;
-
         } else if (keycode == getResetKey()) {
             // Restore showing value the selected value
             focusedDate.setTime(value.getTime());
             renderCalendar();
             return true;
-
         } else if (keycode == getCloseKey()) {
             // TODO fire listener, on users responsibility??
 
@@ -1402,24 +1391,15 @@ public class VCalendarPanel extends FocusableFlexTable implements
             boolean shift) {
         if (!isEnabled() || isReadonly()) {
             return false;
-        }
-
-        else if (resolution == Resolution.YEAR) {
+        } else if (resolution == Resolution.YEAR) {
             return handleNavigationYearMode(keycode, ctrl, shift);
-        }
-
-        else if (resolution == Resolution.MONTH) {
+        } else if (resolution == Resolution.MONTH) {
             return handleNavigationMonthMode(keycode, ctrl, shift);
-        }
-
-        else if (resolution == Resolution.DAY) {
+        } else if (resolution == Resolution.DAY) {
             return handleNavigationDayMode(keycode, ctrl, shift);
-        }
-
-        else {
+        } else {
             return handleNavigationDayMode(keycode, ctrl, shift);
         }
-
     }
 
     /**
index 6a3e389c0afa600bdcf5d8f449698cea23f29b87..f2fec9362704bdd55291658497dfa03c718ee023 100644 (file)
@@ -25,7 +25,7 @@ import com.vaadin.shared.ui.dd.AcceptCriterion;
 import com.vaadin.v7.ui.AbstractSelect;
 
 @AcceptCriterion(AbstractSelect.TargetItemIs.class)
-final public class VIsOverId extends VAcceptCriterion {
+public final class VIsOverId extends VAcceptCriterion {
 
     @Override
     protected boolean accept(VDragEvent drag, UIDL configuration) {
index 1536f720cd47d4a1396251f29c09a3762140c853..dfb643fc1434579a66ac20046a57cad4620b5f1b 100644 (file)
@@ -23,7 +23,7 @@ import com.vaadin.shared.ui.dd.AcceptCriterion;
 import com.vaadin.v7.ui.AbstractSelect;
 
 @AcceptCriterion(AbstractSelect.AcceptItem.class)
-final public class VItemIdIs extends VAcceptCriterion {
+public final class VItemIdIs extends VAcceptCriterion {
 
     @Override
     protected boolean accept(VDragEvent drag, UIDL configuration) {
index 75c93858776c863614450a7f85e67a65101251f0..24514714a9fdf822eb6cc5e7a2378d4ee5cd4da2 100644 (file)
@@ -26,7 +26,7 @@ import com.vaadin.v7.client.ui.VTree.TreeNode;
 import com.vaadin.v7.ui.Tree;
 
 @AcceptCriterion(Tree.TargetInSubtree.class)
-final public class VTargetInSubtree extends VAcceptCriterion {
+public final class VTargetInSubtree extends VAcceptCriterion {
 
     @Override
     protected boolean accept(VDragEvent drag, UIDL configuration) {
index 8e6d9a524358d157ed2bd6072f7c2a0bc0cc8164..3bfae760057a5550d3cf85d6da9a2fb9d1bec9fe 100644 (file)
@@ -205,7 +205,7 @@ public abstract class ScrollbarBundle implements DeferredWorker {
      *
      * @see VerticalScrollbarBundle#getElement()
      */
-    public final static class VerticalScrollbarBundle extends ScrollbarBundle {
+    public static final class VerticalScrollbarBundle extends ScrollbarBundle {
 
         @Override
         public void setStylePrimaryName(String primaryStyleName) {
@@ -275,7 +275,7 @@ public abstract class ScrollbarBundle implements DeferredWorker {
      *
      * @see HorizontalScrollbarBundle#getElement()
      */
-    public final static class HorizontalScrollbarBundle
+    public static final class HorizontalScrollbarBundle
             extends ScrollbarBundle {
 
         @Override
index c883a26716f0da5593081d8b9cb893be743ef930..99b26bb9de24ef4ce5b802b91ce28bda02063882 100644 (file)
@@ -29,7 +29,7 @@ public class RowHeightChangedEvent extends GwtEvent<RowHeightChangedHandler> {
     /**
      * Handler type.
      */
-    public final static Type<RowHeightChangedHandler> TYPE = new Type<RowHeightChangedHandler>();
+    public static final Type<RowHeightChangedHandler> TYPE = new Type<RowHeightChangedHandler>();
 
     public static final Type<RowHeightChangedHandler> getType() {
         return TYPE;
index 399ed128a23cad0a7afe7169efb978f9e0750662..4c8dbc16af378fb07c388e19cd640e23a75e8ac5 100644 (file)
@@ -31,7 +31,7 @@ public class ColumnReorderEvent<T> extends GwtEvent<ColumnReorderHandler<T>> {
     /**
      * Handler type.
      */
-    private final static Type<ColumnReorderHandler<?>> TYPE = new Type<ColumnReorderHandler<?>>();
+    private static final Type<ColumnReorderHandler<?>> TYPE = new Type<ColumnReorderHandler<?>>();
 
     public static final Type<ColumnReorderHandler<?>> getType() {
         return TYPE;
index ba5089341168bb47dea6502d82ba86c1c9b48b78..04b067cf6342499bc2390975aad04602039c52b2 100644 (file)
@@ -32,7 +32,7 @@ public class ColumnResizeEvent<T> extends GwtEvent<ColumnResizeHandler<T>> {
     /**
      * Handler type.
      */
-    private final static Type<ColumnResizeHandler<?>> TYPE = new Type<ColumnResizeHandler<?>>();
+    private static final Type<ColumnResizeHandler<?>> TYPE = new Type<ColumnResizeHandler<?>>();
 
     private Column<?, T> column;
 
index e58b678e3db8650cd103e2075108a989b02c2b64..8937308dd876b56eba09d7ed7ec6de61ab00b060 100644 (file)
@@ -31,7 +31,7 @@ import com.vaadin.v7.client.widgets.Grid.Column;
 public class ColumnVisibilityChangeEvent<T>
         extends GwtEvent<ColumnVisibilityChangeHandler<T>> {
 
-    private final static Type<ColumnVisibilityChangeHandler<?>> TYPE = new Type<ColumnVisibilityChangeHandler<?>>();
+    private static final Type<ColumnVisibilityChangeHandler<?>> TYPE = new Type<ColumnVisibilityChangeHandler<?>>();
 
     public static final Type<ColumnVisibilityChangeHandler<?>> getType() {
         return TYPE;
index c46a19229244549135a87e49aacaf969426d0744..ae9ca4c3dac5147edf140c3cc1bc79d6a7ac2b0b 100644 (file)
@@ -30,7 +30,7 @@ public class SelectAllEvent<T> extends GwtEvent<SelectAllHandler<T>> {
     /**
      * Handler type.
      */
-    private final static Type<SelectAllHandler<?>> TYPE = new Type<SelectAllHandler<?>>();;
+    private static final Type<SelectAllHandler<?>> TYPE = new Type<SelectAllHandler<?>>();;
 
     private SelectionModel.Multi<T> selectionModel;
 
index 4eea41c0ffed5a834513c292422dcce28695be9f..8b71ccc52af4b698978fb67dc8efdf03f63b987e 100644 (file)
@@ -323,7 +323,7 @@ public class Escalator extends Widget
              * {@link com.google.gwt.dom.client.NativeEvent NativeEvent} isn't
              * properly populated with the correct values.
              */
-            private final static class CustomTouchEvent
+            private static final class CustomTouchEvent
                     extends JavaScriptObject {
                 protected CustomTouchEvent() {
                 }
@@ -1481,7 +1481,7 @@ public class Escalator extends Widget
             return elem;
         }
 
-        abstract protected void recalculateSectionHeight();
+        protected abstract void recalculateSectionHeight();
 
         /**
          * Returns the height of all rows in the row container.
@@ -1731,7 +1731,7 @@ public class Escalator extends Widget
          * @return <code>true</code> iff this the given element, or any of its
          *         descendants, can be frozen
          */
-        abstract protected boolean rowCanBeFrozen(TableRowElement tr);
+        protected abstract boolean rowCanBeFrozen(TableRowElement tr);
 
         /**
          * Iterates through all the cells in a column and returns the width of
index a59e61d9dc344bd25f84a4433c8b0d04584e8d56..eb88bb895f6026c9a425c281a9087372e7feb202 100644 (file)
@@ -2295,7 +2295,7 @@ public class Grid<T> extends ResizeComposite implements HasSelectionHandlers<T>,
         }
     }
 
-    public static abstract class AbstractGridKeyEvent<HANDLER extends AbstractGridKeyEventHandler>
+    public abstract static class AbstractGridKeyEvent<HANDLER extends AbstractGridKeyEventHandler>
             extends KeyEvent<HANDLER> {
 
         /**
@@ -2360,7 +2360,7 @@ public class Grid<T> extends ResizeComposite implements HasSelectionHandlers<T>,
         protected abstract void doDispatch(HANDLER handler, Section section);
     }
 
-    public static abstract class AbstractGridMouseEvent<HANDLER extends AbstractGridMouseEventHandler>
+    public abstract static class AbstractGridMouseEvent<HANDLER extends AbstractGridMouseEventHandler>
             extends MouseEvent<HANDLER> {
 
         /**
@@ -4702,7 +4702,7 @@ public class Grid<T> extends ResizeComposite implements HasSelectionHandlers<T>,
      * @param <T>
      *            the row type
      */
-    public static abstract class Column<C, T> {
+    public abstract static class Column<C, T> {
 
         /**
          * Default renderer for GridColumns. Renders everything into text
@@ -8955,17 +8955,17 @@ public class Grid<T> extends ResizeComposite implements HasSelectionHandlers<T>,
      * @param parent
      *            The parent to set
      */
-    private static native final void setParent(Widget widget, Grid<?> parent)
+    private static final native void setParent(Widget widget, Grid<?> parent)
     /*-{
         widget.@com.google.gwt.user.client.ui.Widget::setParent(Lcom/google/gwt/user/client/ui/Widget;)(parent);
     }-*/;
 
-    private static native final void onAttach(Widget widget)
+    private static final native void onAttach(Widget widget)
     /*-{
         widget.@Widget::onAttach()();
     }-*/;
 
-    private static native final void onDetach(Widget widget)
+    private static final native void onDetach(Widget widget)
     /*-{
         widget.@Widget::onDetach()();
     }-*/;
index 100a8df49b681c17ea31f735f7778181854da8f5..61d9d4f55fed855ef2e096e21098bd0c82029f4a 100644 (file)
@@ -151,7 +151,7 @@ public abstract class AbstractInMemoryContainer<ITEMIDTYPE, PROPERTYIDCLASS, ITE
         }
     }
 
-    private static abstract class BaseItemAddOrRemoveEvent extends EventObject
+    private abstract static class BaseItemAddOrRemoveEvent extends EventObject
             implements Serializable {
         protected Object itemId;
         protected int index;
index 9c71af7c5c71df02ee086f59fac77c1544e47f94..dd1f28305d9c3b4e9bca30f93f3e3d6f092dd73a 100644 (file)
@@ -75,13 +75,13 @@ public class FilesystemContainer implements Container.Hierarchical {
      */
     public static Collection<String> FILE_PROPERTIES;
 
-    private final static Method FILEITEM_LASTMODIFIED;
+    private static final Method FILEITEM_LASTMODIFIED;
 
-    private final static Method FILEITEM_NAME;
+    private static final Method FILEITEM_NAME;
 
-    private final static Method FILEITEM_ICON;
+    private static final Method FILEITEM_ICON;
 
-    private final static Method FILEITEM_SIZE;
+    private static final Method FILEITEM_SIZE;
 
     static {
 
index 7b26eccfc69d0d9f074a1963975fdf9b28901f2a..2e1ac13e9a4155f741594b5f45fd609f1a75bb49 100644 (file)
@@ -182,7 +182,7 @@ public abstract class PropertyFormatter<T> extends AbstractProperty<String>
      *            datasource.
      * @return
      */
-    abstract public String format(T value);
+    public abstract String format(T value);
 
     /**
      * Parse string and convert it to format compatible with datasource.
@@ -196,7 +196,7 @@ public abstract class PropertyFormatter<T> extends AbstractProperty<String>
      *             Any type of exception can be thrown to indicate that the
      *             conversion was not succesful.
      */
-    abstract public T parse(String formattedValue) throws Exception;
+    public abstract T parse(String formattedValue) throws Exception;
 
     /**
      * Sets the Property's read-only mode to the specified status.
index b80835706698f70aa36c2b9c31d0d69c13db239a..3dea636644a8aac988ba64d4e965f2be42ef20ad 100644 (file)
@@ -99,7 +99,7 @@ public class TextFileProperty extends AbstractProperty<String> {
                     : new InputStreamReader(fis, charset);
             BufferedReader r = new BufferedReader(isr);
             StringBuilder b = new StringBuilder();
-            char buf[] = new char[8 * 1024];
+            char[] buf = new char[8 * 1024];
             int len;
             while ((len = r.read(buf)) != -1) {
                 b.append(buf, 0, len);
index acfba8abfd8eb823976adb3a43bffe3be17f03c8..53eacc20e2056c10d429632d5b5c3e8aa0a46a6d 100644 (file)
@@ -39,7 +39,7 @@ import com.vaadin.server.VaadinSession;
 @Deprecated
 public class DefaultConverterFactory implements ConverterFactory {
 
-    private final static Logger log = Logger
+    private static final Logger log = Logger
             .getLogger(DefaultConverterFactory.class.getName());
 
     @Override
index 45ef2cd57e23635d7838c05c2beb384ce465a537..b1fc3738ed6a589151e58c19dd0438fe10589b31 100644 (file)
@@ -33,7 +33,7 @@ import com.vaadin.v7.data.util.converter.Converter.ConversionException;
  * @deprecated As of 8.0, no replacement available.
  */
 @Deprecated
-final public class ColumnProperty implements Property {
+public final class ColumnProperty implements Property {
     private static final long serialVersionUID = -3694463129581802457L;
 
     private RowItem owner;
index 5750de6b36ac41fb867efb3995495c022c645630..070b15fc5cd74d0657527ab5715e10e1c0f13edc 100644 (file)
@@ -159,7 +159,7 @@ public class SQLContainer implements Container, Container.Filterable,
 
     @Override
     public Object addItem() throws UnsupportedOperationException {
-        Object emptyKey[] = new Object[queryDelegate.getPrimaryKeyColumns()
+        Object[] emptyKey = new Object[queryDelegate.getPrimaryKeyColumns()
                 .size()];
         RowId itemId = new TemporaryRowId(emptyKey);
         // Create new empty column properties for the row item.
index d1c884c360e21d59db69214982495e850ec54568..b435757e8e4dd0c65b94dd0aceef84d4fd6c0434 100644 (file)
@@ -45,7 +45,7 @@ public class QueryBuilder implements Serializable {
         addFilterTranslator(new SimpleStringTranslator());
     }
 
-    public synchronized static void addFilterTranslator(
+    public static synchronized void addFilterTranslator(
             FilterTranslator translator) {
         filterTranslators.add(translator);
     }
@@ -79,7 +79,7 @@ public class QueryBuilder implements Serializable {
      *            the statement helper to update with the value(s) of the filter
      * @return a string representing the filter.
      */
-    public synchronized static String getWhereStringForFilter(Filter filter,
+    public static synchronized String getWhereStringForFilter(Filter filter,
             StatementHelper sh) {
         for (FilterTranslator ft : filterTranslators) {
             if (ft.translatesFilter(filter)) {
index dac491bc2792a2eb61f68f1137a744616a69d8b2..5c5b4d21ed0b8b75f84af8cbf78185bcc7f68115 100644 (file)
@@ -124,7 +124,7 @@ public interface FieldEvents {
      * @since 6.5
      */
     @Deprecated
-    public static abstract class TextChangeEvent extends Component.Event {
+    public abstract static class TextChangeEvent extends Component.Event {
 
         public TextChangeEvent(Component source) {
             super(source);
index 2b6e9b43bffdc568c2d12138be4c05a98f97e4c4..baa8f1b34ee43a3e8545e297d58f23f48989a27f 100644 (file)
@@ -437,7 +437,7 @@ public abstract class AbstractColorPicker extends AbstractLegacyComponent
      * Sets the default styles of the component
      *
      */
-    abstract protected void setDefaultStyles();
+    protected abstract void setDefaultStyles();
 
     /**
      * Shows a popup-window for color selection.
index e89c72fa4ce25539505aa328221bfaa75cf7fbed..70fabdc9b356a3b74a762f6d8b96870d749e2635 100644 (file)
@@ -2046,7 +2046,7 @@ public abstract class AbstractSelect extends AbstractField<Object> implements
      * @since 6.3
      *
      */
-    private static abstract class AbstractItemSetCriterion
+    private abstract static class AbstractItemSetCriterion
             extends ClientSideCriterion {
         protected final Collection<Object> itemIds = new HashSet<Object>();
         protected AbstractSelect select;
index 4315990d3a0eb49cf9f7e75cd82ce6e9fb0b0e1e..2b480437cfc65262a811b1de5112f561073fb1be 100644 (file)
@@ -327,7 +327,7 @@ public class Grid extends AbstractComponent
      * @since 7.6.1
      */
     @Deprecated
-    public final static class DetailComponentManager
+    public static final class DetailComponentManager
             extends AbstractGridExtension implements DataGenerator {
 
         /**
@@ -982,7 +982,7 @@ public class Grid extends AbstractComponent
      * Base class for editor related events
      */
     @Deprecated
-    public static abstract class EditorEvent extends Component.Event {
+    public abstract static class EditorEvent extends Component.Event {
 
         private Object itemID;
 
@@ -1437,7 +1437,7 @@ public class Grid extends AbstractComponent
      * reusable.
      */
     @Deprecated
-    public static abstract class AbstractSelectionModel extends
+    public abstract static class AbstractSelectionModel extends
             AbstractGridExtension implements SelectionModel, DataGenerator {
         protected final LinkedHashSet<Object> selection = new LinkedHashSet<Object>();
 
@@ -2646,7 +2646,7 @@ public class Grid extends AbstractComponent
                 }
             }
 
-            abstract protected String getCellTagName();
+            protected abstract String getCellTagName();
 
             void detach() {
                 for (CELLTYPE cell : cells.values()) {
@@ -4314,7 +4314,7 @@ public class Grid extends AbstractComponent
      *            the type this renderer knows how to present
      */
     @Deprecated
-    public static abstract class AbstractRenderer<T>
+    public abstract static class AbstractRenderer<T>
             extends AbstractGridExtension implements Renderer<T> {
 
         private final Class<T> presentationType;
@@ -4473,7 +4473,7 @@ public class Grid extends AbstractComponent
      * @since 7.5
      */
     @Deprecated
-    public static abstract class AbstractGridExtension
+    public abstract static class AbstractGridExtension
             extends AbstractExtension {
 
         /**
index 8fd401e393fbca5d9f307474377b2bc334173b80..65613d00cc181c7989e616e1c3ecfad330429a8d 100644 (file)
@@ -5232,7 +5232,7 @@ public class Table extends AbstractSelect implements Action.Container,
      * during that drag and drop operation.
      */
     @Deprecated
-    public static abstract class TableDropCriterion
+    public abstract static class TableDropCriterion
             extends ServerSideCriterion {
 
         private Table table;
index a5e272b604b254a7295c13a93ea6af62adddcb10..a6e13f73c722991777e596c216fe6a4608e3661f 100644 (file)
@@ -1636,7 +1636,7 @@ public class Tree extends AbstractSelect implements Container.Hierarchical,
      * during that drag and drop operation.
      */
     @Deprecated
-    public static abstract class TreeDropCriterion extends ServerSideCriterion {
+    public abstract static class TreeDropCriterion extends ServerSideCriterion {
 
         private Tree tree;
 
index 2bc90c175c8f10a8ceeaaf917f9ff141e98fad1f..f7506511573acef474acb8c120211befbb34fa64 100644 (file)
@@ -6,9 +6,9 @@ import java.util.concurrent.atomic.AtomicLong;
 public class BeanItemContainerGenerator {
 
     public static class PortableRandom {
-        private final static long multiplier = 0x5DEECE66DL;
-        private final static long addend = 0xBL;
-        private final static long mask = (1L << 48) - 1;
+        private static final long multiplier = 0x5DEECE66DL;
+        private static final long addend = 0xBL;
+        private static final long mask = (1L << 48) - 1;
         private AtomicLong seed;
 
         public PortableRandom(long seed) {
index 516e4eb63d67a8f1c4e5dad3ce0a1193c72655eb..a22b704b8496b0192312dbb5507130813c918cb3 100644 (file)
@@ -12,7 +12,7 @@ import org.junit.Test;
 public class PerformanceTestIndexedContainerTest {
 
     private static final int REPEATS = 10;
-    private final static int ITEMS = 50000;
+    private static final int ITEMS = 50000;
     private static final long ADD_ITEM_FAIL_THRESHOLD = 200;
     // TODO should improve performance of these methods
     private static final long ADD_ITEM_AT_FAIL_THRESHOLD = 5000;
index df833b0cbacdc89cc7eafa13216abd9e5f1c0631..b84388ad9cd041546507e9bea79cb48df2652ef0 100644 (file)
@@ -41,11 +41,11 @@ import com.vaadin.v7.ui.Table.TableContextClickEvent;
  */
 public class ContextClickListenerTest extends AbstractComponent {
 
-    private final static ContextClickEvent contextClickEvent = EasyMock
+    private static final ContextClickEvent contextClickEvent = EasyMock
             .createMock(ContextClickEvent.class);
-    private final static GridContextClickEvent gridContextClickEvent = EasyMock
+    private static final GridContextClickEvent gridContextClickEvent = EasyMock
             .createMock(GridContextClickEvent.class);
-    private final static TableContextClickEvent tableContextClickEvent = EasyMock
+    private static final TableContextClickEvent tableContextClickEvent = EasyMock
             .createMock(TableContextClickEvent.class);
 
     private final AssertListener contextListener = new AssertListener();
index 05fb77614075dcbb6a484a55e80820f94feebe88..aba50226f70b411f4be9f5dbe189c47d805def42 100644 (file)
@@ -595,7 +595,7 @@ public class DataCommunicator<T> extends AbstractExtension {
      * @since 8.1
      *
      */
-    protected DataKeyMapper<T> createKeyMapper(ValueProvider<T,Object> identifierGetter) {
+    protected DataKeyMapper<T> createKeyMapper(ValueProvider<T, Object> identifierGetter) {
         return new KeyMapper<T>(identifierGetter);
     }
 
index 2b0a0f62861b39a97f8678ec42f1410e541e2c14..c12d683f20c4cc58b591b7bf78edace3aafb9fba 100644 (file)
@@ -165,7 +165,7 @@ public interface FieldEvents {
         public void blur(BlurEvent event);
     }
 
-    public static abstract class FocusAndBlurServerRpcImpl
+    public abstract static class FocusAndBlurServerRpcImpl
             implements FocusAndBlurServerRpc {
 
         private final Component component;
index 66e0110a4cb2fb4baefce22fb1c9e4b3e06980bf..c8b70b649ea92f01914955848e670b98845fdaf2 100644 (file)
@@ -41,5 +41,5 @@ public abstract class ShortcutListener extends ShortcutAction
     }
 
     @Override
-    abstract public void handleAction(Object sender, Object target);
+    public abstract void handleAction(Object sender, Object target);
 }
index b0dab24fe063a57b0f9acb607292187ac48d1324..b0e13963bd70d2d7a1d5c47f6a8d67792e47985a 100644 (file)
@@ -30,7 +30,7 @@ import com.vaadin.server.PaintTarget;
  */
 public class Or extends ClientSideCriterion {
     private static final long serialVersionUID = 1L;
-    private final AcceptCriterion criteria[];
+    private final AcceptCriterion[] criteria;
 
     /**
      * @param criteria
index 9e1134efed0487fb8bdc08f9e51cabea480e14d0..f65fc610c93465df3635ea4e7e2de4e021695e44 100644 (file)
@@ -46,7 +46,7 @@ import com.vaadin.ui.Window;
 @SuppressWarnings({ "serial", "deprecation" })
 public class ComponentSizeValidator implements Serializable {
 
-    private final static int LAYERS_SHOWN = 4;
+    private static final int LAYERS_SHOWN = 4;
 
     /**
      * Recursively checks given component and its subtree for invalid layout
index a2df4203c871b079e9065260cca28a12c6d55e30..3be04fc2c4710ada3ec0f1d34bb45789e9c6b160 100644 (file)
@@ -49,7 +49,7 @@ public class JsonPaintTarget implements PaintTarget {
 
     /* Document type declarations */
 
-    private final static String UIDL_ARG_NAME = "name";
+    private static final String UIDL_ARG_NAME = "name";
 
     private final Deque<String> mOpenTags;
 
@@ -206,7 +206,7 @@ public class JsonPaintTarget implements PaintTarget {
      * @return A new string instance where all occurrences of XML sensitive
      *         characters are substituted with entities.
      */
-    static public String escapeXML(String xml) {
+    public static String escapeXML(String xml) {
         if (xml == null || xml.length() <= 0) {
             return "";
         }
@@ -248,7 +248,7 @@ public class JsonPaintTarget implements PaintTarget {
      *            The string to escape
      * @return Escaped version of the string
      */
-    static public String escapeJSON(String s) {
+    public static String escapeJSON(String s) {
         // FIXME: Move this method to another class as other classes use it
         // also.
         if (s == null) {
index 441580df84a4536ab98978990ede8712613bb5f5..d4a7d0e62d6f7bc2836d94fe39a49912e1d563df 100644 (file)
@@ -63,7 +63,7 @@ public class VaadinPortlet extends GenericPortlet
      * Base class for portlet requests that need access to HTTP servlet
      * requests.
      */
-    public static abstract class VaadinHttpAndPortletRequest
+    public abstract static class VaadinHttpAndPortletRequest
             extends VaadinPortletRequest {
 
         /**
index 638eeb047fe5a76cde99fde90c1a73d6ee3c0561..edfbd1cc28b23f8d24547416258bc8765b1ed8bd 100644 (file)
@@ -1019,7 +1019,7 @@ public class VaadinServlet extends HttpServlet implements Constants {
     private void streamContent(HttpServletResponse response, InputStream is)
             throws IOException {
         final OutputStream os = response.getOutputStream();
-        final byte buffer[] = new byte[DEFAULT_BUFFER_SIZE];
+        final byte[] buffer = new byte[DEFAULT_BUFFER_SIZE];
         int bytes;
         while ((bytes = is.read(buffer)) >= 0) {
             os.write(buffer, 0, bytes);
index 08bcffa442229d504c1c8cc16168a5fb77ae7f04..fb02819c23c95a741fdf864b860d993e075bbc23 100644 (file)
@@ -308,7 +308,7 @@ public class FileUploadHandler implements RequestHandler {
                         "The multipart stream ended unexpectedly");
             }
             bout.write(readByte);
-            if(bout.size() > MULTIPART_BOUNDARY_LINE_LIMIT) {
+            if (bout.size() > MULTIPART_BOUNDARY_LINE_LIMIT) {
                 throw new IOException(
                         "The multipart stream does not contain boundary");
             }
@@ -564,7 +564,7 @@ public class FileUploadHandler implements RequestHandler {
                 throw new NoInputStreamException();
             }
 
-            final byte buffer[] = new byte[MAX_UPLOAD_BUFFER_SIZE];
+            final byte[] buffer = new byte[MAX_UPLOAD_BUFFER_SIZE];
             long lastStreamingEvent = 0;
             int bytesReadToBuffer = 0;
             do {
index 6eeaa8852017a6373c657acd36e7addca7af3a70..f287b4334f065551d678326c71538114cbe10c92 100644 (file)
@@ -308,7 +308,7 @@ public class PushHandler {
         // We don't want to use callWithUi here, as it assumes there's a client
         // request active and does requestStart and requestEnd among other
         // things.
-        if(event == null){
+        if (event == null) {
             getLogger().log(Level.SEVERE,
                     "Could not get event. This should never happen.");
             return;
@@ -316,7 +316,7 @@ public class PushHandler {
 
         AtmosphereResource resource = event.getResource();
 
-        if(resource == null){
+        if (resource == null) {
             getLogger().log(Level.SEVERE,
                     "Could not get resource. This should never happen.");
             return;
index 50b23f80a3b38432809021d017488127668b7139..f93803de9991772087bb49ae8f77766826f58a7a 100644 (file)
@@ -58,7 +58,7 @@ public class ClassPathExplorer {
     /**
      * File filter that only accepts directories.
      */
-    private final static FileFilter DIRECTORIES_ONLY = (File f) ->
+    private static final FileFilter DIRECTORIES_ONLY = (File f) ->
             f.exists() && f.isDirectory();
 
     /**
@@ -301,7 +301,7 @@ public class ClassPathExplorer {
      *
      * @return filtered list of class path entries
      */
-    private final static List<String> getRawClasspathEntries() {
+    private static final List<String> getRawClasspathEntries() {
         // try to keep the order of the classpath
         List<String> locations = new ArrayList<>();
 
@@ -339,7 +339,7 @@ public class ClassPathExplorer {
      *            string
      * @return map of classpath locations, see {@link #classpathLocations}
      */
-    private final static Map<String, URL> getClasspathLocations(
+    private static final Map<String, URL> getClasspathLocations(
             List<String> rawClasspathEntries) {
         long start = System.currentTimeMillis();
         // try to keep the order of the classpath
@@ -428,7 +428,7 @@ public class ClassPathExplorer {
      * @param file
      * @param locations
      */
-    private final static void include(String name, File file,
+    private static final void include(String name, File file,
             Map<String, URL> locations) {
         if (!file.exists()) {
             return;
index 2fdf2bf382518b5e8b95d74c17a6165e3d7b743d..2584eaf6eaa36828cd83187e4780e9176f20757e 100644 (file)
@@ -144,7 +144,7 @@ public abstract class AbstractLocalDateField
         if (value == null) return "";
         DateTimeFormatter dateTimeFormatter = DateTimeFormatter.ofLocalizedDate(FormatStyle.SHORT);
         Locale locale = getLocale();
-        if (locale != null){
+        if (locale != null) {
             dateTimeFormatter = dateTimeFormatter.withLocale(locale);
         }
         return value.format(dateTimeFormatter);
index 755b25b7424283d0159ecc2e88b6251503e79149..fcdbf4b04febd082761ed409cdcbb3d3511922c7 100644 (file)
@@ -183,7 +183,7 @@ public class Composite extends AbstractComponent implements HasComponents {
 
     private Component getRootOrThrow() {
         Component root = getCompositionRoot();
-        if(root == null) throw new IllegalStateException("Composition root has not been set");
+        if (root == null) throw new IllegalStateException("Composition root has not been set");
         return root;
     }
 
@@ -202,7 +202,7 @@ public class Composite extends AbstractComponent implements HasComponents {
 
     private <T> T getRootAbstractComponentPropertyOrNull(SerializableFunction<AbstractComponent, T> getter) {
         Component root = getCompositionRoot();
-        if(root instanceof AbstractComponent) {
+        if (root instanceof AbstractComponent) {
             return getter.apply((AbstractComponent) root);
         }
         return null;
index 39d66bd0b041400feb45e05b8fabc6025c2c12f7..35988bb84ddd6ea2d4e1a86a32041c48adf9604b 100644 (file)
@@ -199,8 +199,8 @@ public class MenuBar extends AbstractComponent
                     tmpItem.getCommand().menuSelected(tmpItem);
                 }
             }
-        } // if
-    }// changeVariables
+        }
+    }
 
     /**
      * Constructs an empty, horizontal menu
@@ -941,7 +941,7 @@ public class MenuBar extends AbstractComponent
             this.checked = checked;
             markAsDirty();
         }
-    }// class MenuItem
+    }
 
     @Override
     public void writeDesign(Element design, DesignContext designContext) {
@@ -1091,4 +1091,4 @@ public class MenuBar extends AbstractComponent
         result.add("html-content-allowed");
         return result;
     }
-}// class MenuBar
+}
index 3858975f065697c4203a73790a1f4e80d689e5d9..e54d8e052a5bb004d2d0bbfe5cd775dc2923b369 100644 (file)
@@ -235,7 +235,7 @@ public class Tree<T> extends Composite
      *
      * @since 8.1
      */
-    public final static class TreeMultiSelectionModel<T>
+    public static final class TreeMultiSelectionModel<T>
             extends MultiSelectionModelImpl<T> {
 
         @Override
index 0fbfe9c6b5ba442b09f89ace41e4e97352b208cf..cdf0c6aaeefced9768d3af8c3fc1c6c718461822 100644 (file)
@@ -58,10 +58,10 @@ public class DesignAttributeHandler implements Serializable {
         return Logger.getLogger(DesignAttributeHandler.class.getName());
     }
 
-    private final static Map<Class<?>, AttributeCacheEntry> cache = new ConcurrentHashMap<>();
+    private static final Map<Class<?>, AttributeCacheEntry> cache = new ConcurrentHashMap<>();
 
     // translates string <-> object
-    private final static DesignFormatter FORMATTER = new DesignFormatter();
+    private static final DesignFormatter FORMATTER = new DesignFormatter();
 
     /**
      * Returns the currently used formatter. All primitive types and all types
index 4cbac69df5d3dcd6a1d4a68b6a53f0bb145a0c19..500e8736e76cf7464a634603fdb590f6c3ed4404 100644 (file)
@@ -42,13 +42,13 @@ public class FileTypeResolver implements Serializable {
     /**
      * Default icon given if no icon is specified for a mime-type.
      */
-    static public Resource DEFAULT_ICON = new ThemeResource(
+    public static Resource DEFAULT_ICON = new ThemeResource(
             "../runo/icons/16/document.png");
 
     /**
      * Default mime-type.
      */
-    static public String DEFAULT_MIME_TYPE = "application/octet-stream";
+    public static String DEFAULT_MIME_TYPE = "application/octet-stream";
 
     /**
      * Initial file extension to mime-type mapping.
index d2d25b477720586c6c124f9e5437b5b591f192d6..085c0ec348e98c13d30baadb8ea9ad90402e7e34 100644 (file)
@@ -271,7 +271,7 @@ public class ClassesSerializableTest {
      *
      * @return List of class path segment strings
      */
-    private final static List<String> getRawClasspathEntries() {
+    private static final List<String> getRawClasspathEntries() {
         // try to keep the order of the classpath
         List<String> locations = new ArrayList<>();
 
@@ -382,7 +382,7 @@ public class ClassesSerializableTest {
      *            File representing the directory to scan
      * @return collection of fully qualified class names in the directory
      */
-    private final static Collection<String> findClassesInDirectory(
+    private static final Collection<String> findClassesInDirectory(
             String parentPackage, File parent) {
         if (parent.isHidden()
                 || parent.getPath().contains(File.separator + ".")) {
index 0a7ae5fe75762956e5774cd76ef2d9d3842e9a21..946a51a9f1daa138391ec2bf9b5bec9fc6ca2e5b 100644 (file)
@@ -112,7 +112,7 @@ public class ClasspathHelper {
         }
     }
 
-    private final static List<String> getRawClasspathEntries() {
+    private static final List<String> getRawClasspathEntries() {
         List<String> locations = new ArrayList<>();
 
         String pathSep = System.getProperty("path.separator");
index 59de2695bc1cb55ab4bba7f68acb89b057fe1613..6835b5fb0b3cf189f80fdc0c438209d5fdd245bf 100644 (file)
@@ -232,7 +232,7 @@ public class ApplicationRunnerServlet extends LegacyVaadinServlet {
         return getApplicationRunnerURIs(request).applicationClassname;
     }
 
-    private final static class ProxyDeploymentConfiguration
+    private static final class ProxyDeploymentConfiguration
             implements InvocationHandler, Serializable {
         private final DeploymentConfiguration originalConfiguration;
 
index 006bcd8fbad5a91b8f4933976b0aa5eced5a7fbf..93ddea656f53ef0a512c449694e3a04703b74704 100644 (file)
@@ -61,7 +61,7 @@ import com.vaadin.launcher.util.BrowserLauncher;
 public class DevelopmentServerLauncher {
 
     private static final String KEYSTORE = "src/main/resources/com/vaadin/launcher/keystore";
-    private final static int serverPort = 8888;
+    private static final int serverPort = 8888;
 
     /**
      * Main function for running Jetty.
index 10959a73b1b27739437872516e170c9b30124dcf..c438c7b40ccd00bcda5a85f31417eede0080d4a4 100644 (file)
@@ -39,7 +39,7 @@ public abstract class AbstractComponentTestCase<T extends AbstractComponent>
 
     private List<T> testComponents = new ArrayList<>();
 
-    abstract protected Class<T> getTestClass();
+    protected abstract Class<T> getTestClass();
 
     protected static ThemeResource uncacheableThemeResource(
             String resourceLocation) {
@@ -51,7 +51,7 @@ public abstract class AbstractComponentTestCase<T extends AbstractComponent>
         return new ThemeResource(resourceLocation);
     }
 
-    abstract protected void initializeComponents();
+    protected abstract void initializeComponents();
 
     @Override
     protected void setup(VaadinRequest request) {
index 1b716e5ebb4a1a8b06ab970e0f064da045573de7..667c0e80ce7337e7c989f4ed4a22d17422b5ebfc 100644 (file)
@@ -26,8 +26,8 @@ import com.vaadin.ui.VerticalLayout;
 
 public class ButtonClick extends AbstractReindeerTestUI {
 
-    public final static String SUCCESS_TEXT = "Click received succesfully!";
-    public final static String WRONG_BUTTON_TEXT = "Wrong button clicked.";
+    public static final String SUCCESS_TEXT = "Click received succesfully!";
+    public static final String WRONG_BUTTON_TEXT = "Wrong button clicked.";
 
     @Override
     protected void setup(VaadinRequest request) {
index 70016c9fb805a3d71a257493b6a036c1f69faddf..bace18dbb5627430f781c8a93d1106336d6a9f8a 100644 (file)
@@ -41,16 +41,16 @@ public class MenuBarsWithNesting extends AbstractReindeerTestUI {
     private final Label label = new Label("Initial content");
 
     // The captions and icons used in the second MenuBar.
-    public final static String[] itemNames = { "Icon item", "Arrow down",
+    public static final String[] itemNames = { "Icon item", "Arrow down",
             "Arrow up", "Warning" };
-    private final static Resource[] itemIcons = {
+    private static final Resource[] itemIcons = {
             new ThemeResource("window/img/restore.png"), VaadinIcons.ARROW_DOWN,
             VaadinIcons.ARROW_UP, VaadinIcons.WARNING };
 
     // The last menu item is nested with the following submenu items.
-    public final static String[] nestedItemnames = { "No icon", "Font icon",
+    public static final String[] nestedItemnames = { "No icon", "Font icon",
             "Image icon" };
-    private final static Resource[] nestedItemIcons = { null, VaadinIcons.LINK,
+    private static final Resource[] nestedItemIcons = { null, VaadinIcons.LINK,
             new ThemeResource("window/img/restore.png") };
 
     private MenuBar.Command selectionCommand;
index db69f8c007984b2aff69ab62a659716624fea944..b5c4197e7d88fbe36e51ce78b4ebbdffe6171de9 100644 (file)
@@ -21,7 +21,7 @@ import com.vaadin.v7.ui.Table;
 
 public class TableNavigationPageDown extends AbstractReindeerTestUI {
 
-    private final static int ROW_NUMBER = 50;
+    private static final int ROW_NUMBER = 50;
 
     @Override
     protected void setup(VaadinRequest req) {
index 5147f996d8eb92bd5072b8d0360f39e2c990db20..63530fac1c54065f7945682ae9c29074d4a5dfd9 100644 (file)
@@ -92,9 +92,9 @@ public class TabKeyboardNavigation extends AbstractReindeerTestUI {
         return 5100;
     }
 
-    public final static String LABEL_ID = "sheetLabel";
+    public static final String LABEL_ID = "sheetLabel";
 
-    public final static String labelID(int index) {
+    public static final String labelID(int index) {
         return LABEL_ID + index;
     }
 
index 73e709d79db788bf05813557e396128a6d6610f6..07640bc7f6e9de9234e37438c3d8c7cadd81e11a 100644 (file)
@@ -279,9 +279,9 @@ public class DDTest2 extends TestBase {
 
     }
 
-    private final static ThemeResource FOLDER = new ThemeResource(
+    private static final ThemeResource FOLDER = new ThemeResource(
             "../runo/icons/16/folder.png");
-    private final static ThemeResource DOC = new ThemeResource(
+    private static final ThemeResource DOC = new ThemeResource(
             "../runo/icons/16/document.png");
 
     private void popuplateTrees() {
index 502b476f3ca8d5fe168c0edcc3d1df5d292d884e..e132712e9edb5b58e726f4491202949ee2a5be52 100644 (file)
@@ -175,9 +175,9 @@ public class DDTest6 extends TestBase {
 
     }
 
-    private final static ThemeResource FOLDER = new ThemeResource(
+    private static final ThemeResource FOLDER = new ThemeResource(
             "../runo/icons/64/folder.png");
-    private final static ThemeResource DOC = new ThemeResource(
+    private static final ThemeResource DOC = new ThemeResource(
             "../runo/icons/64/document.png");
 
     public static class File {
index f57d8418ec0af71d5c7d52bd221b49c05a68a99b..d179f9c9ecc3929e4245e90eb8550540db27443d 100644 (file)
@@ -10,14 +10,14 @@ import com.vaadin.ui.Notification.Type;
 
 public class NotificationGetTypeAndDescription extends AbstractTestUI {
 
-    private final static Type[] types = { Type.WARNING_MESSAGE,
+    private static final Type[] types = { Type.WARNING_MESSAGE,
             Type.ERROR_MESSAGE, Type.HUMANIZED_MESSAGE,
             Type.TRAY_NOTIFICATION };
-    public final static String[] type_names = { "warning", "error", "humanized",
+    public static final String[] type_names = { "warning", "error", "humanized",
             "tray_notification" };
-    public final static String[] captions = { "warningC", "errorC",
+    public static final String[] captions = { "warningC", "errorC",
             "humanizedC", "tray_notificationC" };
-    public final static String[] descriptions = { "warning", "error",
+    public static final String[] descriptions = { "warning", "error",
             "humanized", "tray_notification" };
 
     @Override
index c950a5cdaff2baaeb09860f4f86bdeb6d01a323d..0f5baef0245eda302feae2a007fe55d757f88850 100644 (file)
@@ -23,7 +23,7 @@ import com.vaadin.ui.AbstractOrderedLayout;
  * @since
  * @author Vaadin Ltd
  */
-abstract public class BaseAlignment extends BaseLayoutTestUI {
+public abstract class BaseAlignment extends BaseLayoutTestUI {
 
     @Override
     protected void setup(VaadinRequest request) {
index 44c3201cf4643761ee3b71cc49d2a71b84a90cd9..dd5649c6dd02342decf7652f195c489bb65a342f 100644 (file)
@@ -3,9 +3,9 @@ package com.vaadin.tests.util;
 import java.util.concurrent.atomic.AtomicLong;
 
 public class PortableRandom {
-    private final static long multiplier = 0x5DEECE66DL;
-    private final static long addend = 0xBL;
-    private final static long mask = (1L << 48) - 1;
+    private static final long multiplier = 0x5DEECE66DL;
+    private static final long addend = 0xBL;
+    private static final long mask = (1L << 48) - 1;
     private AtomicLong seed;
 
     public PortableRandom(long seed) {
index 8498218e88afb0af3adf141fd6fbd00641ac0ba7..0f5b55a612a9208f5069cfa15baaca287b1aa111 100644 (file)
@@ -71,7 +71,7 @@ public class TestWidgetConnector extends AbstractComponentConnector {
     private final TestWidgetRegistry registry = GWT
             .create(TestWidgetRegistry.class);
 
-    public static abstract class TestWidgetRegistry {
+    public abstract static class TestWidgetRegistry {
         private Map<String, Invoker> creators = new HashMap<>();
 
         // Called by generated sub class
index af3e45df26898822bc1709828d486f805e4efbe1..bb3b09aedfae558401afc5631d1e109a76eccbbd 100644 (file)
@@ -52,7 +52,7 @@ public class CsrfButtonConnector extends AbstractComponentConnector {
         return GWT.create(VButton.class);
     }
 
-    public final static String ID = "CsrfButton";
+    public static final String ID = "CsrfButton";
 
     @Override
     public void init() {
index 9014ed68b3bb1f9c4df434173612670963cab910..e70c7c87f6782f3c0884a5c65cd909b936dc9686 100644 (file)
@@ -211,7 +211,7 @@ public class GridBasicClientFeaturesWidget
     /**
      * Our basic data object
      */
-    public final static class Data {
+    public static final class Data {
         Object value;
     }
 
index f2a109c1cd0609df3ac69fbcffbf4c7fb05e30c7..78491a30cfb6380cdf7bbf33b41d02523a74227f 100644 (file)
@@ -32,7 +32,7 @@ public class GwtRpc extends AbstractReindeerTestUI {
     /**
      * Id of the button triggering the test case.
      */
-    public final static String BUTTON_ID = "gwtRpcButton";
+    public static final String BUTTON_ID = "gwtRpcButton";
 
     @Override
     protected void setup(VaadinRequest request) {
index 452f61ee1bb40f4742968f0ffbd6d94c0615938f..22bfbfffdf47b9d98ca6a2aa8a8605f273bc7a68 100644 (file)
@@ -30,7 +30,7 @@ import com.vaadin.tests.components.grid.basicfeatures.EscalatorBasicClientFeatur
 public class EscalatorColumnFreezingTest
         extends EscalatorBasicClientFeaturesTest {
 
-    private final static Pattern TRANSFORM_PATTERN = Pattern.compile(// @formatter:off
+    private static final Pattern TRANSFORM_PATTERN = Pattern.compile(// @formatter:off
             // any start of the string
             ".*"
 
@@ -48,7 +48,7 @@ public class EscalatorColumnFreezingTest
 
             // @formatter:on
 
-    private final static Pattern LEFT_PATTERN = Pattern
+    private static final Pattern LEFT_PATTERN = Pattern
             .compile(".*left: (\\d+)px.*", Pattern.CASE_INSENSITIVE);
 
     private static final int NO_FREEZE = -1;
index f3c97cfeea8cc6075426c4461c60ca85e4e267ef..449a9528724523973c53654c9e4e25ebcb58a3d8 100644 (file)
@@ -48,7 +48,7 @@ public class EscalatorSpacerTest extends EscalatorBasicClientFeaturesTest {
     // translate3d(0px, 40px, 123px);
     // translate3d(24px, 15.251px, 0);
     // translate(0, 40px);
-    private final static String TRANSLATE_VALUE_REGEX =
+    private static final String TRANSLATE_VALUE_REGEX =
             "translate(?:3d|)" // "translate" or "translate3d"
             + "\\(" // literal "("
                 + "(" // start capturing the x argument
@@ -74,7 +74,7 @@ public class EscalatorSpacerTest extends EscalatorBasicClientFeaturesTest {
 
     // 40px;
     // 12.34px
-    private final static String PIXEL_VALUE_REGEX =
+    private static final String PIXEL_VALUE_REGEX =
             "(" // capture the pixel value
                 + "[0-9]+" // the pixel argument
                 + "(?:" // start of the subpixel part of the value
@@ -87,16 +87,16 @@ public class EscalatorSpacerTest extends EscalatorBasicClientFeaturesTest {
     //@formatter:on
 
     // also matches "-webkit-transform";
-    private final static Pattern TRANSFORM_CSS_PATTERN = Pattern
+    private static final Pattern TRANSFORM_CSS_PATTERN = Pattern
             .compile("transform: (.*?);");
-    private final static Pattern TOP_CSS_PATTERN = Pattern.compile(
+    private static final Pattern TOP_CSS_PATTERN = Pattern.compile(
             "top: ([0-9]+(?:\\.[0-9]+)?(?:px)?);?", Pattern.CASE_INSENSITIVE);
-    private final static Pattern LEFT_CSS_PATTERN = Pattern.compile(
+    private static final Pattern LEFT_CSS_PATTERN = Pattern.compile(
             "left: ([0-9]+(?:\\.[0-9]+)?(?:px)?);?", Pattern.CASE_INSENSITIVE);
 
-    private final static Pattern TRANSLATE_VALUE_PATTERN = Pattern
+    private static final Pattern TRANSLATE_VALUE_PATTERN = Pattern
             .compile(TRANSLATE_VALUE_REGEX);
-    private final static Pattern PIXEL_VALUE_PATTERN = Pattern
+    private static final Pattern PIXEL_VALUE_PATTERN = Pattern
             .compile(PIXEL_VALUE_REGEX, Pattern.CASE_INSENSITIVE);
 
     @Before
index 52b806dcd709fc62118903cc7133890782e0f104..98604c2a6fc6ec74fa4df058b6eb4ca4a504e66a 100644 (file)
@@ -156,7 +156,7 @@ public class TabKeyboardNavigationTest extends MultiBrowserTest {
     /*
      * Delay for PhantomJS.
      */
-    private final static int DELAY = 10;
+    private static final int DELAY = 10;
 
     private void assertSheet(int index) {
         String labelCaption = "Tab " + index;
index 66eb9564042bdefce3e18913894419fa8d133a80..a36faf0c27f13b43797ae45a89fb2b1bafb2872c 100644 (file)
@@ -91,7 +91,7 @@ public class TabSelectionRevertedByServerTest extends MultiBrowserTest {
     /*
      * Delay for PhantomJS.
      */
-    private final static int DELAY = 10;
+    private static final int DELAY = 10;
 
     /*
      * Provide the tab at specified index.
index 5e6ab39c6b8d755d423d93ebbbf4744eac32c98b..587836fa6a77bdcc87d07f68c46feb06047a6c99 100644 (file)
@@ -33,7 +33,7 @@ import com.vaadin.tests.tb3.MultiBrowserTest;
  * @author Vaadin Ltd
  */
 public class AbstractTextElementSetValueTest extends MultiBrowserTest {
-    private final static String TYPED_STRING = "this is typed string";
+    private static final String TYPED_STRING = "this is typed string";
 
     @Before
     public void init() {
index c7f4b8d11e0f48a27605dbea1552df906efe62f2..d96f5df84a1bfd3aecd314c02acf7c45fb8bcea6 100644 (file)
@@ -29,7 +29,7 @@ import com.vaadin.tests.tb3.MultiBrowserTest;
 
 public class CompatibilityAbstractTextElementSetValueTest
         extends MultiBrowserTest {
-    private final static String TYPED_STRING = "this is typed string";
+    private static final String TYPED_STRING = "this is typed string";
 
     @Before
     public void init() {