Remove unused argument from multiplyUnsafe

Author: paulmillr

src/abstract/edwards.ts

@@ -542,12 +542,12 @@ export function edwards(
     // Keeps the same subgroup-scalar contract: 0 is allowed for public-scalar callers, but
     // n and larger values are rejected instead of being reduced mod n to the identity point.
     // Accepts optional accumulator to merge with multiply (important for sparse scalars)
-    multiplyUnsafe(scalar: bigint, acc = Point.ZERO): Point {
+    multiplyUnsafe(scalar: bigint): Point {
       // 0 <= scalar < L
       if (!Fn.isValid(scalar)) throw new RangeError('invalid scalar: expected 0 <= sc < curve.n');
       if (scalar === _0n) return Point.ZERO;
       if (this.is0() || scalar === _1n) return this;
-      return wnaf.unsafe(this, scalar, (p) => normalizeZ(Point, p), acc);
+      return wnaf.unsafe(this, scalar, (p) => normalizeZ(Point, p));
     }
 
     // Checks if point is of small order.

src/abstract/weierstrass.ts

@@ -1093,9 +1093,10 @@ export function weierstrass<T>(
      * It's faster, but should only be used when you don't care about
      * an exposed secret key e.g. sig verification, which works over *public* keys.
      */
-    multiplyUnsafe(sc: bigint): Point {
+    multiplyUnsafe(scalar: bigint): Point {
       const { endo } = extraOpts;
       const p = this as Point;
+      const sc = scalar;
       // Public-scalar callers may need 0, but n and larger values stay rejected here too.
       // Reducing them mod n would turn bad caller input into an accidental identity point.
       if (!Fn.isValid(sc)) throw new RangeError('invalid scalar: out of range'); // 0 is valid