Set next in the ValueObserver wrather than wrap it.

Reviewers: O4 Material Motion Apple platform reviewers, O2 Material Motion, markwei Reviewed By: O4 Material Motion Apple platform reviewers, O2 Material Motion, markwei Tags: #material_motion Differential Revision: http://codereview.cc/D2121
material-motion · Dec 6, 2016 · ec6d4b1 · ec6d4b1
1 parent fd5869d
commit ec6d4b1
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 9 deletions.
diff --git a/src/IndefiniteObservable.swift b/src/IndefiniteObservable.swift
@@ -107,15 +107,11 @@ public let noopUnsubscription: (() -> Void)? = nil
 public final class ValueObserver<T> {
   public typealias Value = T
 
-  init(_ next: @escaping (Value) -> Void) {
-    _next = next
+  public init(_ next: @escaping (T) -> Void) {
+    self.next = next
   }
 
-  public func next(_ value: Value) {
-    _next(value)
-  }
-
-  private let _next: (Value) -> Void
+  public let next: (T) -> Void
 }
 
 // MARK: Private

diff --git a/tests/unit/ObservableTests.swift b/tests/unit/ObservableTests.swift
@@ -128,8 +128,8 @@ class ObservableTests: XCTestCase {
   func testFilteringValues() {
     let value = CGPoint(x: 0, y: 10)
     let observable = IndefiniteObservable<(Bool, CGPoint)> { observer in
-      observer.next((false, value))
-      observer.next((true, value))
+      observer.next(false, value)
+      observer.next(true, value)
       return noopUnsubscription
     }