diff options
author | Maros Marsalek <mmarsale@cisco.com> | 2016-09-05 12:10:26 +0200 |
---|---|---|
committer | Marek Gradzki <mgradzki@cisco.com> | 2016-09-07 06:50:30 +0000 |
commit | 928d0b286b586c76658449480291b9445a52b74b (patch) | |
tree | 8e189476429a969b07f48bcf8ae9f8a693b444b0 /lisp/lisp2vpp/src/main/java/io/fd/honeycomb/lisp/translate | |
parent | 22c537234245863509e987f7781d3d769045a79b (diff) |
Cleanup TODOs and FIXMEs
- Fix minor ones
- Report bigger and include issue number in comment
- Pull common dependencies into dependency management of common/parents
Change-Id: I06a6ac37c52b603fd73ed42023d6b2e7fa18010f
Signed-off-by: Maros Marsalek <mmarsale@cisco.com>
Diffstat (limited to 'lisp/lisp2vpp/src/main/java/io/fd/honeycomb/lisp/translate')
4 files changed, 6 insertions, 9 deletions
diff --git a/lisp/lisp2vpp/src/main/java/io/fd/honeycomb/lisp/translate/read/AdjacencyCustomizer.java b/lisp/lisp2vpp/src/main/java/io/fd/honeycomb/lisp/translate/read/AdjacencyCustomizer.java index 1a9721ca0..50baf6b0e 100755 --- a/lisp/lisp2vpp/src/main/java/io/fd/honeycomb/lisp/translate/read/AdjacencyCustomizer.java +++ b/lisp/lisp2vpp/src/main/java/io/fd/honeycomb/lisp/translate/read/AdjacencyCustomizer.java @@ -66,7 +66,7 @@ public class AdjacencyCustomizer extends FutureJVppCustomizer public void readCurrentAttributes(@Nonnull final InstanceIdentifier<Adjacency> id, @Nonnull final AdjacencyBuilder builder, @Nonnull final ReadContext ctx) throws ReadFailedException { - //TODO - finish after https://jira.fd.io/browse/VPP-362 + // TODO - finish after https://jira.fd.io/browse/VPP-362 throw new ReadFailedException(id, new OperationNotSupportedException("Operation not supported")); } } diff --git a/lisp/lisp2vpp/src/main/java/io/fd/honeycomb/lisp/translate/read/factory/LocatorSetsReaderFactory.java b/lisp/lisp2vpp/src/main/java/io/fd/honeycomb/lisp/translate/read/factory/LocatorSetsReaderFactory.java index 3d0801969..cc500e159 100755 --- a/lisp/lisp2vpp/src/main/java/io/fd/honeycomb/lisp/translate/read/factory/LocatorSetsReaderFactory.java +++ b/lisp/lisp2vpp/src/main/java/io/fd/honeycomb/lisp/translate/read/factory/LocatorSetsReaderFactory.java @@ -34,7 +34,7 @@ import org.openvpp.jvpp.core.future.FutureJVppCore; /** - * Produces reader for {@link LocatorSets} <br> and all its inhired child readers + * Produces reader for {@link LocatorSets} <br> and all its inhired child readers. */ public class LocatorSetsReaderFactory extends AbstractLispReaderFactoryBase implements ReaderFactory { @@ -42,8 +42,7 @@ public class LocatorSetsReaderFactory extends AbstractLispReaderFactoryBase impl private LocatorSetsReaderFactory(final InstanceIdentifier<LispState> lispStateId, final FutureJVppCore vppApi, final NamingContext interfaceContext, - final NamingContext locatorSetContext - ) { + final NamingContext locatorSetContext) { super(lispStateId, vppApi); this.interfaceContext = interfaceContext; this.locatorSetContext = locatorSetContext; diff --git a/lisp/lisp2vpp/src/main/java/io/fd/honeycomb/lisp/translate/util/EidConverter.java b/lisp/lisp2vpp/src/main/java/io/fd/honeycomb/lisp/translate/util/EidConverter.java index 39d0fe7f0..b0ca7fc04 100755 --- a/lisp/lisp2vpp/src/main/java/io/fd/honeycomb/lisp/translate/util/EidConverter.java +++ b/lisp/lisp2vpp/src/main/java/io/fd/honeycomb/lisp/translate/util/EidConverter.java @@ -206,7 +206,6 @@ public final class EidConverter { } else if (address instanceof Mac) { return MAC; } else { - //TODO - other types throw new IllegalStateException("Unknown type detected"); } } @@ -256,7 +255,6 @@ public final class EidConverter { case MAC: return TranslateUtils.parseMac(((Mac) address).getMac().getValue()); default: - //TODO - other types throw new IllegalArgumentException("Unsupported type"); } } diff --git a/lisp/lisp2vpp/src/main/java/io/fd/honeycomb/lisp/translate/write/LocatorSetCustomizer.java b/lisp/lisp2vpp/src/main/java/io/fd/honeycomb/lisp/translate/write/LocatorSetCustomizer.java index c2db3b845..d6eaf4f68 100755 --- a/lisp/lisp2vpp/src/main/java/io/fd/honeycomb/lisp/translate/write/LocatorSetCustomizer.java +++ b/lisp/lisp2vpp/src/main/java/io/fd/honeycomb/lisp/translate/write/LocatorSetCustomizer.java @@ -47,7 +47,7 @@ import org.openvpp.jvpp.core.future.FutureJVppCore; /** - * Customizer for {@link LocatorSet} entity + * Customizer for {@link LocatorSet} entity. * * @see LocatorSet */ @@ -77,7 +77,7 @@ public class LocatorSetCustomizer extends FutureJVppCustomizer final String locatorSetName = dataAfter.getName(); checkNotNull(locatorSetName, "LocatorSet name is null"); - //TODO check and fill mapping when api returns index of created locator set + // TODO VPP-323 check and fill mapping when api returns index of created locator set // checkState(!locatorSetContext.containsIndex(locatorSetName, writeContext.getMappingContext()), // "Locator set with name %s allready defined", locatorSetName); @@ -102,7 +102,7 @@ public class LocatorSetCustomizer extends FutureJVppCustomizer public void updateCurrentAttributes(@Nonnull InstanceIdentifier<LocatorSet> id, @Nonnull LocatorSet dataBefore, @Nonnull LocatorSet dataAfter, - WriteContext writeContext) throws WriteFailedException { + @Nonnull WriteContext writeContext) throws WriteFailedException { throw new UnsupportedOperationException("Operation not supported"); } |