Ignore:
Timestamp:
2016-10-07T10:20:53+02:00 (3 years ago)
Author:
Don-vip
Message:

sonar - squid:S1141 - Try-catch blocks should not be nested

File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/org/openstreetmap/josm/gui/history/HistoryLoadTask.java

    r10106 r11087  
    171171                    break;
    172172                }
    173                 String msg = getLoadingMessage(pid);
    174                 progressMonitor.indeterminateSubTask(tr(msg, Long.toString(pid.getUniqueId())));
    175                 reader = null;
    176                 HistoryDataSet ds;
    177                 try {
    178                     reader = new OsmServerHistoryReader(pid.getType(), pid.getUniqueId());
    179                     ds = loadHistory(reader, progressMonitor);
    180                 } catch (OsmTransferException e) {
    181                     if (canceled)
    182                         return;
    183                     throw e;
    184                 }
    185                 loadedData.mergeInto(ds);
     173                loadHistory(pid);
    186174            }
    187175        } catch (OsmTransferException e) {
     
    189177            return;
    190178        }
     179    }
     180
     181    private void loadHistory(PrimitiveId pid) throws OsmTransferException {
     182        String msg = getLoadingMessage(pid);
     183        progressMonitor.indeterminateSubTask(tr(msg, Long.toString(pid.getUniqueId())));
     184        reader = null;
     185        HistoryDataSet ds;
     186        try {
     187            reader = new OsmServerHistoryReader(pid.getType(), pid.getUniqueId());
     188            ds = loadHistory(reader, progressMonitor);
     189        } catch (OsmTransferException e) {
     190            if (canceled)
     191                return;
     192            throw e;
     193        }
     194        loadedData.mergeInto(ds);
    191195    }
    192196
Note: See TracChangeset for help on using the changeset viewer.