Fix eclipse/checkstyle warnings

[yangtools.git] / yang / yang-common / src / main / java / org / opendaylight / yangtools / yang / common / RpcResultBuilder.java

diff --git a/yang/yang-common/src/main/java/org/opendaylight/yangtools/yang/common/RpcResultBuilder.java b/yang/yang-common/src/main/java/org/opendaylight/yangtools/yang/common/RpcResultBuilder.java
index 7b5ef531c9033d191146902dd14b8cc1a292d1de..be6801fc86580bea267a2130adc47ae93928c516 100644 (file)
--- a/yang/yang-common/src/main/java/org/opendaylight/yangtools/yang/common/RpcResultBuilder.java
+++ b/yang/yang-common/src/main/java/org/opendaylight/yangtools/yang/common/RpcResultBuilder.java
@@ -34,8 +34,8 @@ public final class RpcResultBuilder<T> implements Builder<RpcResult<T>> {
         private final T result;
         private final boolean successful;
 
-        RpcResultImpl( final boolean successful, final T result,
-                       final Collection<RpcError> errors ) {
+        RpcResultImpl(final boolean successful, final T result,
+                       final Collection<RpcError> errors) {
             this.successful = successful;
             this.result = result;
             this.errors = errors;
@@ -74,9 +74,9 @@ public final class RpcResultBuilder<T> implements Builder<RpcResult<T>> {
         private final ErrorType errorType;
         private final Throwable cause;
 
-        RpcErrorImpl( final ErrorSeverity severity, final ErrorType errorType,
+        RpcErrorImpl(final ErrorSeverity severity, final ErrorType errorType,
                 final String tag, final String message, final String applicationTag, final String info,
-                final Throwable cause ) {
+                final Throwable cause) {
             this.severity = severity;
             this.errorType = errorType;
             this.tag = tag;
@@ -134,7 +134,7 @@ public final class RpcResultBuilder<T> implements Builder<RpcResult<T>> {
     private T result;
     private final boolean successful;
 
-    private RpcResultBuilder( final boolean successful, final T result ) {
+    private RpcResultBuilder(final boolean successful, final T result) {
         this.successful = successful;
         this.result = result;
     }
@@ -151,7 +151,7 @@ public final class RpcResultBuilder<T> implements Builder<RpcResult<T>> {
      *
      * @param result the result value
      */
-    public static <T> RpcResultBuilder<T> success( final T result ) {
+    public static <T> RpcResultBuilder<T> success(final T result) {
         return new RpcResultBuilder<>(true, result);
     }
 
@@ -160,7 +160,7 @@ public final class RpcResultBuilder<T> implements Builder<RpcResult<T>> {
      *
      * @param builder builder for the result value
      */
-    public static <T> RpcResultBuilder<T> success( final Builder<T> builder ) {
+    public static <T> RpcResultBuilder<T> success(final Builder<T> builder) {
         return success(builder.build());
     }
 
@@ -176,7 +176,7 @@ public final class RpcResultBuilder<T> implements Builder<RpcResult<T>> {
      *
      * @param success true if successful, false otherwise.
      */
-    public static <T> RpcResultBuilder<T> status( final boolean success ) {
+    public static <T> RpcResultBuilder<T> status(final boolean success) {
         return new RpcResultBuilder<>(success, null);
     }
 
@@ -185,9 +185,9 @@ public final class RpcResultBuilder<T> implements Builder<RpcResult<T>> {
      *
      * @param other the other RpcResult.
      */
-    public static <T> RpcResultBuilder<T> from( final RpcResult<T> other ) {
+    public static <T> RpcResultBuilder<T> from(final RpcResult<T> other) {
         return new RpcResultBuilder<>(other.isSuccessful(), other.getResult())
-                                                      .withRpcErrors( other.getErrors() );
+                                                      .withRpcErrors(other.getErrors());
     }
 
     /**
@@ -200,9 +200,9 @@ public final class RpcResultBuilder<T> implements Builder<RpcResult<T>> {
      *
      * @return an RpcError
      */
-    public static RpcError newError( final ErrorType errorType, final String tag, final String message ) {
-        return new RpcErrorImpl( ErrorSeverity.ERROR, errorType,
-                tag != null ? tag : "operation-failed", message, null, null, null );
+    public static RpcError newError(final ErrorType errorType, final String tag, final String message) {
+        return new RpcErrorImpl(ErrorSeverity.ERROR, errorType,
+                tag != null ? tag : "operation-failed", message, null, null, null);
     }
 
     /**
@@ -212,17 +212,17 @@ public final class RpcResultBuilder<T> implements Builder<RpcResult<T>> {
      * @param tag a short string that identifies the general type of error condition. See
      *        {@link RpcError#getTag} for a list of suggested values.
      * @param message a string suitable for human display that describes the error condition.
-     * * @param applicationTag a short string that identifies the specific type of error condition.
+     * @param applicationTag a short string that identifies the specific type of error condition.
      * @param info a string containing additional information to provide extended
      *        and/or implementation-specific debugging information.
      * @param cause the exception that triggered the error.
      *
      * @return an RpcError
      */
-    public static RpcError newError(  final ErrorType errorType, final String tag, final String message,
-            final String applicationTag, final String info, final Throwable cause ) {
-        return new RpcErrorImpl( ErrorSeverity.ERROR, errorType,
-                tag != null ? tag : "operation-failed", message, applicationTag, info, cause );
+    public static RpcError newError(final ErrorType errorType, final String tag, final String message,
+            final String applicationTag, final String info, final Throwable cause) {
+        return new RpcErrorImpl(ErrorSeverity.ERROR, errorType,
+                tag != null ? tag : "operation-failed", message, applicationTag, info, cause);
     }
 
     /**
@@ -235,8 +235,8 @@ public final class RpcResultBuilder<T> implements Builder<RpcResult<T>> {
      *
      * @return an RpcError
      */
-    public static RpcError newWarning( final ErrorType errorType, final String tag, final String message ) {
-        return new RpcErrorImpl( ErrorSeverity.WARNING, errorType, tag, message, null, null, null );
+    public static RpcError newWarning(final ErrorType errorType, final String tag, final String message) {
+        return new RpcErrorImpl(ErrorSeverity.WARNING, errorType, tag, message, null, null, null);
     }
 
     /**
@@ -246,17 +246,17 @@ public final class RpcResultBuilder<T> implements Builder<RpcResult<T>> {
      * @param tag a short string that identifies the general type of warning condition. See
      *        {@link RpcError#getTag} for a list of suggested values.
      * @param message a string suitable for human display that describes the warning condition.
-     * * @param applicationTag a short string that identifies the specific type of warning condition.
+     * @param applicationTag a short string that identifies the specific type of warning condition.
      * @param info a string containing additional information to provide extended
      *        and/or implementation-specific debugging information.
      * @param cause the exception that triggered the warning.
      *
      * @return an RpcError
      */
-    public static RpcError newWarning(  final ErrorType errorType, final String tag, final String message,
-            final String applicationTag, final String info, final Throwable cause ) {
-        return new RpcErrorImpl( ErrorSeverity.WARNING, errorType, tag, message,
-                                 applicationTag, info, cause );
+    public static RpcError newWarning(final ErrorType errorType, final String tag, final String message,
+            final String applicationTag, final String info, final Throwable cause) {
+        return new RpcErrorImpl(ErrorSeverity.WARNING, errorType, tag, message,
+                                 applicationTag, info, cause);
     }
 
     /**
@@ -264,7 +264,7 @@ public final class RpcResultBuilder<T> implements Builder<RpcResult<T>> {
      *
      * @param result the result value
      */
-    public RpcResultBuilder<T> withResult( final T result ) {
+    public RpcResultBuilder<T> withResult(final T result) {
         this.result = result;
         return this;
     }
@@ -274,26 +274,26 @@ public final class RpcResultBuilder<T> implements Builder<RpcResult<T>> {
      *
      * @param builder builder for the result value
      */
-    public RpcResultBuilder<T> withResult( final Builder<T> builder ) {
+    public RpcResultBuilder<T> withResult(final Builder<T> builder) {
         return withResult(builder.build());
     }
 
-    private void addError( final ErrorSeverity severity, final ErrorType errorType,
+    private void addError(final ErrorSeverity severity, final ErrorType errorType,
             final String tag, final String message, final String applicationTag, final String info,
-            final Throwable cause ) {
+            final Throwable cause) {
 
-        addError( new RpcErrorImpl( severity, errorType,
+        addError(new RpcErrorImpl(severity, errorType,
                                     tag != null ? tag : "operation-failed", message,
-                                    applicationTag, info, cause ) );
+                                    applicationTag, info, cause));
     }
 
-    private void addError( final RpcError error ) {
+    private void addError(final RpcError error) {
 
         if (errors == null) {
             errors = new ImmutableList.Builder<>();
         }
 
-        errors.add( error );
+        errors.add(error);
     }
 
     /**
@@ -304,8 +304,8 @@ public final class RpcResultBuilder<T> implements Builder<RpcResult<T>> {
      *        {@link RpcError#getTag} for a list of suggested values.
      * @param message a string suitable for human display that describes the warning condition.
      */
-    public RpcResultBuilder<T> withWarning( final ErrorType errorType, final String tag, final String message ) {
-        addError( ErrorSeverity.WARNING, errorType, tag, message, null, null, null );
+    public RpcResultBuilder<T> withWarning(final ErrorType errorType, final String tag, final String message) {
+        addError(ErrorSeverity.WARNING, errorType, tag, message, null, null, null);
         return this;
     }
 
@@ -321,9 +321,9 @@ public final class RpcResultBuilder<T> implements Builder<RpcResult<T>> {
      *        and/or implementation-specific debugging information.
      * @param cause the exception that triggered the warning.
      */
-    public RpcResultBuilder<T> withWarning( final ErrorType errorType, final String tag, final String message,
-            final String applicationTag, final String info, final Throwable cause ) {
-        addError( ErrorSeverity.WARNING, errorType, tag, message, applicationTag, info, cause );
+    public RpcResultBuilder<T> withWarning(final ErrorType errorType, final String tag, final String message,
+            final String applicationTag, final String info, final Throwable cause) {
+        addError(ErrorSeverity.WARNING, errorType, tag, message, applicationTag, info, cause);
         return this;
     }
 
@@ -333,8 +333,8 @@ public final class RpcResultBuilder<T> implements Builder<RpcResult<T>> {
      * @param errorType the conceptual layer at which the error occurred.
      * @param message a string suitable for human display that describes the error condition.
      */
-    public RpcResultBuilder<T> withError( final ErrorType errorType, final String message ) {
-        addError( ErrorSeverity.ERROR, errorType, null, message, null, null, null );
+    public RpcResultBuilder<T> withError(final ErrorType errorType, final String message) {
+        addError(ErrorSeverity.ERROR, errorType, null, message, null, null, null);
         return this;
     }
 
@@ -346,8 +346,8 @@ public final class RpcResultBuilder<T> implements Builder<RpcResult<T>> {
      *        {@link RpcError#getTag} for a list of suggested values.
      * @param message a string suitable for human display that describes the error condition.
      */
-    public RpcResultBuilder<T> withError( final ErrorType errorType, final String tag, final String message ) {
-        addError( ErrorSeverity.ERROR, errorType, tag, message, null, null, null );
+    public RpcResultBuilder<T> withError(final ErrorType errorType, final String tag, final String message) {
+        addError(ErrorSeverity.ERROR, errorType, tag, message, null, null, null);
         return this;
     }
 
@@ -358,9 +358,9 @@ public final class RpcResultBuilder<T> implements Builder<RpcResult<T>> {
      * @param message a string suitable for human display that describes the error condition.
      * @param cause the exception that triggered the error.
      */
-    public RpcResultBuilder<T> withError( final ErrorType errorType, final String message,
-                                          final Throwable cause ) {
-        addError( ErrorSeverity.ERROR, errorType, null, message, null, null, cause );
+    public RpcResultBuilder<T> withError(final ErrorType errorType, final String message,
+                                          final Throwable cause) {
+        addError(ErrorSeverity.ERROR, errorType, null, message, null, null, cause);
         return this;
     }
 
@@ -376,9 +376,9 @@ public final class RpcResultBuilder<T> implements Builder<RpcResult<T>> {
      *        and/or implementation-specific debugging information.
      * @param cause the exception that triggered the error.
      */
-    public RpcResultBuilder<T> withError( final ErrorType errorType, final String tag, final String message,
-            final String applicationTag, final String info, final Throwable cause ) {
-        addError( ErrorSeverity.ERROR, errorType, tag, message, applicationTag, info, cause );
+    public RpcResultBuilder<T> withError(final ErrorType errorType, final String tag, final String message,
+            final String applicationTag, final String info, final Throwable cause) {
+        addError(ErrorSeverity.ERROR, errorType, tag, message, applicationTag, info, cause);
         return this;
     }
 
@@ -387,8 +387,8 @@ public final class RpcResultBuilder<T> implements Builder<RpcResult<T>> {
      *
      * @param error the RpcError
      */
-    public RpcResultBuilder<T> withRpcError( final RpcError error ) {
-        addError( error );
+    public RpcResultBuilder<T> withRpcError(final RpcError error) {
+        addError(error);
         return this;
     }
 
@@ -397,10 +397,10 @@ public final class RpcResultBuilder<T> implements Builder<RpcResult<T>> {
      *
      * @param errors the list of RpcErrors
      */
-    public RpcResultBuilder<T> withRpcErrors( final Collection<RpcError> errors ) {
+    public RpcResultBuilder<T> withRpcErrors(final Collection<RpcError> errors) {
         if (errors != null) {
             for (RpcError error : errors) {
-                addError( error );
+                addError(error);
             }
         }
         return this;
@@ -416,14 +416,13 @@ public final class RpcResultBuilder<T> implements Builder<RpcResult<T>> {
     /**
      * Builds RpcResult and wraps it in a Future
      *
+     * <p>
      * This is a convenience method to assist those writing rpcs
      * that produce immediate results.  It allows you to replace
      *
-     * Futures.immediateFuture(rpcResult.build())
-     *
+     * {@code Futures.immediateFuture(rpcResult.build())}
      * with
-     *
-     * rpcResult.buildFuture();
+     * {@code rpcResult.buildFuture();}
      *
      * @return Future for RpcResult built by RpcResultBuilder
      *