[jmapviewer] 04/05: Drop patches included upstream. Refresh remaining patches.

Bas Couwenberg sebastic at debian.org
Mon Nov 27 09:03:08 UTC 2017


This is an automated email from the git hooks/post-receive script.

sebastic pushed a commit to branch master
in repository jmapviewer.

commit 5156c3c62306c1da8569401eb3d4cbec6111567d
Author: Bas Couwenberg <sebastic at xs4all.nl>
Date:   Mon Nov 27 08:28:38 2017 +0100

    Drop patches included upstream. Refresh remaining patches.
---
 debian/changelog                                   |  2 +
 debian/patches/01-build_less.patch                 |  2 +-
 debian/patches/series                              |  2 -
 ...33762-use-ArrayList-instead-of-LinkedList.patch | 56 ----------------------
 ...a-new-method-to-ease-asynchronous-testing.patch | 37 --------------
 5 files changed, 3 insertions(+), 96 deletions(-)

diff --git a/debian/changelog b/debian/changelog
index 531f958..25379e5 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,6 +1,8 @@
 jmapviewer (2.4+dfsg-1) UNRELEASED; urgency=medium
 
   * New upstream release.
+  * Drop patches included upstream.
+    Refresh remaining patches.
 
  -- Bas Couwenberg <sebastic at debian.org>  Mon, 27 Nov 2017 08:26:39 +0100
 
diff --git a/debian/patches/01-build_less.patch b/debian/patches/01-build_less.patch
index 59886b6..7d75890 100644
--- a/debian/patches/01-build_less.patch
+++ b/debian/patches/01-build_less.patch
@@ -12,7 +12,7 @@ this Debian package.
  <?xml version="1.0" encoding="UTF-8" standalone="no"?>
  <project default="all" name="Compile and build java classes plus jar archives">
  
--	<target name="all" depends="clean,build,svn_info,pack,create_run_jar,findbugs,checkstyle,javadoc,create_release_zip,create_source_release_zip" />
+-	<target name="all" depends="clean,build,svn_info,pack,create_run_jar,spotbugs,checkstyle,javadoc,create_release_zip,create_source_release_zip" />
 +	<target name="all" depends="clean,build,svn_info,pack,create_run_jar" />
  
  	<target name="clean">
diff --git a/debian/patches/series b/debian/patches/series
index 64dad1b..32e8c24 100644
--- a/debian/patches/series
+++ b/debian/patches/series
@@ -1,4 +1,2 @@
 01-build_less.patch
 03-use-installed-library-in-demo.patch
-svn-r33762-use-ArrayList-instead-of-LinkedList.patch
-svn-r33804-add-a-new-method-to-ease-asynchronous-testing.patch
diff --git a/debian/patches/svn-r33762-use-ArrayList-instead-of-LinkedList.patch b/debian/patches/svn-r33762-use-ArrayList-instead-of-LinkedList.patch
deleted file mode 100644
index af4cff8..0000000
--- a/debian/patches/svn-r33762-use-ArrayList-instead-of-LinkedList.patch
+++ /dev/null
@@ -1,56 +0,0 @@
-Description: http://errorprone.info/bugpattern/JdkObsolete: use ArrayList instead of LinkedList
-Author: donvip
-Origin: https://trac.openstreetmap.org/changeset/33762/subversion
-
---- a/src/org/openstreetmap/gui/jmapviewer/JMapViewer.java
-+++ b/src/org/openstreetmap/gui/jmapviewer/JMapViewer.java
-@@ -10,8 +10,8 @@ import java.awt.event.ActionEvent;
- import java.awt.event.ActionListener;
- import java.awt.event.MouseEvent;
- import java.net.URL;
-+import java.util.ArrayList;
- import java.util.Collections;
--import java.util.LinkedList;
- import java.util.List;
- 
- import javax.swing.ImageIcon;
-@@ -139,9 +139,9 @@ public class JMapViewer extends JPanel i
-     public JMapViewer(TileCache tileCache) {
-         tileSource = new OsmTileSource.Mapnik();
-         tileController = new TileController(tileSource, tileCache, this);
--        mapMarkerList = Collections.synchronizedList(new LinkedList<MapMarker>());
--        mapPolygonList = Collections.synchronizedList(new LinkedList<MapPolygon>());
--        mapRectangleList = Collections.synchronizedList(new LinkedList<MapRectangle>());
-+        mapMarkerList = Collections.synchronizedList(new ArrayList<MapMarker>());
-+        mapPolygonList = Collections.synchronizedList(new ArrayList<MapPolygon>());
-+        mapRectangleList = Collections.synchronizedList(new ArrayList<MapRectangle>());
-         mapMarkersVisible = true;
-         mapRectanglesVisible = true;
-         mapPolygonsVisible = true;
-@@ -769,7 +769,7 @@ public class JMapViewer extends JPanel i
-     protected void paintPolygon(Graphics g, MapPolygon polygon) {
-         List<? extends ICoordinate> coords = polygon.getPoints();
-         if (coords != null && coords.size() >= 3) {
--            List<Point> points = new LinkedList<>();
-+            List<Point> points = new ArrayList<>();
-             for (ICoordinate c : coords) {
-                 Point p = getMapPosition(c, false);
-                 if (p == null) {
-@@ -781,7 +781,7 @@ public class JMapViewer extends JPanel i
-             if (scrollWrapEnabled) {
-                 int tilesize = tileSource.getTileSize();
-                 int mapSize = tilesize << zoom;
--                List<Point> pointsWrapped = new LinkedList<>(points);
-+                List<Point> pointsWrapped = new ArrayList<>(points);
-                 boolean keepWrapping = true;
-                 while (keepWrapping) {
-                     for (Point p : pointsWrapped) {
-@@ -792,7 +792,7 @@ public class JMapViewer extends JPanel i
-                     }
-                     polygon.paint(g, pointsWrapped);
-                 }
--                pointsWrapped = new LinkedList<>(points);
-+                pointsWrapped = new ArrayList<>(points);
-                 keepWrapping = true;
-                 while (keepWrapping) {
-                     for (Point p : pointsWrapped) {
diff --git a/debian/patches/svn-r33804-add-a-new-method-to-ease-asynchronous-testing.patch b/debian/patches/svn-r33804-add-a-new-method-to-ease-asynchronous-testing.patch
deleted file mode 100644
index 135f971..0000000
--- a/debian/patches/svn-r33804-add-a-new-method-to-ease-asynchronous-testing.patch
+++ /dev/null
@@ -1,37 +0,0 @@
-Description: add a new method to ease asynchronous testing (patch by ris)
-Author: ris
-Origin: https://trac.openstreetmap.org/changeset/33804/subversion
-Bug: http://josm.openstreetmap.de/intertrac/ticket:15539
-
---- a/src/org/openstreetmap/gui/jmapviewer/OsmTileLoader.java
-+++ b/src/org/openstreetmap/gui/jmapviewer/OsmTileLoader.java
-@@ -177,6 +177,11 @@ public class OsmTileLoader implements Ti
-     }
- 
-     @Override
-+    public boolean hasOutstandingTasks() {
-+        return jobDispatcher.getTaskCount() > jobDispatcher.getCompletedTaskCount();
-+    }
-+
-+    @Override
-     public void cancelOutstandingTasks() {
-         jobDispatcher.getQueue().clear();
-     }
---- a/src/org/openstreetmap/gui/jmapviewer/interfaces/TileLoader.java
-+++ b/src/org/openstreetmap/gui/jmapviewer/interfaces/TileLoader.java
-@@ -26,4 +26,15 @@ public interface TileLoader {
-      * to loading = false / loaded = false
-      */
-     void cancelOutstandingTasks();
-+
-+    /**
-+     * Determines whether this {@link TileLoader} has tasks which have not completed.
-+     * @return whether this {@link TileLoader} has tasks which have not completed. This answer may well be
-+     * "approximate" given that many implementations will be using mechanisms where a queue's state can change
-+     * during the computation.
-+     */
-+    default boolean hasOutstandingTasks() {
-+        // default implementation supplied just to make transition easier for external implementors
-+        throw new UnsupportedOperationException("Not implemented");
-+    }
- }

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/pkg-grass/jmapviewer.git



More information about the Pkg-grass-devel mailing list