Reduce `pageLabelInRange` nesting by using an early return

8bf9a5e0 · Robert Knight · 11234150 · 8bf9a5e0
Commit 8bf9a5e0 authored Dec 05, 2023 by Robert Knight
Show whitespace changes
Inline Side-by-side

Showing with 23 additions and 23 deletions

page-range.ts src/sidebar/util/page-range.ts +23 -23

No files found.
--- a/src/sidebar/util/page-range.ts
+++ b/src/sidebar/util/page-range.ts
@@ -8,7 +8,10 @@
 *   specify an empty range.
 */
 export function pageLabelInRange(label: string, range: string): boolean {
-  if (range.includes('-')) {
+  if (!range.includes('-')) {
+    return label === range;
+  }
+
  let [start, end] = range.split('-');
  if (!start) {
    start = label;
@@ -30,7 +33,4 @@ export function pageLabelInRange(label: string, range: string): boolean {
  } else {
    return false;
  }
-  } else {
-    return label === range;
-  }
 }