Assigned bugs to FIXMEs.

[bgpcep.git] / pcep / spi / src / main / java / org / opendaylight / protocol / pcep / spi / pojo / SimpleObjectHandlerRegistry.java

diff --git a/pcep/spi/src/main/java/org/opendaylight/protocol/pcep/spi/pojo/SimpleObjectHandlerRegistry.java b/pcep/spi/src/main/java/org/opendaylight/protocol/pcep/spi/pojo/SimpleObjectHandlerRegistry.java
index 6d7a7121c40e3828d365fc922f2db847bc8d260d..5fbb0e8262567bbb474fa500e7b1406470095592 100644 (file)
--- a/pcep/spi/src/main/java/org/opendaylight/protocol/pcep/spi/pojo/SimpleObjectHandlerRegistry.java
+++ b/pcep/spi/src/main/java/org/opendaylight/protocol/pcep/spi/pojo/SimpleObjectHandlerRegistry.java
@@ -8,11 +8,12 @@
 package org.opendaylight.protocol.pcep.spi.pojo;
 
 import org.opendaylight.protocol.concepts.HandlerRegistry;
-import org.opendaylight.protocol.pcep.UnknownObject;
 import org.opendaylight.protocol.pcep.spi.ObjectHandlerRegistry;
 import org.opendaylight.protocol.pcep.spi.ObjectParser;
 import org.opendaylight.protocol.pcep.spi.ObjectSerializer;
-import org.opendaylight.protocol.util.Util;
+import org.opendaylight.protocol.pcep.spi.PCEPErrors;
+import org.opendaylight.protocol.pcep.spi.UnknownObject;
+import org.opendaylight.protocol.util.Values;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.pcep.types.rev131005.Object;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.pcep.types.rev131005.ObjectHeader;
 import org.opendaylight.yangtools.yang.binding.DataContainer;
@@ -26,13 +27,13 @@ public final class SimpleObjectHandlerRegistry implements ObjectHandlerRegistry
        private final HandlerRegistry<DataContainer, ObjectParser, ObjectSerializer> handlers = new HandlerRegistry<>();
 
        private static int createKey(final int objectClass, final int objectType) {
-               Preconditions.checkArgument(objectClass >= 0 && objectClass <= Util.UNSIGNED_BYTE_MAX_VALUE);
+               Preconditions.checkArgument(objectClass >= 0 && objectClass <= Values.UNSIGNED_BYTE_MAX_VALUE);
                Preconditions.checkArgument(objectType >= 0 && objectType <= 15);
                return (objectClass << 4) | objectType;
        }
 
        public AutoCloseable registerObjectParser(final int objectClass, final int objectType, final ObjectParser parser) {
-               Preconditions.checkArgument(objectClass >= 0 && objectClass <= Util.UNSIGNED_BYTE_MAX_VALUE);
+               Preconditions.checkArgument(objectClass >= 0 && objectClass <= Values.UNSIGNED_BYTE_MAX_VALUE);
                Preconditions.checkArgument(objectType >= 0 && objectType <= 15);
                return this.handlers.registerParser(createKey(objectClass, objectType), parser);
        }
@@ -48,26 +49,22 @@ public final class SimpleObjectHandlerRegistry implements ObjectHandlerRegistry
                        return ret;
                }
 
-               boolean foundClass = false;
+               final boolean foundClass = false;
 
-               // FIXME: search the parsers, check classes
-               //e.getError() == PCEPErrors.UNRECOGNIZED_OBJ_CLASS || e.getError() == PCEPErrors.UNRECOGNIZED_OBJ_TYPE
+               // FIXME: BUG-187: search the parsers, check classes
 
                if (!foundClass) {
                        return new ObjectParser() {
                                @Override
                                public Object parseObject(final ObjectHeader header, final byte[] buffer) {
-                                       // FIXME: appropriate error (unrecognized object class)
-                                       return new UnknownObject(null);
+                                       return new UnknownObject(PCEPErrors.UNRECOGNIZED_OBJ_CLASS);
                                }
                        };
-
                } else {
                        return new ObjectParser() {
                                @Override
                                public Object parseObject(final ObjectHeader header, final byte[] buffer) {
-                                       // FIXME: appropriate error (unrecognized object type)
-                                       return new UnknownObject(null);
+                                       return new UnknownObject(PCEPErrors.UNRECOGNIZED_OBJ_TYPE);
                                }
                        };
                }