The following document contains the results of PMD 5.1.2.
Violation | Line |
---|---|
Avoid modifiers which are implied by the context | 33 |
Avoid modifiers which are implied by the context | 41 |
Avoid modifiers which are implied by the context | 48 |
Avoid modifiers which are implied by the context | 62 |
Avoid modifiers which are implied by the context | 69 |
Violation | Line |
---|---|
Avoid modifiers which are implied by the context | 30 |
Avoid modifiers which are implied by the context | 38 |
Avoid modifiers which are implied by the context | 46 |
Avoid modifiers which are implied by the context | 53 |
Avoid modifiers which are implied by the context | 60 |
Avoid modifiers which are implied by the context | 67 |
Violation | Line |
---|---|
Avoid modifiers which are implied by the context | 30 |
Avoid modifiers which are implied by the context | 38 |
Avoid modifiers which are implied by the context | 46 |
Avoid modifiers which are implied by the context | 53 |
Avoid modifiers which are implied by the context | 60 |
Avoid modifiers which are implied by the context | 67 |
Violation | Line |
---|---|
Avoid modifiers which are implied by the context | 31 |
Avoid modifiers which are implied by the context | 39 |
Avoid modifiers which are implied by the context | 46 |
Avoid modifiers which are implied by the context | 53 |
Avoid modifiers which are implied by the context | 61 |
Avoid modifiers which are implied by the context | 69 |
Violation | Line |
---|---|
Avoid modifiers which are implied by the context | 4 |
Avoid modifiers which are implied by the context | 6 |
Violation | Line |
---|---|
Avoid unused private fields such as 'x'. | 34 |
Avoid unused private fields such as 'y'. | 35 |
Avoid unused private fields such as 'x_start'. | 36 |
Avoid unused private fields such as 'y_start'. | 37 |
Avoid unused private fields such as 'multiTouch'. | 38 |
Violation | Line |
---|---|
Avoid unused private fields such as 'x'. | 34 |
Avoid unused private fields such as 'y'. | 35 |
Avoid unused private fields such as 'x_start'. | 37 |
Avoid unused private fields such as 'y_start'. | 38 |
Avoid unused private fields such as 'multiTouch'. | 39 |
Violation | Line |
---|---|
Avoid modifiers which are implied by the context | 34 |
Violation | Line |
---|---|
Unnecessary use of fully qualified name 'com.google.gwt.event.shared.GwtEvent.Type' due to existing import 'com.google.gwt.event.shared.GwtEvent' | 56 |
Violation | Line |
---|---|
Avoid modifiers which are implied by the context | 27 |
Violation | Line |
---|---|
Avoid modifiers which are implied by the context | 39 |
Avoid modifiers which are implied by the context | 51 |
Avoid modifiers which are implied by the context | 63 |
Avoid modifiers which are implied by the context | 75 |
Avoid modifiers which are implied by the context | 87 |
Violation | Line |
---|---|
Unnecessary final modifier in final class | 34–36 |
Unnecessary final modifier in final class | 43–45 |
Violation | Line |
---|---|
Avoid modifiers which are implied by the context | 31 |
Avoid modifiers which are implied by the context | 38 |
Avoid modifiers which are implied by the context | 40 |
Violation | Line |
---|---|
Avoid modifiers which are implied by the context | 32 |
Violation | Line |
---|---|
Avoid modifiers which are implied by the context | 32 |
Violation | Line |
---|---|
Avoid modifiers which are implied by the context | 32 |
Violation | Line |
---|---|
Avoid modifiers which are implied by the context | 32 |
Violation | Line |
---|---|
Avoid modifiers which are implied by the context | 34 |
Violation | Line |
---|---|
Avoid modifiers which are implied by the context | 26 |
Violation | Line |
---|---|
Avoid modifiers which are implied by the context | 26 |
Avoid modifiers which are implied by the context | 28–30 |
Avoid modifiers which are implied by the context | 29 |
Violation | Line |
---|---|
Avoid modifiers which are implied by the context | 34 |
Violation | Line |
---|---|
Unnecessary use of fully qualified name 'com.google.gwt.event.shared.GwtEvent.Type' due to existing import 'com.google.gwt.event.shared.GwtEvent' | 67 |
Violation | Line |
---|---|
Avoid modifiers which are implied by the context | 32 |
Violation | Line |
---|---|
These nested if statements could be combined | 202–205 |
Violation | Line |
---|---|
Avoid modifiers which are implied by the context | 33 |
Violation | Line |
---|---|
Avoid modifiers which are implied by the context | 32 |
Avoid modifiers which are implied by the context | 39 |
Violation | Line |
---|---|
Unnecessary use of fully qualified name 'com.google.gwt.event.shared.GwtEvent.Type' due to existing import 'com.google.gwt.event.shared.GwtEvent' | 64 |
Violation | Line |
---|---|
Avoid modifiers which are implied by the context | 33 |
Violation | Line |
---|---|
Avoid modifiers which are implied by the context | 34 |
Avoid modifiers which are implied by the context | 42 |
Avoid modifiers which are implied by the context | 50 |
Violation | Line |
---|---|
Unnecessary use of fully qualified name 'com.google.gwt.event.shared.GwtEvent.Type' due to existing import 'com.google.gwt.event.shared.GwtEvent' | 54 |
Violation | Line |
---|---|
Avoid modifiers which are implied by the context | 32 |
Violation | Line |
---|---|
Unnecessary use of fully qualified name 'com.google.gwt.event.shared.GwtEvent.Type' due to existing import 'com.google.gwt.event.shared.GwtEvent' | 60 |
Violation | Line |
---|---|
Avoid modifiers which are implied by the context | 32 |
Violation | Line |
---|---|
Unnecessary use of fully qualified name 'com.google.gwt.event.shared.GwtEvent.Type' due to existing import 'com.google.gwt.event.shared.GwtEvent' | 58 |
Violation | Line |
---|---|
Unnecessary use of fully qualified name 'com.google.gwt.event.shared.GwtEvent.Type' due to existing import 'com.google.gwt.event.shared.GwtEvent' | 85 |
Violation | Line |
---|---|
Avoid modifiers which are implied by the context | 32 |
Violation | Line |
---|---|
These nested if statements could be combined | 191–194 |
These nested if statements could be combined | 265–269 |
Violation | Line |
---|---|
Avoid modifiers which are implied by the context | 40 |
Avoid modifiers which are implied by the context | 49 |
Avoid modifiers which are implied by the context | 58 |
Avoid modifiers which are implied by the context | 67 |
Violation | Line |
---|---|
Avoid unused imports such as 'com.google.gwt.event.dom.client.ContextMenuEvent' | 52 |
Avoid unused imports such as 'com.google.gwt.event.dom.client.ContextMenuHandler' | 53 |
Avoid unused imports such as 'com.google.gwt.user.client.ui.RootPanel' | 81 |
Avoid modifiers which are implied by the context | 165 |
Avoid unused private fields such as 'detachedScrollPosition'. | 440 |
Avoid unused private fields such as 'initialContentReceived'. | 692 |
Avoid unused private fields such as 'touchScrollDelegate'. | 733 |
Avoid unused private methods such as 'sendColumnWidthUpdates(Collection)'. | 974 |
Overriding method merely calls super | 2208–2211 |
Useless parentheses. | 2370–2371 |
Useless parentheses. | 2396–2397 |
Avoid empty if statements | 2444–2446 |
Avoid unused local variables such as 'hCell'. | 2452 |
Useless parentheses. | 2497 |
These nested if statements could be combined | 2513–2526 |
These nested if statements could be combined | 2515–2525 |
Useless parentheses. | 2577 |
Useless parentheses. | 2635–2636 |
Useless parentheses. | 2636–2637 |
Useless parentheses. | 2688 |
Useless parentheses. | 2737 |
Useless parentheses. | 2757 |
These nested if statements could be combined | 3302–3346 |
These nested if statements could be combined | 3773–3776 |
These nested if statements could be combined | 3955–3961 |
These nested if statements could be combined | 4657–4660 |
Useless parentheses. | 5207 |
Unnecessary use of fully qualified name 'com.google.gwt.dom.client.Element' due to existing import 'com.google.gwt.dom.client.Element' | 5459 |
Unnecessary use of fully qualified name 'com.google.gwt.dom.client.Element' due to existing import 'com.google.gwt.dom.client.Element' | 5945 |
These nested if statements could be combined | 6146–6257 |
Useless parentheses. | 6187 |
Useless parentheses. | 6284–6285 |
Useless parentheses. | 6382 |
Unnecessary use of fully qualified name 'com.google.gwt.dom.client.Element' due to existing import 'com.google.gwt.dom.client.Element' | 6693 |
Avoid unused method parameters such as 'rowUidl'. | 6782 |
Avoid unused method parameters such as 'rowUidl'. | 6789 |
Useless parentheses. | 6894 |
Useless parentheses. | 6894 |
Useless parentheses. | 7130–7131 |
Useless parentheses. | 7188 |
These nested if statements could be combined | 7352–7361 |
Useless parentheses. | 7464 |
Useless parentheses. | 7490 |
Useless parentheses. | 7516–7517 |
Useless parentheses. | 7526–7527 |
Ensure you override both equals() and hashCode() | 7545 |
Useless parentheses. | 7550 |
Useless parentheses. | 7550 |
Avoid unused private methods such as 'getLogger()'. | 8435 |
Violation | Line |
---|---|
Useless parentheses. | 251 |
Useless parentheses. | 252 |
Avoid empty if statements | 263–264 |
Avoid empty if statements | 265–266 |
Avoid empty if statements | 266–267 |
These nested if statements could be combined | 328–331 |
Useless parentheses. | 364 |
Useless parentheses. | 433 |
Useless parentheses. | 434 |
Useless parentheses. | 445 |
Useless parentheses. | 572 |
Violation | Line |
---|---|
Unnecessary use of fully qualified name 'com.google.gwt.dom.client.Element' due to existing import 'com.google.gwt.dom.client.Element' | 151 |
Unnecessary use of fully qualified name 'com.google.gwt.dom.client.Element' due to existing import 'com.google.gwt.dom.client.Element' | 277 |
Useless parentheses. | 810 |
Useless parentheses. | 810 |
Violation | Line |
---|---|
Avoid unused imports such as 'com.google.gwt.dom.client.Style' | 20 |
Avoid unused imports such as 'com.google.gwt.dom.client.Style.Unit' | 22 |
Avoid unused imports such as 'com.vaadin.client.ComponentConnector' | 27 |
Avoid unused imports such as 'com.vaadin.client.ConnectorHierarchyChangeEvent' | 28 |
Avoid unused imports such as 'com.vaadin.client.LayoutManager' | 29 |
Avoid unused imports such as 'com.vaadin.client.Profiler' | 31 |
Avoid unused imports such as 'com.vaadin.client.ui.AbstractSingleComponentContainerConnector' | 33 |
No need to import a type that lives in the same package | 40 |
Avoid unused imports such as 'com.vaadin.shared.ui.panel.PanelState' | 45 |
Violation | Line |
---|---|
Avoid unused imports such as 'com.vaadin.shared.ui.Connect' | 50 |
Avoid unused local variables such as 'totalRowsHaveChanged'. | 188 |
Useless parentheses. | 188 |
These nested if statements could be combined | 328–348 |
Violation | Line |
---|---|
These nested if statements could be combined | 58–66 |
Useless parentheses. | 82–83 |
Violation | Line |
---|---|
Avoid modifiers which are implied by the context | 217 |
Avoid modifiers which are implied by the context | 218 |
Avoid modifiers which are implied by the context | 220 |
Violation | Line |
---|---|
Unnecessary use of fully qualified name 'com.vaadin.ui.Component.Event' due to existing import 'com.vaadin.ui.Component' | 240 |
Unnecessary use of fully qualified name 'com.vaadin.ui.Component.Event' due to existing import 'com.vaadin.ui.Component' | 274 |
Unnecessary use of fully qualified name 'com.vaadin.ui.Component.Event' due to existing import 'com.vaadin.ui.Component' | 307 |
Violation | Line |
---|---|
Overriding method merely calls super | 128–130 |
Violation | Line |
---|---|
Overriding method merely calls super | 62–64 |
Avoid modifiers which are implied by the context | 151 |
Avoid modifiers which are implied by the context | 152–153 |
Avoid modifiers which are implied by the context | 161 |
Avoid modifiers which are implied by the context | 162–163 |
Violation | Line |
---|---|
Avoid unused constructor parameters such as 'handler'. | 65 |
Violation | Line |
---|---|
Useless parentheses. | 158 |
Useless parentheses. | 158 |
Violation | Line |
---|---|
Avoid unused private fields such as 'log'. | 56 |
Overriding method merely calls super | 74–76 |
Overriding method merely calls super | 115–117 |
Violation | Line |
---|---|
Useless parentheses. | 74 |
Violation | Line |
---|---|
Useless parentheses. | 332 |
Useless parentheses. | 335 |
Violation | Line |
---|---|
Useless parentheses. | 123 |
Overriding method merely calls super | 143–145 |
Violation | Line |
---|---|
Avoid modifiers which are implied by the context | 52–57 |
Violation | Line |
---|---|
Useless parentheses. | 117 |
Violation | Line |
---|---|
Avoid modifiers which are implied by the context | 66–73 |
Violation | Line |
---|---|
These nested if statements could be combined | 124–126 |
Violation | Line |
---|---|
Avoid unused private methods such as 'setIndicatorVisible(boolean)'. | 140 |
Violation | Line |
---|---|
Useless parentheses. | 79 |
Violation | Line |
---|---|
Overriding method merely calls super | 48–50 |
Violation | Line |
---|---|
Useless parentheses. | 181 |
Violation | Line |
---|---|
Overriding method merely calls super | 176–178 |
Unnecessary use of fully qualified name 'com.google.gwt.user.client.Element' due to existing import 'com.google.gwt.user.client.Element' | 222 |
Violation | Line |
---|---|
Useless parentheses. | 60 |
Violation | Line |
---|---|
Avoid modifiers which are implied by the context | 40 |
Violation | Line |
---|---|
Unnecessary final modifier in final class | 46–50 |
Violation | Line |
---|---|
Avoid unused private fields such as 'rules'. | 48 |
Avoid unused private methods such as 'searchForRule(JavaScriptObject,String,boolean)'. | 80–82 |
Violation | Line |
---|---|
Avoid modifiers which are implied by the context | 45 |
Avoid modifiers which are implied by the context | 47 |
Violation | Line |
---|---|
Avoid unused private methods such as 'scaleDimension(int,double)'. | 111 |
Violation | Line |
---|---|
These nested if statements could be combined | 231–233 |
Violation | Line |
---|---|
Useless parentheses. | 235 |
Violation | Line |
---|---|
Avoid modifiers which are implied by the context | 45 |
Avoid modifiers which are implied by the context | 47 |
Avoid modifiers which are implied by the context | 50 |
Violation | Line |
---|---|
Avoid unused local variables such as 'style'. | 195 |
Useless parentheses. | 402 |
Violation | Line |
---|---|
These nested if statements could be combined | 115–118 |
Violation | Line |
---|---|
Overriding method merely calls super | 160–162 |
Violation | Line |
---|---|
Avoid modifiers which are implied by the context | 41 |
Violation | Line |
---|---|
Avoid unused constructor parameters such as 'connector'. | 48 |
Violation | Line |
---|---|
Useless parentheses. | 143 |
Useless parentheses. | 146 |
Violation | Line |
---|---|
Avoid modifiers which are implied by the context | 44 |
Violation | Line |
---|---|
Unnecessary use of fully qualified name 'com.vaadin.data.util.converter.Converter.ConversionException' due to existing import 'com.vaadin.data.util.converter.Converter' | 130 |
Unnecessary use of fully qualified name 'com.vaadin.data.util.converter.Converter.ConversionException' due to existing import 'com.vaadin.data.util.converter.Converter' | 135 |
Violation | Line |
---|---|
Avoid unused imports such as 'com.vaadin.client.VConsole' | 26 |
These nested if statements could be combined | 338–340 |
These nested if statements could be combined | 437–447 |
Avoid unnecessary return statements | 512 |
Violation | Line |
---|---|
Avoid modifiers which are implied by the context | 526–528 |
Avoid modifiers which are implied by the context | 536 |
Avoid modifiers which are implied by the context | 557–560 |
Avoid modifiers which are implied by the context | 566 |
Avoid modifiers which are implied by the context | 599–615 |
Avoid modifiers which are implied by the context | 601–605 |
Avoid modifiers which are implied by the context | 613–614 |
Avoid modifiers which are implied by the context | 623–666 |
Avoid modifiers which are implied by the context | 625–628 |
Avoid modifiers which are implied by the context | 630–633 |
Avoid modifiers which are implied by the context | 635–638 |
Avoid modifiers which are implied by the context | 646–647 |
Avoid modifiers which are implied by the context | 655–656 |
Avoid modifiers which are implied by the context | 664–665 |
Avoid modifiers which are implied by the context | 668–682 |
Avoid modifiers which are implied by the context | 669–672 |
Avoid modifiers which are implied by the context | 680–681 |
Violation | Line |
---|---|
Avoid modifiers which are implied by the context | 19 |
Violation | Line |
---|---|
These nested if statements could be combined | 112–114 |