/*
 * Copyright (c) 2017, Adam <Adam@sigterm.info>
 * All rights reserved.
 *
 * Redistribution and use in source and binary forms, with or without
 * modification, are permitted provided that the following conditions are met:
 *
 * 1. Redistributions of source code must retain the above copyright notice, this
 *    list of conditions and the following disclaimer.
 * 2. Redistributions in binary form must reproduce the above copyright notice,
 *    this list of conditions and the following disclaimer in the documentation
 *    and/or other materials provided with the distribution.
 *
 * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS" AND
 * ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED
 * WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE
 * DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT OWNER OR CONTRIBUTORS BE LIABLE FOR
 * ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES
 * (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES;
 * LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND
 * ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
 * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS
 * SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
 */
package net.runelite.api.model;

import java.util.ArrayList;
import java.util.List;
import net.runelite.api.Point;

/**
 * Provides utility methods for computing the convex hull of a list of
 * <em>n</em> points.
 * <p>
 * The implementation uses the Jarvis march algorithm and runs in O(nh)
 * time in the worst case, where n is the number of points and h the
 * number of points on the convex hull.
 */
public class Jarvis
{
	/**
	 * Computes and returns the convex hull of the passed points.
	 * <p>
	 * The size of the list must be at least 4, otherwise this method will
	 * return null.
	 *
	 * @param points list of points
	 * @return list containing the points part of the convex hull
	 */
	public static List<Point> convexHull(List<Point> points)
	{
		if (points.size() < 3)
		{
			return null;
		}

		List<Point> ch = new ArrayList<>();

		// find the left most point
		Point left = findLeftMost(points);

		// current point we are on
		Point current = left;

		do
		{
			ch.add(current);

			// the next point - all points are to the right of the
			// line between current and next
			Point next = null;

			for (Point p : points)
			{
				if (next == null)
				{
					next = p;
					continue;
				}

				int cp = crossProduct(current, p, next);
				if (cp > 0 || (cp == 0 && current.distanceTo(p) > current.distanceTo(next)))
				{
					next = p;
				}
			}

			// Points can be null if they are behind or very close to the camera.
			if (next == null)
			{
				return null;
			}

			assert ch.size() <= points.size() : "hull has more points than graph";
			current = next;
		}
		while (current != left);

		return ch;
	}

	private static Point findLeftMost(List<Point> points)
	{
		Point left = null;

		for (Point p : points)
		{
			if (left == null || p.getX() < left.getX())
			{
				left = p;
			}
			else if (p.getX() == left.getX() && p.getY() < left.getY())
			{
				left = p;
			}
		}

		return left;
	}

	private static int crossProduct(Point p, Point q, Point r)
	{
		int val = (q.getY() - p.getY()) * (r.getX() - q.getX())
			- (q.getX() - p.getX()) * (r.getY() - q.getY());
		return val;
	}
}

@@ -2370,16 +2370,216 @@
 urrent);
+%0A%09%09%09assert ch.size() %3C= points.size() : %22hull has more points than graph%22;%0A%09%09%09if (ch.size() %3E points.size())%0A%09%09%09%7B%0A%09%09%09%09// Just to make sure we never somehow get stuck in this loop%0A%09%09%09%09return null;%0A%09%09%09%7D
 %0A%0A%09%09%09// 
@@ -2789,19 +2789,20 @@
 %09%7D%0A%0A%09%09%09%09
-int
+long
  cp = cr
@@ -3071,82 +3071,8 @@
 %09%7D%0A%0A
-%09%09%09assert ch.size() %3C= points.size() : %22hull has more points than graph%22;%0A
 %09%09%09c
@@ -3442,19 +3442,20 @@
  static 
-int
+long
  crossPr
@@ -3496,18 +3496,25 @@
 %7B%0A%09%09
-int
+long
  val = 
+(long)
 (q.g
@@ -3560,16 +3560,22 @@
 ))%0A%09%09%09- 
+(long)
 (q.getX(

Fix Jarvis infinite loop