Changeset 5054 in josm


Ignore:
Timestamp:
Mar 8, 2012 12:11:38 PM (15 months ago)
Author:
bastiK
Message:

see #6797: load map images in background, in case they are loaded over a network. Show temporary image in the meantime.

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

Legend:

Unmodified
Added
Removed

  • trunk/src/org/openstreetmap/josm/data/osm/visitor/paint/MapPainter.java

    r5045 r5054  
    2222import java.awt.geom.Point2D; 
    2323import java.awt.geom.Rectangle2D; 
    24 import java.awt.image.BufferedImage; 
    2524import java.util.Arrays; 
    2625import java.util.Collection; 
    … …  
    738737         * 
    739738         */ 
     739        Rectangle box = bs.getBox(); 
    740740        if (bs.hAlign == HorizontalTextAlignment.RIGHT) { 
    741             x += bs.box.x + bs.box.width + 2; 
     741            x += box.x + box.width + 2; 
    742742        } else { 
    743743            FontRenderContext frc = g.getFontRenderContext(); 
    … …  
    747747                x -= textWidth / 2; 
    748748            } else if (bs.hAlign == HorizontalTextAlignment.LEFT) { 
    749                 x -= - bs.box.x + 4 + textWidth; 
     749                x -= - box.x + 4 + textWidth; 
    750750            } else throw new AssertionError(); 
    751751        } 
    752752 
    753753        if (bs.vAlign == VerticalTextAlignment.BOTTOM) { 
    754             y += bs.box.y + bs.box.height; 
     754            y += box.y + box.height; 
    755755        } else { 
    756756            FontRenderContext frc = g.getFontRenderContext(); 
    757757            LineMetrics metrics = text.font.getLineMetrics(s, frc); 
    758758            if (bs.vAlign == VerticalTextAlignment.ABOVE) { 
    759                 y -= - bs.box.y + metrics.getDescent(); 
     759                y -= - box.y + metrics.getDescent(); 
    760760            } else if (bs.vAlign == VerticalTextAlignment.TOP) { 
    761                 y -= - bs.box.y - metrics.getAscent(); 
     761                y -= - box.y - metrics.getAscent(); 
    762762            } else if (bs.vAlign == VerticalTextAlignment.CENTER) { 
    763763                y += (metrics.getAscent() - metrics.getDescent()) / 2; 
    764764            } else if (bs.vAlign == VerticalTextAlignment.BELOW) { 
    765                 y += bs.box.y + bs.box.height + metrics.getAscent() + 2; 
     765                y += box.y + box.height + metrics.getAscent() + 2; 
    766766            } else throw new AssertionError(); 
    767767        } 
    … …  
    803803    } 
    804804 
    805     public void drawArea(Way w, Color color, MapImage<BufferedImage> fillImage, TextElement text) { 
     805    public void drawArea(Way w, Color color, MapImage fillImage, TextElement text) { 
    806806        drawArea(w, getPath(w), color, fillImage, text); 
    807807    } 
    808808 
    809     protected void drawArea(OsmPrimitive osm, Path2D.Double path, Color color, MapImage<BufferedImage> fillImage, TextElement text) { 
     809    protected void drawArea(OsmPrimitive osm, Path2D.Double path, Color color, MapImage fillImage, TextElement text) { 
    810810 
    811811        Shape area = path.createTransformedShape(nc.getAffineTransform()); 
    … …  
    816816                g.fill(area); 
    817817            } else { 
    818                 TexturePaint texture = new TexturePaint(fillImage.img, 
     818                TexturePaint texture = new TexturePaint(fillImage.getImage(), 
    819819                        //                        new Rectangle(polygon.xpoints[0], polygon.ypoints[0], fillImage.getWidth(), fillImage.getHeight())); 
    820                         new Rectangle(0, 0, fillImage.img.getWidth(null), fillImage.img.getHeight(null))); 
     820                        new Rectangle(0, 0, fillImage.getWidth(), fillImage.getHeight())); 
    821821                g.setPaint(texture); 
    822822                Float alpha = Utils.color_int2float(fillImage.alpha); 
    … …  
    873873    } 
    874874 
    875     public void drawArea(Relation r, Color color, MapImage<BufferedImage> fillImage, TextElement text) { 
     875    public void drawArea(Relation r, Color color, MapImage fillImage, TextElement text) { 
    876876        Multipolygon multipolygon = MultipolygonCache.getInstance().get(nc, r); 
    877877        if (!r.isDisabled() && !multipolygon.getOuterWays().isEmpty()) { 
    … …  
    915915    } 
    916916 
    917     public void drawRestriction(Relation r, MapImage<Image> icon) { 
     917    public void drawRestriction(Relation r, MapImage icon) { 
    918918        Way fromWay = null; 
    919919        Way toWay = null; 
    … …  
    10861086        } 
    10871087 
    1088         drawRestriction(inactive || r.isDisabled() ? icon.getDisabled() : icon.img, 
     1088        drawRestriction(inactive || r.isDisabled() ? icon.getDisabled() : icon.getImage(), 
    10891089                pVia, vx, vx2, vy, vy2, iconAngle, r.isSelected()); 
    10901090    } 

  • trunk/src/org/openstreetmap/josm/gui/MapView.java

    r4627 r5054  
    460460     * Draw the component. 
    461461     */ 
    462     @Override public void paint(Graphics g) { 
     462    @Override public synchronized void paint(Graphics g) { 
    463463        if (BugReportExceptionHandler.exceptionHandlingInProgress()) 
    464464            return; 
    … …  
    833833    } 
    834834 
    835     public void preferenceChanged(PreferenceChangeEvent e) { 
     835    public synchronized void preferenceChanged(PreferenceChangeEvent e) { 
    836836        paintPreferencesChanged = true; 
    837837    } 

  • trunk/src/org/openstreetmap/josm/gui/mappaint/AreaElemStyle.java

    r4960 r5054  
    55 
    66import java.awt.Color; 
    7 import java.awt.image.BufferedImage; 
    8  
    9 import javax.swing.ImageIcon; 
    107 
    118import org.openstreetmap.josm.Main; 
    … …  
    2724     */ 
    2825    public Color color; 
    29     public MapImage<BufferedImage> fillImage; 
     26    public MapImage fillImage; 
    3027    public TextElement text; 
    3128 
    32     protected AreaElemStyle(Cascade c, Color color, MapImage<BufferedImage> fillImage, TextElement text) { 
     29    protected AreaElemStyle(Cascade c, Color color, MapImage fillImage, TextElement text) { 
    3330        super(c, -1000f); 
    3431        CheckParameterUtil.ensureParameterNotNull(color); 
    … …  
    3936 
    4037    public static AreaElemStyle create(Cascade c) { 
    41         MapImage<BufferedImage> fillImage = null; 
     38        MapImage fillImage = null; 
    4239        Color color = null; 
    4340 
    4441        IconReference iconRef = c.get("fill-image", null, IconReference.class); 
    4542        if (iconRef != null) { 
    46             ImageIcon icon = MapPaintStyles.getIcon(iconRef, -1, -1); 
    47             if (icon != null) { 
    48                 if (!(icon.getImage() instanceof BufferedImage)) 
    49                     throw new RuntimeException(); 
    50                 fillImage = new MapImage<BufferedImage>(iconRef.iconName, iconRef.source); 
    51                 fillImage.img = (BufferedImage) icon.getImage(); 
     43            fillImage = new MapImage(iconRef.iconName, iconRef.source); 
     44            fillImage.getImage(); 
    5245 
    53                 color = new Color(fillImage.img.getRGB( 
    54                         fillImage.img.getWidth() / 2, fillImage.img.getHeight() / 2) 
    55                 ); 
     46            color = new Color(fillImage.getImage().getRGB( 
     47                    fillImage.getWidth() / 2, fillImage.getHeight() / 2) 
     48            ); 
    5649 
    57                 fillImage.alpha = Math.min(255, Math.max(0, Integer.valueOf(Main.pref.getInteger("mappaint.fill-image-alpha", 255)))); 
    58                 Integer pAlpha = Utils.color_float2int(c.get("fill-opacity", null, float.class)); 
    59                 if (pAlpha != null) { 
    60                     fillImage.alpha = pAlpha; 
    61                 } 
     50            fillImage.alpha = Math.min(255, Math.max(0, Integer.valueOf(Main.pref.getInteger("mappaint.fill-image-alpha", 255)))); 
     51            Integer pAlpha = Utils.color_float2int(c.get("fill-opacity", null, float.class)); 
     52            if (pAlpha != null) { 
     53                fillImage.alpha = pAlpha; 
    6254            } 
    6355        } else { 
    … …  
    7870            text = TextElement.create(c, PaintColors.AREA_TEXT.get(), true); 
    7971        } 
    80          
     72 
    8173        if (color != null) 
    8274            return new AreaElemStyle(c, color, fillImage, text); 

  • trunk/src/org/openstreetmap/josm/gui/mappaint/BoxTextElemStyle.java

    r4869 r5054  
    2222    public enum VerticalTextAlignment { ABOVE, TOP, CENTER, BOTTOM, BELOW } 
    2323 
     24    public static interface BoxProvider { 
     25        BoxProviderResult get(); 
     26    } 
     27 
     28    public static class BoxProviderResult { 
     29        private Rectangle box; 
     30        private boolean temporary; 
     31 
     32        public BoxProviderResult(Rectangle box, boolean temporary) { 
     33            this.box = box; 
     34            this.temporary = temporary; 
     35        } 
     36 
     37        /** 
     38         * The box 
     39         */ 
     40        public Rectangle getBox() { 
     41            return box; 
     42        } 
     43 
     44        /** 
     45         * True, if the box can change in future calls of the BoxProvider get() method 
     46         */ 
     47        public boolean isTemporary() { 
     48            return temporary; 
     49        } 
     50    } 
     51 
     52    public static class SimpleBoxProvider implements BoxProvider { 
     53        private Rectangle box; 
     54 
     55        public SimpleBoxProvider(Rectangle box) { 
     56            this.box = box; 
     57        } 
     58 
     59        @Override 
     60        public BoxProviderResult get() { 
     61            return new BoxProviderResult(box, false); 
     62        } 
     63 
     64        @Override 
     65        public int hashCode() { 
     66            return box.hashCode(); 
     67        } 
     68 
     69        @Override 
     70        public boolean equals(Object obj) { 
     71            if (obj == null || !(obj instanceof BoxProvider)) 
     72                return false; 
     73            final BoxProvider other = (BoxProvider) obj; 
     74            BoxProviderResult resultOther = other.get(); 
     75            if (resultOther.isTemporary()) return false; 
     76            return box.equals(resultOther.getBox()); 
     77        } 
     78    } 
     79 
    2480    public static final Rectangle ZERO_BOX = new Rectangle(0, 0, 0, 0); 
    2581 
    2682    public TextElement text; 
    27     public Rectangle box; 
     83    // Either boxProvider or box is not null. If boxProvider is different from 
     84    // null, this means, that the box can still change in future, otherwise 
     85    // it is fixed. 
     86    protected BoxProvider boxProvider; 
     87    protected Rectangle box; 
    2888    public HorizontalTextAlignment hAlign; 
    2989    public VerticalTextAlignment vAlign; 
    3090 
    31     public BoxTextElemStyle(Cascade c, TextElement text, Rectangle box, HorizontalTextAlignment hAlign, VerticalTextAlignment vAlign) { 
     91    public BoxTextElemStyle(Cascade c, TextElement text, BoxProvider boxProvider, Rectangle box, HorizontalTextAlignment hAlign, VerticalTextAlignment vAlign) { 
    3292        super(c, 2000f); 
    3393        CheckParameterUtil.ensureParameterNotNull(text); 
    … …  
    3595        CheckParameterUtil.ensureParameterNotNull(vAlign); 
    3696        this.text = text; 
     97        this.boxProvider = boxProvider; 
    3798        this.box = box == null ? ZERO_BOX : box; 
    3899        this.hAlign = hAlign; 
    … …  
    40101    } 
    41102 
     103    public static BoxTextElemStyle create(Environment env, BoxProvider boxProvider) { 
     104        return create(env, boxProvider, null); 
     105    } 
     106 
    42107    public static BoxTextElemStyle create(Environment env, Rectangle box) { 
     108        return create(env, null, box); 
     109    } 
     110 
     111    public static BoxTextElemStyle create(Environment env, BoxProvider boxProvider, Rectangle box) { 
    43112        initDefaultParameters(); 
    44113        Cascade c = env.mc.getCascade(env.layer); 
    … …  
    74143        } 
    75144 
    76         return new BoxTextElemStyle(c, text, box, hAlign, vAlign); 
     145        return new BoxTextElemStyle(c, text, boxProvider, box, hAlign, vAlign); 
     146    } 
     147 
     148    public Rectangle getBox() { 
     149        if (boxProvider != null) { 
     150            BoxProviderResult result = boxProvider.get(); 
     151            if (!result.isTemporary()) { 
     152                box = result.getBox(); 
     153                boxProvider = null; 
     154            } 
     155            return result.getBox(); 
     156        } 
     157        return box; 
    77158    } 
    78159 
    … …  
    84165        Node n = new Node(); 
    85166        n.put("name", "dummy"); 
    86         SIMPLE_NODE_TEXT_ELEMSTYLE = create(new Environment(n, mc, "default", null), NodeElemStyle.SIMPLE_NODE_ELEMSTYLE.getBox()); 
     167        SIMPLE_NODE_TEXT_ELEMSTYLE = create(new Environment(n, mc, "default", null), NodeElemStyle.SIMPLE_NODE_ELEMSTYLE.getBoxProvider()); 
    87168        if (SIMPLE_NODE_TEXT_ELEMSTYLE == null) throw new AssertionError(); 
    88169    } 
    … …  
    113194            return false; 
    114195        final BoxTextElemStyle other = (BoxTextElemStyle) obj; 
    115         return text.equals(other.text) && 
    116                 box.equals(other.box) && 
    117                 hAlign == other.hAlign && 
    118                 vAlign == other.vAlign; 
     196        if (!text.equals(other.text)) return false; 
     197        if (boxProvider != null) { 
     198            if (!boxProvider.equals(other.boxProvider)) return false; 
     199        } else if (other.boxProvider != null) { 
     200            return false; 
     201        } else { 
     202            if (!box.equals(other.box)) return false; 
     203        } 
     204        if (hAlign != other.hAlign) return false; 
     205        if (vAlign != other.vAlign) return false; 
     206        return true; 
    119207    } 
    120208 
    … …  
    123211        int hash = super.hashCode(); 
    124212        hash = 97 * hash + text.hashCode(); 
    125         hash = 97 * hash + box.hashCode(); 
     213        if (boxProvider != null) { 
     214            hash = 97 * hash + boxProvider.hashCode(); 
     215        } else { 
     216            hash = 97 * hash + box.hashCode(); 
     217        } 
    126218        hash = 97 * hash + hAlign.hashCode(); 
    127219        hash = 97 * hash + vAlign.hashCode(); 

  • trunk/src/org/openstreetmap/josm/gui/mappaint/ElemStyles.java

    r4682 r5054  
    2929    private boolean defaultNodes, defaultLines; 
    3030    private int defaultNodesIdx, defaultLinesIdx; 
    31      
     31 
    3232    public ElemStyles() 
    3333    { 
    … …  
    317317                if (nodeStyle != null) { 
    318318                    sl.add(nodeStyle); 
    319                     addIfNotNull(sl, BoxTextElemStyle.create(env, nodeStyle.getBox())); 
     319                    addIfNotNull(sl, BoxTextElemStyle.create(env, nodeStyle.getBoxProvider())); 
    320320                } else { 
    321                     addIfNotNull(sl, BoxTextElemStyle.create(env, NodeElemStyle.SIMPLE_NODE_ELEMSTYLE.getBox())); 
     321                    addIfNotNull(sl, BoxTextElemStyle.create(env, NodeElemStyle.SIMPLE_NODE_ELEMSTYLE.getBoxProvider())); 
    322322                } 
    323323            } else if (osm instanceof Relation) { 

  • trunk/src/org/openstreetmap/josm/gui/mappaint/LinePatternElemStyle.java

    r4822 r5054  
    11// License: GPL. For details, see LICENSE file. 
    22package org.openstreetmap.josm.gui.mappaint; 
    3  
    4 import java.awt.Image; 
    5  
    6 import javax.swing.ImageIcon; 
    73 
    84import org.openstreetmap.josm.data.osm.OsmPrimitive; 
    … …  
    1713public class LinePatternElemStyle extends ElemStyle { 
    1814 
    19     public MapImage<Image> pattern; 
     15    public MapImage pattern; 
    2016 
    21     public LinePatternElemStyle(Cascade c, MapImage<Image> pattern) { 
     17    public LinePatternElemStyle(Cascade c, MapImage pattern) { 
    2218        super(c, -1f); 
    2319        this.pattern = pattern; 
    … …  
    3026        if (iconRef == null) 
    3127            return null; 
    32         ImageIcon icon = MapPaintStyles.getIcon(iconRef, -1, -1); 
    33         if (icon == null) 
    34             return null; 
    35         MapImage<Image> pattern = new MapImage<Image>(iconRef.iconName, iconRef.source); 
    36         pattern.img = icon.getImage(); 
     28        MapImage pattern = new MapImage(iconRef.iconName, iconRef.source); 
    3729        return new LinePatternElemStyle(c, pattern); 
    3830    } 
    … …  
    4133    public void paintPrimitive(OsmPrimitive primitive, MapPaintSettings paintSettings, MapPainter painter, boolean selected, boolean member) { 
    4234        Way w = (Way)primitive; 
    43         painter.drawLinePattern(w, pattern.img); 
     35        painter.drawLinePattern(w, pattern.getImage()); 
    4436    } 
    4537 

  • trunk/src/org/openstreetmap/josm/gui/mappaint/MapImage.java

    r4823 r5054  
    33 
    44import static org.openstreetmap.josm.tools.Utils.equal; 
     5 
    56import java.awt.Image; 
     7import java.awt.Rectangle; 
     8import java.awt.image.BufferedImage; 
    69 
    710import javax.swing.GrayFilter; 
    811import javax.swing.ImageIcon; 
    912 
    10 public class MapImage<I extends Image> { 
     13import org.openstreetmap.josm.Main; 
     14import org.openstreetmap.josm.gui.mappaint.BoxTextElemStyle.BoxProvider; 
     15import org.openstreetmap.josm.gui.mappaint.BoxTextElemStyle.BoxProviderResult; 
     16import org.openstreetmap.josm.tools.ImageProvider; 
     17import org.openstreetmap.josm.tools.ImageProvider.ImageCallback; 
     18import org.openstreetmap.josm.tools.Utils; 
     19 
     20public class MapImage { 
    1121    /** 
    1222     * ImageIcon can chage while the image is loading. 
    1323     */ 
    14     public I img; 
    15  
    16     public int alpha = 255; 
     24    private BufferedImage img; 
    1725 
    1826    /** 
    19      * The 4 following fields are only used to check for equality. 
     27     * The 5 following fields are only used to check for equality. 
    2028     */ 
     29    public int alpha = 255; 
    2130    public String name; 
    2231    public StyleSource source; 
    … …  
    2433    public int height = -1; 
    2534 
     35    private boolean temporary; 
    2636    private Image disabledImg; 
    2737 
    … …  
    3747    } 
    3848 
    39     // img changes when image is fully loaded and can't be used for equality check. 
     49    public BufferedImage getImage() { 
     50        if (img != null) 
     51            return img; 
     52        temporary = false; 
     53        new ImageProvider(name) 
     54                .setDirs(MapPaintStyles.getIconSourceDirs(source)) 
     55                .setId("mappaint."+source.getPrefName()) 
     56                .setArchive(source.zipIcons) 
     57                .setWidth(width) 
     58                .setHeight(height) 
     59                .setOptional(true) 
     60                .getInBackground(new ImageCallback() { 
     61                    @Override 
     62                    public void finished(ImageIcon result) { 
     63                        synchronized (MapImage.this) { 
     64                            if (result == null) { 
     65                                img = (BufferedImage) MapPaintStyles.getNoIcon_Icon(source).getImage(); 
     66                            } else { 
     67                                img = (BufferedImage) result.getImage(); 
     68                            } 
     69                            if (temporary) { 
     70                                Main.map.mapView.preferenceChanged(null); // otherwise repaint is ignored, because layer hasn't changed 
     71                                Main.map.mapView.repaint(); 
     72                            } 
     73                            temporary = false; 
     74                        } 
     75                    } 
     76                } 
     77        ); 
     78        synchronized (this) { 
     79            if (img == null) { 
     80                img = (BufferedImage) ImageProvider.get("clock").getImage(); 
     81                temporary = true; 
     82            } 
     83        } 
     84        return img; 
     85    } 
     86 
     87    public int getWidth() { 
     88        return getImage().getWidth(null); 
     89    } 
     90 
     91    public int getHeight() { 
     92        return getImage().getHeight(null); 
     93    } 
     94 
     95    public float getAlphaFloat() { 
     96        return Utils.color_int2float(alpha); 
     97    } 
     98 
     99    /** 
     100     * Returns true, if image is not completely loaded and getImage() returns a temporary image. 
     101     */ 
     102    public boolean isTemporary() { 
     103        return temporary; 
     104    } 
     105 
     106    protected class MapImageBoxProvider implements BoxProvider { 
     107        @Override 
     108        public BoxProviderResult get() { 
     109            return new BoxProviderResult(box(), temporary); 
     110        } 
     111 
     112        private Rectangle box() { 
     113            int w = getWidth(), h = getHeight(); 
     114            return new Rectangle(-w/2, -h/2, w, h); 
     115        } 
     116 
     117        private MapImage getParent() { 
     118            return MapImage.this; 
     119        } 
     120 
     121        @Override 
     122        public int hashCode() { 
     123            return MapImage.this.hashCode(); 
     124        } 
     125 
     126        @Override 
     127        public boolean equals(Object obj) { 
     128            if (obj == null || !(obj instanceof BoxProvider)) 
     129                return false; 
     130            if (obj instanceof MapImageBoxProvider) { 
     131                MapImageBoxProvider other = (MapImageBoxProvider) obj; 
     132                return MapImage.this.equals(other.getParent()); 
     133            } else if (temporary) { 
     134                return false; 
     135            } else { 
     136                final BoxProvider other = (BoxProvider) obj; 
     137                BoxProviderResult resultOther = other.get(); 
     138                if (resultOther.isTemporary()) return false; 
     139                return box().equals(resultOther.getBox()); 
     140            } 
     141        } 
     142    } 
     143 
     144    public BoxProvider getBoxProvider() { 
     145        return new MapImageBoxProvider(); 
     146    } 
     147 
    40148    @Override 
    41149    public boolean equals(Object obj) { 
    … …  
    43151            return false; 
    44152        final MapImage other = (MapImage) obj; 
     153        // img changes when image is fully loaded and can't be used for equality check. 
    45154        return  alpha == other.alpha && 
    46155                equal(name, other.name) && 

  • trunk/src/org/openstreetmap/josm/gui/mappaint/MapPaintStyles.java

    r4968 r5054  
    119119    } 
    120120 
    121     private static List<String> getIconSourceDirs(StyleSource source) { 
     121    public static List<String> getIconSourceDirs(StyleSource source) { 
    122122        List<String> dirs = new LinkedList<String>(); 
    123123 

  • trunk/src/org/openstreetmap/josm/gui/mappaint/NodeElemStyle.java

    r4822 r5054  
    66import java.awt.BasicStroke; 
    77import java.awt.Color; 
    8 import java.awt.Image; 
    98import java.awt.Rectangle; 
    109import java.awt.Stroke; 
    11  
    12 import javax.swing.ImageIcon; 
    1310 
    1411import org.openstreetmap.josm.Main; 
    … …  
    2017import org.openstreetmap.josm.gui.mappaint.MapPaintStyles.IconReference; 
    2118import org.openstreetmap.josm.gui.mappaint.StyleCache.StyleList; 
     19import org.openstreetmap.josm.gui.mappaint.BoxTextElemStyle.BoxProvider; 
     20import org.openstreetmap.josm.gui.mappaint.BoxTextElemStyle.SimpleBoxProvider; 
    2221import org.openstreetmap.josm.tools.Pair; 
    2322import org.openstreetmap.josm.tools.Utils; 
    … …  
    2726 */ 
    2827public class NodeElemStyle extends ElemStyle { 
    29     public MapImage<Image> mapImage; 
     28    public MapImage mapImage; 
    3029    public Symbol symbol; 
    31  
    32     private ImageIcon disabledIcon; 
    3330 
    3431    public enum SymbolShape { SQUARE, CIRCLE, TRIANGLE, PENTAGON, HEXAGON, HEPTAGON, OCTAGON, NONAGON, DECAGON } 
    … …  
    9592    public static final StyleList DEFAULT_NODE_STYLELIST_TEXT = new StyleList(NodeElemStyle.SIMPLE_NODE_ELEMSTYLE, BoxTextElemStyle.SIMPLE_NODE_TEXT_ELEMSTYLE); 
    9693 
    97     protected NodeElemStyle(Cascade c, MapImage<Image> mapImage, Symbol symbol) { 
     94    protected NodeElemStyle(Cascade c, MapImage mapImage, Symbol symbol) { 
    9895        super(c, 1000f); 
    9996        this.mapImage = mapImage; 
    … …  
    108105        Cascade c = env.mc.getCascade(env.layer); 
    109106 
    110         MapImage<Image> mapImage = createIcon(env); 
     107        MapImage mapImage = createIcon(env); 
    111108        Symbol symbol = null; 
    112109        if (mapImage == null) { 
    … …  
    122119    } 
    123120 
    124     private static MapImage<Image> createIcon(Environment env) { 
     121    private static MapImage createIcon(Environment env) { 
    125122        Cascade c = env.mc.getCascade(env.layer); 
    126123        Cascade c_def = env.mc.getCascade("default"); 
    127124 
    128         IconReference iconRef = c.get("icon-image", null, IconReference.class); 
     125        final IconReference iconRef = c.get("icon-image", null, IconReference.class); 
    129126        if (iconRef == null) 
    130127            return null; 
    … …  
    145142        int height = heightF == null ? -1 : Math.round(heightF); 
    146143 
    147         MapImage<Image> mapImage = new MapImage<Image>(iconRef.iconName, iconRef.source); 
    148  
    149         ImageIcon icon = MapPaintStyles.getIcon(iconRef, width, height); 
    150         if (icon == null) { 
    151             mapImage.img = MapPaintStyles.getNoIcon_Icon(iconRef.source).getImage(); 
    152         } else { 
    153             mapImage.img = icon.getImage(); 
    154             mapImage.alpha = Math.min(255, Math.max(0, Integer.valueOf(Main.pref.getInteger("mappaint.icon-image-alpha", 255)))); 
    155             Integer pAlpha = Utils.color_float2int(c.get("icon-opacity", null, float.class)); 
    156             if (pAlpha != null) { 
    157                 mapImage.alpha = pAlpha; 
    158             } 
    159             mapImage.width = width; 
    160             mapImage.height = height; 
     144        final MapImage mapImage = new MapImage(iconRef.iconName, iconRef.source); 
     145 
     146        mapImage.width = width; 
     147        mapImage.height = height; 
     148 
     149        mapImage.alpha = Math.min(255, Math.max(0, Integer.valueOf(Main.pref.getInteger("mappaint.icon-image-alpha", 255)))); 
     150        Integer pAlpha = Utils.color_float2int(c.get("icon-opacity", null, float.class)); 
     151        if (pAlpha != null) { 
     152            mapImage.alpha = pAlpha; 
    161153        } 
    162154        return mapImage; 
    … …  
    243235            Node n = (Node) primitive; 
    244236            if (mapImage != null && painter.isShowIcons()) { 
    245                 painter.drawNodeIcon(n, (painter.isInactiveMode() || n.isDisabled()) ? mapImage.getDisabled() : mapImage.img, 
     237                painter.drawNodeIcon(n, (painter.isInactiveMode() || n.isDisabled()) ? mapImage.getDisabled() : mapImage.getImage(), 
    246238                        Utils.color_int2float(mapImage.alpha), selected, member); 
    247239            } else if (symbol != null) { 
    … …  
    309301    } 
    310302 
    311     public Rectangle getBox() { 
     303    public BoxProvider getBoxProvider() { 
    312304        if (mapImage != null) { 
    313             int w = mapImage.img.getWidth(null), h = mapImage.img.getHeight(null); 
    314             return new Rectangle(-w/2, -h/2, w, h); 
     305            return mapImage.getBoxProvider(); 
    315306        } else if (symbol != null) { 
    316             return new Rectangle(-symbol.size/2, -symbol.size/2, symbol.size, symbol.size); 
     307            return new SimpleBoxProvider(new Rectangle(-symbol.size/2, -symbol.size/2, symbol.size, symbol.size)); 
    317308        } else { 
    318309            // This is only executed once, so no performance concerns. 
    … …  
    324315                    Main.pref.getInteger("mappaint.node.tagged-size", 3) 
    325316            ); 
    326             return new Rectangle(-size/2, -size/2, size, size); 
     317            return new SimpleBoxProvider(new Rectangle(-size/2, -size/2, size, size)); 
    327318        } 
    328319    } 

  • trunk/src/org/openstreetmap/josm/tools/ImageProvider.java

    r5033 r5054  
    3232import java.util.HashMap; 
    3333import java.util.Map; 
     34import java.util.concurrent.Executors; 
     35import java.util.concurrent.ExecutorService; 
    3436import java.util.regex.Matcher; 
    3537import java.util.regex.Pattern; 
    … …  
    104106    private static Map<String, ImageResource> cache = new HashMap<String, ImageResource>(); 
    105107 
     108    private final static ExecutorService imageFetcher = Executors.newSingleThreadExecutor(); 
     109 
     110    public interface ImageCallback { 
     111        void finished(ImageIcon result); 
     112    } 
     113 
    106114    /** 
    107115     * @param subdir    Subdirectory the image lies in. 
    … …  
    256264        else 
    257265            return ir.getImageIcon(new Dimension(width, height)); 
     266    } 
     267 
     268    /** 
     269     * Load the image in a background thread. 
     270     * 
     271     * This method returns immediately and runs the image request 
     272     * asynchronously. 
     273     * 
     274     * @param callback is called, when the image is ready. This can happen 
     275     * before the call to getInBackground returns or it may be invoked some 
     276     * time (seconds) later. 
     277     * If no image is available, a null value is returned to callback (just 
     278     * like ImageProvider.get). 
     279     */ 
     280    public void getInBackground(final ImageCallback callback) { 
     281        if (name.startsWith("http://") || name.startsWith("wiki://")) { 
     282            Runnable fetch = new Runnable() { 
     283                @Override 
     284                public void run() { 
     285                    ImageIcon result = get(); 
     286                    callback.finished(result); 
     287                } 
     288            }; 
     289            imageFetcher.submit(fetch); 
     290        } else { 
     291            ImageIcon result = get(); 
     292            callback.finished(result); 
     293        } 
    258294    } 
    259295 
Note: See TracChangeset for help on using the changeset viewer.