Remove redundant ->() in refmt code

RationalJS · Nov 30, 2018 · 649a569 · 649a569
1 parent 5d16528
commit 649a569
Show file tree

Hide file tree

Showing 5 changed files with 231 additions and 301 deletions.
diff --git a/src/Future.re b/src/Future.re
@@ -11,7 +11,7 @@ let make = resolver => {
     switch (data^) {
     | None =>
       data := Some(result);
-      (callbacks^)->Belt.List.reverse->(Belt.List.forEach(cb => cb(result)));
+      (callbacks^)->Belt.List.reverse->Belt.List.forEach(cb => cb(result));
       /* Clean up memory usage */
       callbacks := [];
     | Some(_) => () /* Do nothing; theoretically not possible */
@@ -60,39 +60,33 @@ let get = (Future(getFn), f) => getFn(f);
  * Future Belt.Result convenience functions,
  * for working with a type Future.t( Belt.Result.t('a,'b) )
  */
-let mapOk = (future, f) => future->(map(r => Belt.Result.map(r, f)));
+let mapOk = (future, f) => future->map(r => Belt.Result.map(r, f));
 
 let mapError = (future, f) =>
   future
-  ->(
-      map(r =>
-        switch (r) {
-        | Belt.Result.Error(v) => Belt.Result.Error(f(v))
-        | Ok(a) => Belt.Result.Ok(a)
-        }
-      )
+  ->map(r =>
+      switch (r) {
+      | Belt.Result.Error(v) => Belt.Result.Error(f(v))
+      | Ok(a) => Belt.Result.Ok(a)
+      }
     );
 
 let flatMapOk = (future, f) =>
   future
-  ->(
-      flatMap(r =>
-        switch (r) {
-        | Belt.Result.Ok(v) => f(v)
-        | Belt.Result.Error(e) => value(Belt.Result.Error(e))
-        }
-      )
+  ->flatMap(r =>
+      switch (r) {
+      | Belt.Result.Ok(v) => f(v)
+      | Belt.Result.Error(e) => value(Belt.Result.Error(e))
+      }
     );
 
 let flatMapError = (future, f) =>
   future
-  ->(
-      flatMap(r =>
-        switch (r) {
-        | Belt.Result.Ok(v) => value(Belt.Result.Ok(v))
-        | Belt.Result.Error(e) => f(e)
-        }
-      )
+  ->flatMap(r =>
+      switch (r) {
+      | Belt.Result.Ok(v) => value(Belt.Result.Ok(v))
+      | Belt.Result.Error(e) => f(e)
+      }
     );
 
 let mapOk2 = (fa, fb, f) => flatMapOk(fa, a => mapOk(fb, b => f(a, b)));
@@ -107,24 +101,20 @@ let mapOk5 = (fa, fb, fc, fd, fe, f) =>
 
 let tapOk = (future, f) =>
   future
-  ->(
-      tap(r =>
-        switch (r) {
-        | Belt.Result.Ok(v) => f(v)->ignore
-        | Error(_) => ()
-        }
-      )
+  ->tap(r =>
+      switch (r) {
+      | Belt.Result.Ok(v) => f(v)->ignore
+      | Error(_) => ()
+      }
     );
 
 let tapError = (future, f) =>
   future
-  ->(
-      tap(r =>
-        switch (r) {
-        | Belt.Result.Error(v) => f(v)->ignore
-        | Ok(_) => ()
-        }
-      )
+  ->tap(r =>
+      switch (r) {
+      | Belt.Result.Error(v) => f(v)->ignore
+      | Ok(_) => ()
+      }
     );
 
 let (>>=) = flatMapOk;

diff --git a/src/FutureJs.re b/src/FutureJs.re
@@ -30,19 +30,17 @@ let fromPromise = (promise, errorTransformer) =>
 
 let toPromise = future =>
   Js.Promise.make((~resolve, ~reject as _) =>
-    future->(Future.get(value => resolve(. value)))
+    future->Future.get(value => resolve(. value))
   );
 
 let resultToPromise = future =>
   Js.Promise.make((~resolve, ~reject) =>
     future
-    ->(
-        Future.map(result =>
-          switch (result) {
-          | Belt.Result.Ok(result) => resolve(. result)
-          | Belt.Result.Error(error) => reject(. error)
-          }
-        )
+    ->Future.map(result =>
+        switch (result) {
+        | Belt.Result.Ok(result) => resolve(. result)
+        | Belt.Result.Error(error) => reject(. error)
+        }
       )
     ->ignore
   );
diff --git a/src/FutureResult.re b/src/FutureResult.re
@@ -1,79 +1,65 @@
 let deprecate = (ft, name, more) =>
   ft
-  ->(
-      Future.tap(_ =>
-        Js.Console.warn("FutureResult." ++ name ++ " is deprecated." ++ more)
-      )
+  ->Future.tap(_ =>
+      Js.Console.warn("FutureResult." ++ name ++ " is deprecated." ++ more)
     );
 
 let mapOk = (future, f) =>
   future
-  ->(
-      Future.map(r =>
-        switch (r) {
-        | Js.Result.Ok(v) => f(v)
-        | Error(e) => Js.Result.Error(e)
-        }
-      )
+  ->Future.map(r =>
+      switch (r) {
+      | Js.Result.Ok(v) => f(v)
+      | Error(e) => Js.Result.Error(e)
+      }
     )
-  ->(deprecate("mapOk", " Please user Future.mapOk instead."));
+  ->deprecate("mapOk", " Please user Future.mapOk instead.");
 
 let flatMapOk = (future, f) =>
   future
-  ->(
-      Future.flatMap(r =>
-        switch (r) {
-        | Js.Result.Ok(v) => f(v)
-        | Error(e) => Future.value(Js.Result.Error(e))
-        }
-      )
+  ->Future.flatMap(r =>
+      switch (r) {
+      | Js.Result.Ok(v) => f(v)
+      | Error(e) => Future.value(Js.Result.Error(e))
+      }
     )
-  ->(deprecate("flatMapOk", ""));
+  ->deprecate("flatMapOk", "");
 
 let mapError = (future, f) =>
   future
-  ->(
-      Future.map(r =>
-        switch (r) {
-        | Js.Result.Error(v) => f(v)
-        | Ok(a) => Js.Result.Ok(a)
-        }
-      )
+  ->Future.map(r =>
+      switch (r) {
+      | Js.Result.Error(v) => f(v)
+      | Ok(a) => Js.Result.Ok(a)
+      }
     )
-  ->(deprecate("mapError", " Please user Future.mapError instead."));
+  ->deprecate("mapError", " Please user Future.mapError instead.");
 
 let flatMapError = (future, f) =>
   future
-  ->(
-      Future.flatMap(r =>
-        switch (r) {
-        | Js.Result.Error(v) => f(v)
-        | Ok(a) => Future.value(Js.Result.Ok(a))
-        }
-      )
+  ->Future.flatMap(r =>
+      switch (r) {
+      | Js.Result.Error(v) => f(v)
+      | Ok(a) => Future.value(Js.Result.Ok(a))
+      }
     )
-  ->(deprecate("mapOk", ""));
+  ->deprecate("mapOk", "");
 
 let tapOk = (future, f) =>
   future
-  ->(
-      Future.tap(r =>
-        switch (r) {
-        | Js.Result.Ok(v) => f(v)->ignore
-        | Error(_) => ()
-        }
-      )
+  ->Future.tap(r =>
+      switch (r) {
+      | Js.Result.Ok(v) => f(v)->ignore
+      | Error(_) => ()
+      }
     )
-  ->(deprecate("tapOk", " Please user Future.tapOk instead."));
+  ->deprecate("tapOk", " Please user Future.tapOk instead.");
 
 let tapError = (future, f) =>
   future
-  ->(
-      Future.tap(r =>
-        switch (r) {
-        | Js.Result.Error(v) => f(v)->ignore
-        | Ok(_) => ()
-        }
-      )
+  ->Future.tap(r =>
+      switch (r) {
+      | Js.Result.Error(v) => f(v)->ignore
+      | Ok(_) => ()
+      }
     )
-  ->(deprecate("tapError", " Please user Future.tapError instead."));
+  ->deprecate("tapError", " Please user Future.tapError instead.");