Skip to content

1.x: just() construction to call the onCreate execution hook#3958

Merged
zsxwing merged 2 commits intoReactiveX:1.xfrom
akarnokd:JustHook
May 23, 2016
Merged

1.x: just() construction to call the onCreate execution hook#3958
zsxwing merged 2 commits intoReactiveX:1.xfrom
akarnokd:JustHook

Conversation

@akarnokd
Copy link
Member

This PR adds the call to RxJavaObservableExecutionHook.onCreate() to the just() - ScalarSynchronousObservable construction.

Related: #2656.

*/
public final class ScalarSynchronousObservable<T> extends Observable<T> {
/** The execution hook instance. */
static RxJavaObservableExecutionHook hook = RxJavaPlugins.getInstance().getObservableExecutionHook();
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: final

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah, you override it in the test. Then it needs a comment or test should use reflection.

@akarnokd
Copy link
Member Author

Updated javadoc and test.

@artem-zinnatullin
Copy link
Contributor

👍

2 similar comments
@ZacSweers
Copy link
Contributor

👍

@zsxwing
Copy link
Member

zsxwing commented May 23, 2016

👍

@zsxwing zsxwing merged commit 3ea9265 into ReactiveX:1.x May 23, 2016
@akarnokd akarnokd deleted the JustHook branch May 23, 2016 23:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants