Do not freeze returned values

FossilOrigin-Name: e58d2d667861170182f20effe84863edecfd2a7b2c9ecad57d5b90faf83e2653
iMarv · Apr 29, 2021 · dbedd74 · dbedd74
1 parent bb6c65e
commit dbedd74
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 10 deletions.
diff --git a/mod.test.ts b/mod.test.ts
@@ -25,14 +25,6 @@ Rhum.testPlan("option", () => {
     });
   });
 
-  Rhum.testSuite("match", () => {
-    Rhum.testCase("should freeze value", () => {
-      const val = match("testi");
-
-      Rhum.asserts.assert(Object.isFrozen(val.unwrap()));
-    });
-  });
-
   Rhum.testSuite("toOption", () => {
     Rhum.testCase("should return value", () => {
       const original: Option<string> = "testi";

diff --git a/mod.ts b/mod.ts
@@ -29,7 +29,7 @@ export function isNone(value: Option<unknown>): value is None {
  * @param maybe Value that is possibly null or undefined
  */
 export function match<T>(maybe: Option<T>): Matcher<T> {
-  return new Matcher(Object.freeze(maybe));
+  return new Matcher(maybe);
 }
 
 interface isSomeArg<T, R> {
@@ -123,7 +123,7 @@ export class Matcher<T> {
    */
   unwrapOr(defaultValue: T): Some<Readonly<T>> {
     if (isNone(this._value)) {
-      return Object.freeze(defaultValue);
+      return defaultValue;
     }
 
     return this._value;