Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Map Error Handling #314

Merged
merged 2 commits into from
Jul 31, 2013
Merged

Map Error Handling #314

merged 2 commits into from
Jul 31, 2013

Commits on Jul 31, 2013

  1. Rename AtomicObserver to SafeObserver 

    Rename to be more descriptive of what it has evolved to.
    @benjchristensen
    benjchristensen committed Jul 31, 2013
    Configuration menu
    Copy the full SHA
    6886142 View commit details
    Browse the repository at this point in the history

  2. Better handling of map function errors 

    - Stop catching the error and passing to onError and instead let the SafeObserver handle it which will then prevent subsequent onNext calls and/or unsubscribe when a failure occurs.
- This also solves the OnErrorResumeNext issue fixed in ReactiveX#312 but those changes still seem valid so I'll leave them.

Related to ReactiveX#216 and ReactiveX#312
    @benjchristensen
    benjchristensen committed Jul 31, 2013
    Configuration menu
    Copy the full SHA
    863a064 View commit details
    Browse the repository at this point in the history