minor refactors

Author: pcattori

packages/react-router/lib/href.ts

@@ -27,12 +27,12 @@ export function href<Path extends keyof Args>(
   ...args: Args[Path]
 ): string {
   let params = args[0];
-  let result = trimEndSplat(path) // Ignore trailing / and /*, we'll handle it below
+  let result = trimTrailingSplat(path) // Ignore trailing / and /*, we'll handle it below
     .replace(
       /\/:([\w-]+)(\?)?/g, // same regex as in .\router\utils.ts: compilePath().
       (_: string, param: string, questionMark: string | undefined) => {
         const isRequired = questionMark === undefined;
-        const value = params ? params[param] : undefined;
+        const value = params?.[param];
         if (isRequired && value === undefined) {
           throw new Error(
             `Path '${path}' requires param '${param}' but it was not provided`,
@@ -45,7 +45,7 @@ export function href<Path extends keyof Args>(
   if (path.endsWith("*")) {
     // treat trailing splat the same way as compilePath, and force it to be as if it were `/*`.
     // `react-router typegen` will not generate the params for a malformed splat, causing a type error, but we can still do the correct thing here.
-    const value = params ? params["*"] : undefined;
+    const value = params?.["*"];
     if (value !== undefined) {
       result += "/" + value;
     }
@@ -55,22 +55,20 @@ export function href<Path extends keyof Args>(
 }
 
 /**
-  Removes a trailing splat and any number of slashes from the end of the path.
-
-  Benchmarks as running faster than `path.replace(/\/*\*?$/, "")`, which backtracks.
+ * Removes a trailing splat and any number of slashes from the end of the path.
+ *
+ * Benchmarked to be faster than `path.replace(/\/*\*?$/, "")`, which backtracks.
  */
-function trimEndSplat(path: string): string {
+function trimTrailingSplat(path: string): string {
   let i = path.length - 1;
   let char = path[i];
-  if (char !== "*" && char !== "/") {
-    return path;
-  }
+  if (char !== "*" && char !== "/") return path;
+
+  // for/break benchmarks faster than do/while
   i--;
   for (; i >= 0; i--) {
-    // for/break benchmarks faster than do/while
-    if (path[i] !== "/") {
-      break;
-    }
+    if (path[i] !== "/") break;
   }
+
   return path.slice(0, i + 1);
 }