No need for a local variable, and use proper log formatting instead of
string concat.
Change-Id: Id5fd1dba61cf984a3aa08f4b2b7ee367ff24e307
Signed-off-by: Robert Varga <robert.varga@pantheon.sk>
package org.opendaylight.yangtools.yang.parser.stmt.rfc6020;
import static org.opendaylight.yangtools.yang.parser.spi.meta.StmtContextUtils.firstAttributeOf;
package org.opendaylight.yangtools.yang.parser.stmt.rfc6020;
import static org.opendaylight.yangtools.yang.parser.spi.meta.StmtContextUtils.firstAttributeOf;
import com.google.common.base.CharMatcher;
import com.google.common.base.Preconditions;
import com.google.common.base.Splitter;
import com.google.common.base.CharMatcher;
import com.google.common.base.Preconditions;
import com.google.common.base.Splitter;
}
public static Status parseStatus(final String value) {
}
public static Status parseStatus(final String value) {
-
- Status status = null;
switch (value) {
case "current":
switch (value) {
case "current":
- status = Status.CURRENT;
- break;
- status = Status.DEPRECATED;
- break;
+ return Status.DEPRECATED;
- status = Status.OBSOLETE;
- break;
+ return Status.OBSOLETE;
- LOG.warn("Invalid 'status' statement: " + value);
+ LOG.warn("Invalid 'status' statement: {}", value);
+ return null;
}
public static Date getLatestRevision(final Iterable<? extends StmtContext<?, ?, ?>> subStmts) {
}
public static Date getLatestRevision(final Iterable<? extends StmtContext<?, ?, ?>> subStmts) {