Fixed reactive implementation

[fabb]

• Fixes most of Problems with unit tests / what unit tests revealed #7 (didn't check for leaks, commented out XCTAssertNil(mockTimer) for now)
• Fixes Reactive Controller doesn't 'reset' #3
• Also fixed a bug in The Rotate and Pan issue on the reactive example is now fixed #6: @morgz You nearly nailed it, but according to the imperative version, the timer should be stopped if either of the gestures ended, not if both ended

[fabb]

@itchingpixels: what do you think?

[almostintuitive]

sorry, I'll have time to look today!:)

[almostintuitive]

really nice, thank you!

I've pushed some code to this branch:
78f39dc fixed_reactive_implementation-ver-2

I've tweaked a little bit on these:

• we could get rid of the optional in Variable<UIGestureRecognizerType?> if the init requirements are rearranged
• instead of returning new Observable from combineLatest, you can just return a value, that makes it slightly more simple.
• using filter operator on the signal level instead of guard in the subscribeNext block

I have been thinking and I haven't found any other straightforward way to filter out multiple began events than to combine the .Began and .Ended signals as you did (then using distinctUntilChanged), that was a nice solution!

what do you think?

[almostintuitive]

I've just updated the commit hash in the last message

[fabb]

@itchingpixels I'd say we merge this PR, and create another one for your changes. The other branch has quite many broken unit tests that still would need to be resolved.

[almostintuitive]

cool, cheers. I've made some last minute changes because previously all unit tests were passing but the actual demo didn't work as expected, and it seems like I forgot to run the tests again after :(