[
https://issues.apache.org/jira/browse/ZOOKEEPER-2825?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16068086#comment-16068086
]
ASF GitHub Bot commented on ZOOKEEPER-2825:
-------------------------------------------
Github user asdf2014 commented on a diff in the pull request:
https://github.com/apache/zookeeper/pull/297#discussion_r124757125
--- Diff:
src/java/main/org/apache/zookeeper/client/ZooKeeperSaslClient.java ---
@@ -590,14 +590,13 @@ public boolean
clientTunneledAuthenticationInProgress() {
// authentication is either in progress, successful, or
failed.
// 1. Authentication hasn't finished yet: we must wait for
it to do so.
- if ((isComplete() == false) &&
- (isFailed() == false)) {
+ if ((!isComplete()) && (!isFailed())) {
return true;
}
// 2. SASL authentication has succeeded or failed..
if (isComplete() || isFailed()) {
--- End diff --
Maybe i understand what you mean, if **`not_complete && not_failed`**
(Line: 593) is `FALSE` means **`complete || non_failed`** and **`complete ||
failed`** and **`non_complete || failed`** is `TRUE`, then the `isComplete() ||
isFailed()` (Line: 599) will always be `TRUE`.
> 1. Remove unnecessary import; 2. `contains` instead of `indexOf > -1` for
> more readable; 3. Standardize `StringBuilder#append` usage for CLIENT module
> ------------------------------------------------------------------------------------------------------------------------------------------------------
>
> Key: ZOOKEEPER-2825
> URL: https://issues.apache.org/jira/browse/ZOOKEEPER-2825
> Project: ZooKeeper
> Issue Type: Improvement
> Components: java client
> Affects Versions: 3.5.3
> Reporter: Benedict Jin
> Assignee: Benedict Jin
> Priority: Minor
> Labels: refactoring
> Fix For: 3.5.4, 3.6.0
>
> Original Estimate: 72h
> Remaining Estimate: 72h
>
> * Remove unncessnary import;
> * `contains` instead of `indexOf > -1` for more readable;
> * Standardize `StringBuilder#append` usage for CLIENT module
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)
