-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
Improve javadoc to fix #11720 #11731
Conversation
Fix #11720 by improving the javadoc.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Minor niggles.
* | ||
* @param source the source to copy from | ||
* @param sink the sink to copy to | ||
* @param callback the callback to notify when the copy is complete | ||
* @see #copy(Source, Sink, Chunk.Processor, Callback) to allows processing of individual {@link Chunk}s, including |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
* @see #copy(Source, Sink, Chunk.Processor, Callback) to allows processing of individual {@link Chunk}s, including | |
* @see #copy(Source, Sink, Chunk.Processor, Callback) to allow processing of individual {@link Chunk}s, including |
* {@link Callback#succeeded() Succeeding} this callback will allow the processing of subsequent chunks. | ||
* {@link Callback#failed(Throwable) Failing} this callback will fail the processing of all chunks. | ||
* @return {@code True} if the chunk will be processed asynchronously and the callback will be called (or may have already been called), | ||
* {@code false} otherwise, in which case subsequent chunks may be processed and the passed callback ignored. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
* {@code false} otherwise, in which case subsequent chunks may be processed and the passed callback ignored. | |
* {@code False} otherwise, in which case subsequent chunks may be processed and the passed callback ignored. |
Or alternatively make both line 910 aand 911 lowercase
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
My preference is both lower case.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM with same niggle as @janbartel.
* {@link Callback#succeeded() Succeeding} this callback will allow the processing of subsequent chunks. | ||
* {@link Callback#failed(Throwable) Failing} this callback will fail the processing of all chunks. | ||
* @return {@code True} if the chunk will be processed asynchronously and the callback will be called (or may have already been called), | ||
* {@code false} otherwise, in which case subsequent chunks may be processed and the passed callback ignored. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
My preference is both lower case.
Fix #11720 by improving the javadoc.