feat: remove setPoints and change fromPoints signature

- match pex-math
- update tests

Author: dmnsgn

README.md

@@ -392,8 +392,7 @@ A ray defined by a starting 3D point origin and a 3D direction vector.
     - [.copy(b)](#module_rect.copy) ⇒ [<code>rect</code>](#module_rect..rect)
     - [.set(a, b)](#module_rect.set) ⇒ [<code>rect</code>](#module_rect..rect)
     - [.isEmpty(a)](#module_rect.isEmpty) ⇒ <code>boolean</code>
-    - [.fromPoints(points)](#module_rect.fromPoints) ⇒ [<code>rect</code>](#module_rect..rect)
-    - [.setPoints(a, points)](#module_rect.setPoints) ⇒ [<code>rect</code>](#module_rect..rect)
+    - [.fromPoints(a, points)](#module_rect.fromPoints) ⇒ [<code>rect</code>](#module_rect..rect)
     - [.getPoints(a, points)](#module_rect.getPoints) ⇒ <code>Array.&lt;module:pex-math~vec2&gt;</code>
     - [.scale(a, n)](#module_rect.scale) ⇒ [<code>rect</code>](#module_rect..rect)
     - [.setSize(a, size)](#module_rect.setSize) ⇒ [<code>rect</code>](#module_rect..rect)
@@ -470,19 +469,7 @@ Checks if a rectangle is empty.
 
 <a name="module_rect.fromPoints"></a>
 
-### rect.fromPoints(points) ⇒ [<code>rect</code>](#module_rect..rect)
-
-Creates a rectangle from a list of points.
-
-**Kind**: static method of [<code>rect</code>](#module_rect)
-
-| Param  | Type                                            |
-| ------ | ----------------------------------------------- |
-| points | <code>Array.&lt;module:pex-math~vec2&gt;</code> |
-
-<a name="module_rect.setPoints"></a>
-
-### rect.setPoints(a, points) ⇒ [<code>rect</code>](#module_rect..rect)
+### rect.fromPoints(a, points) ⇒ [<code>rect</code>](#module_rect..rect)
 
 Updates a rectangle from a list of points.
 

rect.js

@@ -59,22 +59,13 @@ export function isEmpty(a) {
   return a[0][0] > a[1][0] || a[0][1] > a[1][1];
 }
 
-/**
- * Creates a rectangle from a list of points.
- * @param {import("pex-math").vec2[]} points
- * @returns {rect}
- */
-export function fromPoints(points) {
-  return setPoints(create(), points);
-}
-
 /**
  * Updates a rectangle from a list of points.
  * @param {rect} a
  * @param {import("pex-math").vec2[]} points
  * @returns {rect}
  */
-export function setPoints(a, points) {
+export function fromPoints(a, points) {
   for (let i = 0; i < points.length; i++) {
     includePoint(a, points[i]);
   }

test/rect.js

@@ -52,11 +52,8 @@ describe("rect", () => {
     strictEqual(rect.isEmpty(NORM_RECT), false);
   });
 
-  it("fromPoints() should create a rectangle from a list of points", () => {
-    deepEqual(rect.fromPoints(NORM_POINTS), NORM_RECT);
-  });
-  it("setPoints() should update a rectangle from a list of points", () => {
-    deepEqual(rect.setPoints(rect.copy(DEFAULT_RECT), NORM_POINTS), NORM_RECT);
+  it("fromPoints() should update a rectangle from a list of points", () => {
+    deepEqual(rect.fromPoints(rect.copy(DEFAULT_RECT), NORM_POINTS), NORM_RECT);
   });
   it("getPoints() should return a list of 4 points from a rectangle", () => {
     deepEqual(rect.getPoints(rect.copy(NORM_RECT)), NORM_POINTS);