Giter VIP home page Giter VIP logo

Comments (6)

Odrakir avatar Odrakir commented on May 22, 2024

By the way, I'm using this gist for the timer: https://gist.github.com/natecook1000/b0285b518576b22c4dc8

from rxswift.

kzaher avatar kzaher commented on May 22, 2024

Hi,
wow :) It's a bug. Thnx for creating an issue.

So AutoDetach observer was in place and it was disposing in case of sequence was completed, but in case you didn't dispose producer, it would pass events until producer was disposed.

Problem was here

override public func subscribe<O : ObserverType where O.Element == Element>(observer: O) -> Disposable {
        let autoDetachObserver = AutoDetachObserver(observer: observer)

        let disposable = subscribeCore(ObserverOf(autoDetachObserver))
        autoDetachObserver.setDisposable(disposable)

        return autoDetachObserver //<< was disposable 
    }

I can fix this, add additional unit tests and push this to develop branch.

This doesn't seem to me like a critical issue, so are you ok just using develop branch code until next release until this enters master branch?

from rxswift.

Odrakir avatar Odrakir commented on May 22, 2024

Hi, thanks for the fast response.

It's ok, I'm just learning, so I'm not in a hurry. It just helps to know it wasn't me doing something wrong. I'll keep an eye on the fix to see if I can understand what was going on :)

from rxswift.

alcarvalho avatar alcarvalho commented on May 22, 2024

I think I am experiencing this bug, too... but with throttle.

from rxswift.

kzaher avatar kzaher commented on May 22, 2024

The fix should be in develop

commit 24dc2c6
Author: Krunoslav Zaher [email protected]
Date: Fri Jul 24 16:47:49 2015 +0200

Fixes problem with issues/67.

@alcarvalho send me all heisenbugs 😛

from rxswift.

kzaher avatar kzaher commented on May 22, 2024

These changes have been released in the latest release. Closing this issue. Let me know if it happens again.

from rxswift.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.