Ticket #1663: lakewalker_exception_1663.patch

src/org/openstreetmap/josm/plugins/lakewalker/LakewalkerWMS.java

@@ -122 +122 @@
                         
                         // Read from a URL
                         URL url = new URL(urlloc);
-                        this.image = ImageIO.read(url);
+                        this.image = ImageIO.read(url); // this can return null!
                 } catch(MalformedURLException e){
                         System.out.println(e.getMessage());
                 } catch(IOException e){
@@ -131 +131 @@
                         System.out.println(e.getMessage());
                     }
                 
-                this.images.add(this.image);
-                this.imageindex.put(hashkey,this.images.size()-1);
-                
-                this.saveimage(file,this.image);
+                if (this.image != null) {
+                        this.images.add(this.image);
+                        this.imageindex.put(hashkey,this.images.size()-1);
+                        
+                        this.saveimage(file,this.image);
+                }
             }
             
             this.imagex = bottom_left_xy[0];

src/org/openstreetmap/josm/plugins/lakewalker/LakewalkerAction.java

@@ -176 +176 @@
         System.out.println("After removing duplicate nodes, "+nodelist.size()+" nodes remain.");
           
         
+        // if for some reason (image loading failed, ...) nodelist is empty, no more processing required.
+        if (nodelist.size() == 0) {
+                return;
+        }
+        
         /**
          * Turn the arraylist into osm nodes
          */