Changeset 12170 in josm


Ignore:
Timestamp:
2017-05-15T17:43:48+02:00 (6 months ago)
Author:
michael2402
Message:

"See #13175, see #14120: Remove isChanged() method call in map view - all layers should invalidate correctly now."

Location:
trunk/src/org/openstreetmap/josm/gui
Files:
3 edited

Legend:

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

    r12118 r12170  
    487487        Set<MapViewPaintable> invalidated = invalidatedListener.collectInvalidatedLayers();
    488488        for (Layer l: visibleLayers) {
    489             // `isChanged` for backward compatibility, see https://josm.openstreetmap.de/ticket/13175#comment:7
    490             // Layers that still implement it (plugins) will use it to tell the MapView that they have been changed.
    491             // This is why the MapView still uses it in addition to the invalidation events.
    492             if (l.isChanged() || invalidated.contains(l)) {
     489            if (invalidated.contains(l)) {
    493490                break;
    494491            } else {
  • trunk/src/org/openstreetmap/josm/gui/layer/AbstractTileSourceLayer.java

    r12093 r12170  
    16701670    }
    16711671
    1672     @Override
    1673     public boolean isChanged() {
    1674         return false; // we use #invalidate()
    1675     }
    1676 
    16771672    /**
    16781673     * Task responsible for precaching imagery along the gpx track
  • trunk/src/org/openstreetmap/josm/gui/layer/Layer.java

    r12114 r12170  
    507507     * @deprecated This is not supported by multiple map views.
    508508     * Fire an {@link #invalidate()} to trigger a repaint.
    509      * Let this method return false if you only use invalidation events.
    510509     */
    511510    @Deprecated
    512511    public boolean isChanged() {
    513         return true;
     512        return false;
    514513    }
    515514
Note: See TracChangeset for help on using the changeset viewer.