Changeset 11008 in josm for trunk


Ignore:
Timestamp:
2016-09-17T14:57:33+02:00 (3 years ago)
Author:
Don-vip
Message:

fix #13646 - NPE + unit test

Location:
trunk
Files:
1 added
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/org/openstreetmap/josm/gui/layer/LayerPositionStrategy.java

    r10717 r11008  
    5555    static LayerPositionStrategy inFrontOfFirst(final Predicate<Layer> what) {
    5656        return manager -> {
    57             List<Layer> layers = manager.getLayers();
    58             for (int i = 0; i < layers.size(); i++) {
    59                 if (what.test(layers.get(i))) {
    60                     return i;
     57            if (manager != null) {
     58                List<Layer> layers = manager.getLayers();
     59                for (int i = 0; i < layers.size(); i++) {
     60                    if (what.test(layers.get(i))) {
     61                        return i;
     62                    }
    6163                }
     64                return layers.size();
    6265            }
    63             return layers.size();
     66            return 0;
    6467        };
    6568    }
     
    7275    static LayerPositionStrategy afterLast(final Predicate<Layer> what) {
    7376        return manager -> {
    74             List<Layer> layers = manager.getLayers();
    75             for (int i = layers.size() - 1; i >= 0; i--) {
    76                 if (what.test(layers.get(i))) {
    77                     return i + 1;
     77            if (manager != null) {
     78                List<Layer> layers = manager.getLayers();
     79                for (int i = layers.size() - 1; i >= 0; i--) {
     80                    if (what.test(layers.get(i))) {
     81                        return i + 1;
     82                    }
    7883                }
    7984            }
Note: See TracChangeset for help on using the changeset viewer.