Simplify QName.toString() by realizing the namespace is guaranteed
to be non-null.
Change-Id: I06d2ab019fe9a9542e4ca3114235af6b6db42db1
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
@Override
public @NonNull String toString() {
@Override
public @NonNull String toString() {
- final StringBuilder sb = new StringBuilder();
- if (getNamespace() != null) {
- sb.append(QNAME_LEFT_PARENTHESIS).append(getNamespace());
-
- final Optional<Revision> rev = getRevision();
- if (rev.isPresent()) {
- sb.append(QNAME_REVISION_DELIMITER).append(rev.get());
- }
- sb.append(QNAME_RIGHT_PARENTHESIS);
+ final StringBuilder sb = new StringBuilder().append(QNAME_LEFT_PARENTHESIS).append(getNamespace());
+ final Optional<Revision> rev = getRevision();
+ if (rev.isPresent()) {
+ sb.append(QNAME_REVISION_DELIMITER).append(rev.get());
- return sb.append(getLocalName()).toString();
+ return sb.append(QNAME_RIGHT_PARENTHESIS).append(getLocalName()).toString();