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

Fix zip race condition #778

Merged
merged 2 commits into from
Jan 22, 2014
Merged

Fix zip race condition #778

merged 2 commits into from
Jan 22, 2014
+40 −61

Commits on Jan 22, 2014

  1. Fix Zip race condition 

    ItemObserver onNext might not acquire the write lock due to an
onCompleted being handled by another thread. When handling
onCompleted, the ItemObserver does not check for any values that are
ready to be emitted, which might cause OperationZip to never emit
OnNext or OnCompleted.
    @vadims
    vadims committed Jan 22, 2014
    Configuration menu
    Copy the full SHA
    c2baf93 View commit details
    Browse the repository at this point in the history

  2. Simplify OperationZip ItemObserver 

    LinkedList permits null values, no need for NULL_SENTINEL.
    @vadims
    vadims committed Jan 22, 2014
    Configuration menu
    Copy the full SHA
    f3dbf3c View commit details
    Browse the repository at this point in the history