Merge pull request #28 from ArtifactForms/working

Merge Modifier Refactoring

Author: ArtifactForms

documentation/modifiers.md

@@ -99,6 +99,7 @@ desired effects.
 * **PushPullModifier:** Pushes or pulls vertices towards or away from a specified center point.
 * **RandomHolesModifier:**
 * **RemoveDoubleVerticesModifier:** Removes duplicate vertices from the mesh.
+* **RippleModifier:** Applies a ripple effect based on sinusoidal wave functions.
 * **RotateXModifier:** Rotates the mesh around the X-axis.
 * **RotateYModifier:** Rotates the mesh around the Y-axis.
 * **RotateZModifier:** Rotates the mesh around the Z-axis.
@@ -109,6 +110,7 @@ desired effects.
 * **SpherifyModifier:** Spherifies the mesh.
 * **TranslateModifier:** Translates the mesh.
 * **UpdateFaceNormalsModifier** Updates the face normals of the mesh.
+* **WaveModifier:** Applies a wave-like deformation, simulating the appearance of sinusoidal wave.
 * **WireframeModifier:** Converts the mesh to a wireframe representation.
 
 ## Subdivision Modifiers

src/main/java/math/Color.java

@@ -325,10 +325,10 @@ public Color subtractLocal(float r, float g, float b, float a) {
 	 * @return this
 	 */
 	public Color divideLocal(float a) {
-		r /= a;
-		g /= a;
-		b /= a;
-		a /= a;
+		this.r /= a;
+		this.g /= a;
+		this.b /= a;
+		this.a /= a;
 		return this;
 	}
 

src/main/java/math/GeometryUtil.java

@@ -129,7 +129,7 @@ public static double angleBetweenVectors(Vector3f v1, Vector3f v2) {
      *               values between 0 and 1 will return points in between.
      * @return The point along the line segment.
      */
-    public Vector2f getDistributionPoint(Vector2f start, Vector2f end,
+    public static Vector2f getDistributionPoint(Vector2f start, Vector2f end,
             float lambda) {
         float scalar = 1f / (1f + lambda);
         return start.add(end.mult(lambda)).mult(scalar);