Changeset 11331 in josm


Ignore:
Timestamp:
2016-11-28T01:35:12+01:00 (8 years ago)
Author:
Don-vip
Message:

findbugs - RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE

Location:
trunk/src/org/openstreetmap/josm
Files:
4 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/org/openstreetmap/josm/data/cache/BufferedImageCacheEntry.java

    r10938 r11331  
    4444                return img;
    4545            byte[] content = getContent();
    46             if (content != null && content.length > 0) {
     46            if (content.length > 0) {
    4747                img = ImageIO.read(new ByteArrayInputStream(content));
    4848                imageLoaded = true;
  • trunk/src/org/openstreetmap/josm/data/cache/JCSCachedTileLoaderJob.java

    r11288 r11331  
    196196            return false;
    197197        }
    198         byte[] content = cacheData.getContent();
    199         return content != null && content.length > 0;
     198        return cacheData.getContent().length > 0;
    200199    }
    201200
  • trunk/src/org/openstreetmap/josm/data/imagery/TMSCachedTileLoaderJob.java

    r11288 r11331  
    124124            byte[] content = cacheData.getContent();
    125125            try {
    126                 return content != null || cacheData.getImage() != null || isNoTileAtZoom();
     126                return content.length > 0 || cacheData.getImage() != null || isNoTileAtZoom();
    127127            } catch (IOException e) {
    128128                LOG.log(Level.WARNING, "JCS TMS - error loading from cache for tile {0}: {1}", new Object[] {tile.getKey(), e.getMessage()});
     
    325325        if (object != null) {
    326326            byte[] content = object.getContent();
    327             if (content != null && content.length > 0) {
     327            if (content.length > 0) {
    328328                tile.loadImage(new ByteArrayInputStream(content));
    329329                if (tile.getImage() == null) {
  • trunk/src/org/openstreetmap/josm/gui/conflict/pair/AbstractListMerger.java

    r11330 r11331  
    490490        public void actionPerformed(ActionEvent e) {
    491491            int[] mergedRows = mergedEntriesTable.getSelectedRows();
    492             if (mergedRows == null || mergedRows.length == 0)
     492            if (mergedRows.length == 0)
    493493                return;
    494494            int[] myRows = myEntriesTable.getSelectedRows();
     
    520520        public void actionPerformed(ActionEvent e) {
    521521            int[] mergedRows = mergedEntriesTable.getSelectedRows();
    522             if (mergedRows == null || mergedRows.length == 0)
     522            if (mergedRows.length == 0)
    523523                return;
    524524            int[] myRows = myEntriesTable.getSelectedRows();
     
    582582        public void actionPerformed(ActionEvent e) {
    583583            int[] mergedRows = mergedEntriesTable.getSelectedRows();
    584             if (mergedRows == null || mergedRows.length == 0)
     584            if (mergedRows.length == 0)
    585585                return;
    586586            int[] myRows = theirEntriesTable.getSelectedRows();
     
    608608        public void actionPerformed(ActionEvent e) {
    609609            int[] mergedRows = mergedEntriesTable.getSelectedRows();
    610             if (mergedRows == null || mergedRows.length == 0)
     610            if (mergedRows.length == 0)
    611611                return;
    612612            int[] myRows = theirEntriesTable.getSelectedRows();
     
    703703        public void valueChanged(ListSelectionEvent e) {
    704704            int[] rows = mergedEntriesTable.getSelectedRows();
    705             setEnabled(
    706                     rows != null
    707                     && rows.length > 0
     705            setEnabled(rows.length > 0
    708706                    && rows[0] != 0
    709707            );
     
    737735        public void valueChanged(ListSelectionEvent e) {
    738736            int[] rows = mergedEntriesTable.getSelectedRows();
    739             setEnabled(
    740                     rows != null
    741                     && rows.length > 0
     737            setEnabled(rows.length > 0
    742738                    && rows[rows.length -1] != mergedEntriesTable.getRowCount() -1
    743739            );
     
    771767        public void valueChanged(ListSelectionEvent e) {
    772768            int[] rows = mergedEntriesTable.getSelectedRows();
    773             setEnabled(
    774                     rows != null
    775                     && rows.length > 0
    776             );
     769            setEnabled(rows.length > 0);
    777770        }
    778771    }
Note: See TracChangeset for help on using the changeset viewer.