Code Review
/
controller.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
review
|
tree
raw
|
inline
| side by side
Merge "Refactor configuration for ConfigPersister."
[controller.git]
/
opendaylight
/
config
/
config-api
/
src
/
main
/
java
/
org
/
opendaylight
/
controller
/
config
/
api
/
ValidationException.java
diff --git
a/opendaylight/config/config-api/src/main/java/org/opendaylight/controller/config/api/ValidationException.java
b/opendaylight/config/config-api/src/main/java/org/opendaylight/controller/config/api/ValidationException.java
index f27d12399528c38c144fc17ea0354c243365a1a6..82b73d419e402a31f85a4dfeba7266c26d86b802 100644
(file)
--- a/
opendaylight/config/config-api/src/main/java/org/opendaylight/controller/config/api/ValidationException.java
+++ b/
opendaylight/config/config-api/src/main/java/org/opendaylight/controller/config/api/ValidationException.java
@@
-16,9
+16,11
@@
import java.util.Map.Entry;
/**
* This exception is not intended to be used while implementing modules,
/**
* This exception is not intended to be used while implementing modules,
- * itaggregates validation exceptions and sends them back to the user.
+ * it aggregates validation exceptions and sends them back to the user.
+ * Use {@link org.opendaylight.controller.config.api.JmxAttributeValidationException} for
+ * validating modules instead.
*/
*/
-public class ValidationException extends
Runtime
Exception {
+public class ValidationException extends Exception {
private static final long serialVersionUID = -6072893219820274247L;
private final Map<String/* module name */, Map<String/* instance name */, ExceptionMessageWithStackTrace>> failedValidations;
private static final long serialVersionUID = -6072893219820274247L;
private final Map<String/* module name */, Map<String/* instance name */, ExceptionMessageWithStackTrace>> failedValidations;
@@
-118,23
+120,30
@@
public class ValidationException extends RuntimeException {
@Override
public boolean equals(Object obj) {
@Override
public boolean equals(Object obj) {
- if (this == obj)
+ if (this == obj)
{
return true;
return true;
- if (obj == null)
+ }
+ if (obj == null) {
return false;
return false;
- if (getClass() != obj.getClass())
+ }
+ if (getClass() != obj.getClass()) {
return false;
return false;
+ }
ExceptionMessageWithStackTrace other = (ExceptionMessageWithStackTrace) obj;
if (message == null) {
ExceptionMessageWithStackTrace other = (ExceptionMessageWithStackTrace) obj;
if (message == null) {
- if (other.message != null)
+ if (other.message != null)
{
return false;
return false;
- } else if (!message.equals(other.message))
+ }
+ } else if (!message.equals(other.message)) {
return false;
return false;
+ }
if (stackTrace == null) {
if (stackTrace == null) {
- if (other.stackTrace != null)
+ if (other.stackTrace != null)
{
return false;
return false;
- } else if (!stackTrace.equals(other.stackTrace))
+ }
+ } else if (!stackTrace.equals(other.stackTrace)) {
return false;
return false;
+ }
return true;
}
return true;
}