[Patch] Unnecessary JMapViewer Dependencies

[jhuntley]

Someone recently introduced org.openstreetmap.gui.jmapviewer.MapRectangleImpl.java which introduced a ton of cross dependencies with unrelated classes to the project. I'm now having to bring in the whole josm project just to use the JMapViewer. The culprit imported class is org.openstreetmap.josm.data.Bounds. It cross references classes and static functions even in JOSM's Main.java, literally turning the once small project into a bloated dependency of various packages and classes in JOSM. It makes it difficult for those wanting to use and contribute to just the JMapViewer, as it's not longer an independent component. What I'd like to request is to make this Bounds class a composite singular entity rather than making all the cross references. Otherwise, is it possible to remove the references to the Bounds class and replace it with something more simple? I'll gladly help to resolve this entanglement. Thanks,

[jhuntley]

Work around to this is just to remove the MapRectangleImpl.java. I can use the JMapViewer without this class.

[Don-vip]

I'm the one to blame for, sorry for that :( I've attached the patch that fixes this unintentional JOSM dependence.

[bastiK]

In [4438/josm]:

applied #6858 - Unnecessary JMapViewer Dependencies (patch by Don-vip)

[bastiK]

In [o26668].

[jhuntley]

Thanks Don :)