fix docs

vtm/src/org/oscim/event/EventDispatcher.java

@@ -26,7 +26,7 @@ public class EventDispatcher<T, E> {
 		 * 
 		 * @param source the event source
 		 * @param event the event object/type
-		 * @param data, some object involved in this event
+		 * @param data some object involved in this event
 		 */
 		public void onEvent(Object source, Event event, E data);
 	}

vtm/src/org/oscim/layers/MapEventLayer.java

@@ -28,15 +28,8 @@ import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
 /**
- * Changes Viewport for scroll, fling, scale, rotation and tilt gestures
+ * Changes Viewport for move, fling, scale, rotation and tilt gestures.
- * 
- * @TODO:
- *        - better recognition of tilt/rotate/scale state
- *        one could check change of rotation / scale within a
- *        given time to estimate if the mode should be changed:
- *        http://en.wikipedia.org/wiki/Viterbi_algorithm
  */
-
 public class MapEventLayer extends Layer implements Map.InputListener, GestureListener {
 
 	static final Logger log = LoggerFactory.getLogger(MapEventLayer.class);

vtm/src/org/oscim/layers/marker/ItemizedLayer.java

@@ -277,15 +277,14 @@ public abstract class ItemizedLayer<Item extends MarkerItem> extends MarkerLayer
 	}
 
 	/**
+	 * TODO
 	 * If the given Item is found in the overlay, force it to be the current
-	 * focus-bearer. Any registered {@@link
+	 * focus-bearer. Any registered {link ItemizedLayer#OnFocusChangeListener}
-	 * ItemizedOverlay#OnFocusChangeListener} will be notified. This does not
+	 * will be notified. This does not move the map, so if the Item isn't
-	 * move the map, so if the Item isn't already centered, the user may get
+	 * already centered, the user may get confused. If the Item is not found,
-	 * confused. If the Item is not found, this is a no-op. You can also pass
+	 * this is a no-op. You can also pass null to remove focus.
-	 * null to remove focus.
 	 * 
 	 * @param item
-	 *            ...
 	 */
 	public void setFocus(final Item item) {
 		mFocusedItem = item;

vtm/src/org/oscim/map/Map.java

@@ -209,7 +209,7 @@ public abstract class Map {
 	/**
 	 * Get current {@link MapPosition}.
 	 * 
-	 * @param mapPosition
+	 * @return true when MapPosition was updated (has changed)
 	 */
 	public boolean getMapPosition(MapPosition mapPosition) {
 		return mViewport.getMapPosition(mapPosition);

vtm/src/org/oscim/renderer/LayerRenderer.java

@@ -37,7 +37,7 @@ public abstract class LayerRenderer {
 	/**
 	 * Set 'ready for render' state when layer data is ready for rendering.
 	 * 
-	 * @param true if render() should be called, false otherwise.
+	 * @param ready true if render() should be called, false otherwise.
 	 */
 	protected void setReady(boolean ready) {
 		isReady = ready;

vtm/src/org/oscim/theme/RenderThemeHandler.java

@@ -23,8 +23,6 @@ import java.util.ArrayList;
 import java.util.HashMap;
 import java.util.Stack;
 
-import javax.xml.parsers.ParserConfigurationException;
-
 import org.oscim.backend.CanvasAdapter;
 import org.oscim.backend.XMLReaderAdapter;
 import org.oscim.backend.canvas.Bitmap;
@@ -84,8 +82,6 @@ public class RenderThemeHandler extends DefaultHandler {
 	 *         the input stream.
 	 * @throws SAXException
 	 *             if an error occurs while parsing the render theme XML.
-	 * @throws ParserConfigurationException
-	 *             if an error occurs while creating the XML parser.
 	 * @throws IOException
 	 *             if an I/O error occurs while reading from the input stream.
 	 */

vtm/src/org/oscim/tiling/TileManager.java

@@ -333,9 +333,7 @@ public class TileManager {
 	}
 
 	/**
-	 * Unlock tiles and clear all item references.
+	 * Unlock TileSet and clear all item references.
-	 * 
-	 * @param tiles
 	 */
 	public void releaseTiles(TileSet tileSet) {
 		tileSet.releaseTiles();
@@ -508,7 +506,6 @@ public class TileManager {
 	 * 
 	 * @param tile
 	 *            Tile ready for upload in TileRenderLayer
-	 * @return caller does not care
 	 */
 	public void jobCompleted(final MapTile tile, final boolean success) {
 		mMap.post(new Runnable() {

vtm/src/org/oscim/tiling/source/common/LwHttp.java

@@ -67,12 +67,6 @@ public class LwHttp {
 	/**
 	 * @param url
 	 *            Base url for tiles
-	 * @param contentType
-	 *            Expected Content-Type
-	 * @param extension
-	 *            'file' extension, usually .png
-	 * @param deflate
-	 *            true when content uses gzip compression
 	 */
 	public LwHttp(URL url) {
 

vtm/src/org/oscim/tiling/source/common/UrlTileSource.java

@@ -78,9 +78,9 @@ public abstract class UrlTileSource extends TileSource {
 	 * override getTileUrl() for custom url formatting using
 	 * Strings
 	 * 
-	 * @param tile Tile
+	 * @param tile the Tile
-	 * @param path to write url string
+	 * @param buf to write url string
-	 * @param curPos current position
+	 * @param pos current position
 	 * @return new position
 	 */
 	protected int formatTilePath(Tile tile, byte[] buf, int pos) {

vtm/src/org/oscim/utils/LineClipper.java

@@ -19,8 +19,6 @@ package org.oscim.utils;
 /**
  * from http://en.wikipedia.org/wiki/Cohen%E2%80%93
  * Sutherland_algorithm
- * 
- * @adapted by Hannes Janetzek
  */
 
 public class LineClipper {

vtm/src/org/oscim/utils/ScanBox.java

@@ -29,12 +29,10 @@ import org.oscim.core.Tile;
  * <pre>
  * Viewport.getMapViewProjection(box)
  * 	ScanBox sb = new ScanBox(){
- * 		@Override
  *          protected void setVisible(int y, int x1, int x2) {
  *          }
  * 	};
  * sb.scan(pos.x, pos.y, pos.scale, * zoomLevel, coords);
- * 
  * </pre>
  * 
  * where zoomLevel is the zoom-level for which tile coordinates should be
@@ -60,9 +58,7 @@ public abstract class ScanBox {
 		return mBox;
 	}
 
-	/*
+	/* ported from Polymaps: Layer.js */
-	 * ported from Polymaps: Layer.js
-	 */
 
 	static class Edge {
 		float x0, y0, x1, y1, dx, dy;

vtm/src/org/oscim/utils/Tessellator.java

@@ -32,16 +32,6 @@ public class Tessellator {
 	/**
 	 * Special version for ExtrusionLayer to match indices with vertex
 	 * positions.
-	 * 
-	 * @param points
-	 * @param ppos
-	 * @param plen
-	 * @param index
-	 * @param ipos
-	 * @param rings
-	 * @param vertexOffset
-	 * @param outTris
-	 * @return
 	 */
 	public static int tessellate(float[] points, int ppos, int plen, short[] index,
 	        int ipos, int rings, int vertexOffset, VertexItem outTris) {
@@ -116,10 +106,6 @@ public class Tessellator {
 
 	/**
 	 * Untested!
-	 * 
-	 * @param geom
-	 * @param out
-	 * @return
 	 */
 	public static int tessellate(GeometryBuffer geom, GeometryBuffer out) {
 
@@ -171,15 +157,6 @@ public class Tessellator {
 		return 1;
 	}
 
-	/**
-	 * 
-	 * @param geom
-	 * @param scale
-	 * @param outPoints
-	 * @param outTris
-	 * @param vertexOffset
-	 * @return
-	 */
 	public static int tessellate(GeometryBuffer geom, float scale,
 	        VertexItem outPoints, VertexItem outTris, int vertexOffset) {
 

vtm/src/org/oscim/utils/async/AsyncExecutor.java

@@ -42,7 +42,7 @@ public class AsyncExecutor {
 	 * Creates a new AsynchExecutor that allows maxConcurrent {@link Runnable}
 	 * instances to run in parallel.
 	 * 
-	 * @param maxConcurrent
+	 * @param maxConcurrent number of threads.
 	 */
 	public AsyncExecutor(int maxConcurrent) {
 		executor = Executors.newFixedThreadPool(maxConcurrent, new ThreadFactory() {

vtm/src/org/oscim/utils/async/AsyncResult.java

@@ -40,7 +40,7 @@ public class AsyncResult<T> {
 	/**
 	 * @return waits if necessary for the computation to complete and then
 	 *         returns the result
-	 * @throws GdxRuntimeException if there was an error
+	 * @throws RuntimeException if there was an error
 	 */
 	public T get() {
 		try {