feat(math)!: Move Quat.toMat4() to Mat4.createRotation()

src/math/Mat4.js

@@ -449,6 +449,35 @@ export class Mat4 {
 		]);
 	}
 
+	/**
+	 * @param  {import("./Quat.js").QuatParameters} args
+	 */
+	static createRotation(...args) {
+		const q = new Quat(...args);
+
+		// https://github.com/toji/gl-matrix/blob/6866ae93d19bbff032139941cbfe0ae68c4cdead/src/gl-matrix/mat4.js#L1186
+		const x2 = q.x + q.x;
+		const y2 = q.y + q.y;
+		const z2 = q.z + q.z;
+
+		const xx = q.x * x2;
+		const yx = q.y * x2;
+		const yy = q.y * y2;
+		const zx = q.z * x2;
+		const zy = q.z * y2;
+		const zz = q.z * z2;
+		const wx = q.w * x2;
+		const wy = q.w * y2;
+		const wz = q.w * z2;
+
+		return new Mat4([
+			[1 - yy - zz, yx + wz, zx - wy, 0],
+			[yx - wz, 1 - xx - zz, zy + wx, 0],
+			[zx + wy, zy - wx, 1 - xx - yy, 0],
+			[0, 0, 0, 1],
+		]);
+	}
+
 	/**
 	 * @param  {import("./Vec3.js").Vec3Parameters} args
 	 */

src/math/Quat.js

@@ -1,4 +1,3 @@
-import { Mat4 } from "./Mat4.js";
 import { Vec2 } from "./Vec2.js";
 import { Vec3 } from "./Vec3.js";
 import { Vec4 } from "./Vec4.js";
@@ -201,30 +200,6 @@ export class Quat {
 		return vec;
 	}
 
-	toMat4() {
-		// https://github.com/toji/gl-matrix/blob/6866ae93d19bbff032139941cbfe0ae68c4cdead/src/gl-matrix/mat4.js#L1186
-		const x2 = this.x + this.x;
-		const y2 = this.y + this.y;
-		const z2 = this.z + this.z;
-
-		const xx = this.x * x2;
-		const yx = this.y * x2;
-		const yy = this.y * y2;
-		const zx = this.z * x2;
-		const zy = this.z * y2;
-		const zz = this.z * z2;
-		const wx = this.w * x2;
-		const wy = this.w * y2;
-		const wz = this.w * z2;
-
-		return new Mat4([
-			[1 - yy - zz, yx + wz, zx - wy, 0],
-			[yx - wz, 1 - xx - zz, zy + wx, 0],
-			[zx + wy, zy - wx, 1 - xx - yy, 0],
-			[0, 0, 0, 1],
-		]);
-	}
-
 	// http://www.euclideanspace.com/maths/geometry/rotations/conversions/eulerToQuaternion/index.htm
 	/**
 	 * @param {import("./Vec3.js").Vec3Parameters} args

test/unit/src/math/Mat4.test.js

@@ -205,6 +205,36 @@ Deno.test({
 	},
 });
 
+Deno.test({
+	name: "createTranslation",
+	fn() {
+		const mat = Mat4.createTranslation(1, 2, 3);
+		assertVecAlmostEquals(mat.getTranslation(), [1, 2, 3]);
+	},
+});
+
+Deno.test({
+	name: "createRotation",
+	fn() {
+		const quat = Quat.fromAxisAngle(0, 1, 0, Math.PI * 0.5);
+		const v1 = Vec3.forward.rotate(quat);
+
+		const mat = Mat4.createRotation(quat);
+		const v2 = new Vec3(0, 0, 1).multiply(mat);
+
+		assertVecAlmostEquals(v1, v2);
+	},
+});
+
+Deno.test({
+	name: "createScale",
+	fn() {
+		const mat = Mat4.createScale(1, 2, 3);
+
+		assertVecAlmostEquals(mat.getScale(), [1, 2, 3]);
+	},
+});
+
 Deno.test({
 	name: "multiplyMatrices",
 	fn() {

test/unit/src/math/Quat.test.js

@@ -17,19 +17,6 @@ Deno.test({
 	},
 });
 
-Deno.test({
-	name: "toMat4()",
-	fn() {
-		const quat = Quat.fromAxisAngle(0, 1, 0, Math.PI * 0.5);
-		const v1 = Vec3.forward.rotate(quat);
-
-		const mat = quat.toMat4();
-		const v2 = new Vec3(0, 0, 1).multiply(mat);
-
-		assertVecAlmostEquals(v1, v2);
-	},
-});
-
 Deno.test({
 	name: "toString()",
 	fn() {