Changeset 13220 in josm for trunk/tools


Ignore:
Timestamp:
2017-12-18T00:46:58+01:00 (7 years ago)
Author:
Don-vip
Message:

see #15574:

  • additionally refactors ImageDisplay to use ImageEntry instead; stores width and height info while metadata of images are read; might break plugin code (patch by cmuelle8, minor changes)
  • remove double semicolon causing ​https://github.com/pmd/pmd/issues/785
  • enable PMD rule DoNotCallGarbageCollectionExplicitly
  • disable SpotBugs rule ST_WRITE_TO_STATIC_FROM_INSTANCE_METHOD
Location:
trunk/tools
Files:
2 edited

Legend:

Unmodified
Added
Removed

  • trunk/tools/pmd/josm-ruleset.xml

    r13207 r13220  
    157157    <exclude name="ConstructorCallsOverridableMethod"/>
    158158    <exclude name="DataflowAnomalyAnalysis"/>
    159     <exclude name="DoNotCallGarbageCollectionExplicitly"/>
    160159    <exclude name="DoNotCallSystemExit"/>
    161160    <exclude name="DontImportSun"/>

  • trunk/tools/spotbugs/josm-filter.xml

    r12277 r13220  
    11<FindBugsFilter>
    22
    3         <Match>
    4                 <Bug pattern="DM_EXIT" />
    5         </Match>
    6         <Match>
    7                 <Bug pattern="DMI_HARDCODED_ABSOLUTE_FILENAME" />
    8         </Match>
    9         <Match>
    10                 <Bug pattern="EQ_DOESNT_OVERRIDE_EQUALS" />
    11         </Match>
    12         <Match>
    13                 <Bug pattern="IL_INFINITE_LOOP" />
    14         </Match>
    15         <Match>
    16                 <Bug pattern="NM_CONFUSING" />
    17         </Match>
    18         <Match>
    19                 <Bug pattern="RV_RETURN_VALUE_IGNORED_BAD_PRACTICE" />
    20         </Match>
    21         <Match>
    22                 <Bug pattern="SE_BAD_FIELD" />
    23         </Match>
    24         <Match>
    25                 <Bug pattern="SE_TRANSIENT_FIELD_NOT_RESTORED" />
    26         </Match>
    27         <Match>
    28                 <Bug pattern="UI_INHERITANCE_UNSAFE_GETRESOURCE" />
    29         </Match>
    30         <Match>
    31                 <Bug pattern="UWF_FIELD_NOT_INITIALIZED_IN_CONSTRUCTOR" />
    32         </Match>
     3    <Match>
     4        <Bug pattern="DM_EXIT" />
     5    </Match>
     6    <Match>
     7        <Bug pattern="DMI_HARDCODED_ABSOLUTE_FILENAME" />
     8    </Match>
     9    <Match>
     10        <Bug pattern="EQ_DOESNT_OVERRIDE_EQUALS" />
     11    </Match>
     12    <Match>
     13        <Bug pattern="IL_INFINITE_LOOP" />
     14    </Match>
     15    <Match>
     16        <Bug pattern="NM_CONFUSING" />
     17    </Match>
     18    <Match>
     19        <Bug pattern="RV_RETURN_VALUE_IGNORED_BAD_PRACTICE" />
     20    </Match>
     21    <Match>
     22        <Bug pattern="SE_BAD_FIELD" />
     23    </Match>
     24    <Match>
     25        <Bug pattern="SE_TRANSIENT_FIELD_NOT_RESTORED" />
     26    </Match>
     27    <Match>
     28        <Bug pattern="ST_WRITE_TO_STATIC_FROM_INSTANCE_METHOD" />
     29    </Match>
     30    <Match>
     31        <Bug pattern="UI_INHERITANCE_UNSAFE_GETRESOURCE" />
     32    </Match>
     33    <Match>
     34        <Bug pattern="UWF_FIELD_NOT_INITIALIZED_IN_CONSTRUCTOR" />
     35    </Match>
    3336
    34         <Match>
    35                 <Bug pattern="EI_EXPOSE_REP2" />
    36                 <Class name="org.openstreetmap.josm.tools.CopyList" />
    37         </Match>
    38         <Match>
    39                 <Bug pattern="MS_CANNOT_BE_FINAL" />
    40                 <Class name="org.openstreetmap.josm.Main" />
    41         </Match>
    42         <Match>
    43                 <Bug pattern="MS_SHOULD_BE_FINAL" />
    44                 <Class name="org.openstreetmap.josm.Main" />
    45         </Match>
     37    <Match>
     38        <Bug pattern="EI_EXPOSE_REP2" />
     39        <Class name="org.openstreetmap.josm.tools.CopyList" />
     40    </Match>
     41    <Match>
     42        <Bug pattern="MS_CANNOT_BE_FINAL" />
     43        <Class name="org.openstreetmap.josm.Main" />
     44    </Match>
     45    <Match>
     46        <Bug pattern="MS_SHOULD_BE_FINAL" />
     47        <Class name="org.openstreetmap.josm.Main" />
     48    </Match>
    4649
    47         <Match>
    48                 <Class name="~com.*" />
    49         </Match>
     50    <Match>
     51        <Class name="~com.*" />
     52    </Match>
    5053    <Match>
    5154        <Class name="~gnu.getopt.*" />
    … …  
    5457        <Class name="~javax.json.*" />
    5558    </Match>
    56         <Match>
    57                 <Class name="~oauth.signpost.*" />
    58         </Match>
    59         <Match>
    60                 <Class name="~org.apache.*" />
    61         </Match>
     59    <Match>
     60        <Class name="~oauth.signpost.*" />
     61    </Match>
     62    <Match>
     63        <Class name="~org.apache.*" />
     64    </Match>
    6265    <Match>
    6366        <Class name="~org.glassfish.json.*" />
    6467    </Match>
    65         <Match>
    66                 <Class name="~org.jdesktop.swinghelper.debug.*" />
    67         </Match>
    68         <Match>
    69                 <Class name="~org.openstreetmap.gui.jmapviewer.*" />
    70         </Match>
     68    <Match>
     69        <Class name="~org.jdesktop.swinghelper.debug.*" />
     70    </Match>
     71    <Match>
     72        <Class name="~org.openstreetmap.gui.jmapviewer.*" />
     73    </Match>
    7174    <Match>
    7275        <Class name="~org.openstreetmap.josm.gui.mappaint.mapcss.parsergen.*" />
Note: See TracChangeset for help on using the changeset viewer.