summaryrefslogtreecommitdiffstats
path: root/base/common/src/com/netscape/cmscore/policy/PolicyPredicateParser.java
diff options
context:
space:
mode:
authorAbhishek Koneru <akoneru@redhat.com>2012-06-05 12:16:51 -0400
committerEndi Sukma Dewata <edewata@redhat.com>2012-06-05 14:46:08 -0500
commit4a012010233b0a5fd613c9d7734ee01be64e3834 (patch)
tree53affd70c3c2b5d0d909641f847b938e80e874c7 /base/common/src/com/netscape/cmscore/policy/PolicyPredicateParser.java
parentc2128339ba046ceb7ab32954ae07dbb4b41d4e73 (diff)
downloadpki-4a012010233b0a5fd613c9d7734ee01be64e3834.tar.gz
pki-4a012010233b0a5fd613c9d7734ee01be64e3834.tar.xz
pki-4a012010233b0a5fd613c9d7734ee01be64e3834.zip
Fixes for NULL_RETURN cases review comments
Diffstat (limited to 'base/common/src/com/netscape/cmscore/policy/PolicyPredicateParser.java')
-rw-r--r--base/common/src/com/netscape/cmscore/policy/PolicyPredicateParser.java16
1 files changed, 7 insertions, 9 deletions
diff --git a/base/common/src/com/netscape/cmscore/policy/PolicyPredicateParser.java b/base/common/src/com/netscape/cmscore/policy/PolicyPredicateParser.java
index 579d9ff5b..09316ceff 100644
--- a/base/common/src/com/netscape/cmscore/policy/PolicyPredicateParser.java
+++ b/base/common/src/com/netscape/cmscore/policy/PolicyPredicateParser.java
@@ -329,15 +329,13 @@ class PredicateTokenizer {
Debug.trace("Malformed Predicate Expression : No Tokens");
throw new EPolicyException("Malformed Predicate Expression : No Tokens");
}
- if (toReturn == null)
- return null;
- else {
- String trimmed = toReturn.trim();
- if (trimmed == null || trimmed.length() == 0)
- return nextToken();
- else
- return trimmed;
- }
+ String trimmed = toReturn.trim();
+
+ if (trimmed.length() == 0)
+ return nextToken();
+ else
+ return trimmed;
+
}
}