Skip to content

Commit

Permalink
HttpResponseObserver: add default impl for onResponseDataRequested (
Browse files Browse the repository at this point in the history
#2243)

Motivation:

When we added two new callbacks in #2230, we forgot to add a default
impl for one of them.

Modifications:

- Add default noop impl for
`HttpResponseObserver#onResponseDataRequested(long)`;

Result:

`HttpResponseObserver` is backward compatible for existing users.
  • Loading branch information
idelpivnitskiy authored Jun 8, 2022
1 parent 0e38d94 commit 1f6e4ba
Showing 1 changed file with 2 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -177,7 +177,8 @@ interface HttpResponseObserver {
*
* @param n number of requested items
*/
void onResponseDataRequested(long n); // FIXME: 0.43 - consider removing default impl
default void onResponseDataRequested(long n) { // FIXME: 0.43 - consider removing default impl
}

/**
* Callback when the response payload body data chunk was observed.
Expand Down

0 comments on commit 1f6e4ba

Please sign in to comment.