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.

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

Refactoring #6

Merged
merged 5 commits into from
Jan 16, 2013
Merged

Conversation

benjchristensen
Copy link
Member

  • convert to rx.* package names
    • rx.observables
    • rx.observables.operations (private package from javadocs)
    • rx.util
    • rx.lang
    • rx.concurrency (placeholder for schedulers)
  • change Func order to have return type at end => T, R instead of R, T
    • this matches the C# and Guava conventions
  • added Action as Func without return value
    • added overload subscribe methods with Action typed arguments
  • added from() alias methods in Observable

benjchristensen added a commit that referenced this pull request Jan 16, 2013
@benjchristensen benjchristensen merged commit c11126d into ReactiveX:master Jan 16, 2013
rickbw pushed a commit to rickbw/RxJava that referenced this pull request Jan 9, 2014
jihoonson pushed a commit to jihoonson/RxJava that referenced this pull request Mar 6, 2020
…e rate is above (>) the threshold, but it should be above or equal (>=).
jihoonson pushed a commit to jihoonson/RxJava that referenced this pull request Mar 6, 2020
…e rate is above (>) the threshold, but it should be above or equal (>=).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant