From 400b7cad3487756bd658b05202a7e3c3efe10e71 Mon Sep 17 00:00:00 2001
From: Robert Varga <robert.varga@pantheon.tech>
Date: Tue, 17 Apr 2018 19:26:39 +0200
Subject: [PATCH] Fix checkstyle in yang-parser-api

Fix the few violations and start enforcing checkstyle.

Change-Id: I04929e336d369265969989082e42db61c2a485ef
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
---
 yang/yang-parser-api/pom.xml                          | 11 +++++++++++
 .../yangtools/yang/model/parser/api/YangParser.java   |  6 +++---
 2 files changed, 14 insertions(+), 3 deletions(-)

diff --git a/yang/yang-parser-api/pom.xml b/yang/yang-parser-api/pom.xml
index 4ae63ce276..36f875dde9 100644
--- a/yang/yang-parser-api/pom.xml
+++ b/yang/yang-parser-api/pom.xml
@@ -43,4 +43,15 @@
         </dependency>
     </dependencies>
 
+    <build>
+        <plugins>
+            <plugin>
+                <groupId>org.apache.maven.plugins</groupId>
+                <artifactId>maven-checkstyle-plugin</artifactId>
+                <configuration>
+                    <propertyExpansion>checkstyle.violationSeverity=error</propertyExpansion>
+                </configuration>
+            </plugin>
+        </plugins>
+    </build>
 </project>
diff --git a/yang/yang-parser-api/src/main/java/org/opendaylight/yangtools/yang/model/parser/api/YangParser.java b/yang/yang-parser-api/src/main/java/org/opendaylight/yangtools/yang/model/parser/api/YangParser.java
index 1560104c19..2905ab02b1 100644
--- a/yang/yang-parser-api/src/main/java/org/opendaylight/yangtools/yang/model/parser/api/YangParser.java
+++ b/yang/yang-parser-api/src/main/java/org/opendaylight/yangtools/yang/model/parser/api/YangParser.java
@@ -57,7 +57,7 @@ public interface YangParser {
      * @throws IOException when an IO error occurs
      * @throws IllegalArgumentException if the representation is not supported
      */
-    YangParser addSource(final SchemaSourceRepresentation source) throws IOException, YangSyntaxErrorException;
+    YangParser addSource(SchemaSourceRepresentation source) throws IOException, YangSyntaxErrorException;
 
     /**
      * Add main sources. All main sources are present in resulting SchemaContext.
@@ -123,7 +123,7 @@ public interface YangParser {
      *            Set of supported features in the final SchemaContext.
      *            If the set is empty, no features encountered will be supported.
      */
-    YangParser setSupportedFeatures(@Nonnull final Set<QName> supportedFeatures);
+    YangParser setSupportedFeatures(@Nonnull Set<QName> supportedFeatures);
 
     /**
      * Set YANG modules which can be deviated by specified modules during the parsing process.
@@ -158,7 +158,7 @@ public interface YangParser {
     }
 
     /**
-     * Build effective {@link SchemaContext}
+     * Build effective {@link SchemaContext}.
      *
      * @return An effective schema context comprised of configured models.
      * @throws YangSyntaxErrorException When a syntactic error is encountered.
-- 
2.36.6