Muhammad Rehan Saeed

**[Pyry Jahkola](http://twitter.com/pyrtsa)** commented on **2014-02-14 23:52:14** I think these [naming guidelines](https://github.com/ReactiveCocoa/ReactiveSwift/blob/master/Documentation/DesignGuidelines.md) from the Objective-C port of Rx were pretty sensible.

**Muhammad Rehan Saeed** commented on **2014-02-15 08:17:03** I had no idea Objective-C had Rx. Very Interesting guidelines. Especially for the side effects. Had not thought of that.

**[Neil Highley](http://www.neilhighley.com)** commented on **2017-01-25 12:22:09** Updted link: https://github.com/ReactiveCocoa/ReactiveSwift/blob/master/Documentation/DesignGuidelines.md

**Lars Støttrup Nielsen** commented on **2017-05-22 14:01:34** Not really sure where I stand yet on the hot vs cold naming convention. But it can be relevant for a consumer to...

**Michael Khalsa** commented on **2018-04-19 01:33:58** Nice series, thanks! Question about the timer example: Is it best practice to add on .Dispose to the subscriber to have it cleanup (assuming...

**[Muhammad Rehan Saeed](https://rehansaeed.com)** commented on **2018-04-19 09:16:59** > Nice series, thanks! > > Question about the timer example: > Is it best practice to add on .Dispose to the subscriber...

**Michael Khalsa** commented on **2018-04-22 00:29:47** > Here is a really good explanation: > > https://stackoverflow.com/questions/7703366/reactive-observable-subscription-disposal Thank you. It was necessary to remove the `.Dispose` in this case to make...

**David Christensen** commented on **2017-01-11 12:04:21** In your observable ShowWindow example, I believe there is a (potential) race condition. Shouldn't your code be as shown below? Obviously for this particular...

**[Muhammad Rehan Saeed](https://rehansaeed.com)** commented on **2017-01-11 14:25:38** > In your observable ShowWindow example, I believe there is a (potential) race condition. Shouldn't your code be as shown below? Obviously for...

**[Dennis Daume](http://twitter.com/flagbug)** commented on **2014-04-25 20:19:20** I think instead of `obs1.Merge(obs2).FirstAsync` you can write `obs1.Amb(obs2)` to get the first responding sequence :)