Fix for SuperClone and SuperFinalize checks reporting violations on

native methods, #1367

src/main/java/com/puppycrawl/tools/checkstyle/checks/coding/AbstractSuperCheck.java

@@ -229,7 +229,9 @@ public abstract class AbstractSuperCheck
         }
         final DetailAST nameAST = ast.findFirstToken(TokenTypes.IDENT);
         final String name = nameAST.getText();
-        if (!getMethodName().equals(name)) {
+        final DetailAST modifiersAST = ast.findFirstToken(TokenTypes.MODIFIERS);
+        if (!getMethodName().equals(name)
+                || modifiersAST.branchContains(TokenTypes.LITERAL_NATIVE)) {
             return false;
         }
         final DetailAST params = ast.findFirstToken(TokenTypes.PARAMETERS);

src/main/java/com/puppycrawl/tools/checkstyle/checks/coding/SuperCloneCheck.java

@@ -23,6 +23,7 @@ package com.puppycrawl.tools.checkstyle.checks.coding;
 /**
  * <p>
  * Checks that an overriding clone() method invokes super.clone().
+ * Does not check native methods, as they have no possible java defined implementation.
  * </p>
  * <p>
  * Reference:<a

src/main/java/com/puppycrawl/tools/checkstyle/checks/coding/SuperFinalizeCheck.java

@@ -23,6 +23,7 @@ package com.puppycrawl.tools.checkstyle.checks.coding;
 /**
  * <p>
  * Checks that an overriding finalize() method invokes super.finalize().
+ * Does not check native methods, as they have no possible java defined implementation.
  * </p>
  * <p>
  * Reference:<a

src/test/resources/com/puppycrawl/tools/checkstyle/coding/InputClone.java

@@ -93,3 +93,7 @@ class AnotherClass {
 	return null;
     }
 }
+
+class NativeTest {
+    public native Object clone();
+}

src/test/resources/com/puppycrawl/tools/checkstyle/coding/InputFinalize.java

@@ -51,3 +51,7 @@ class MyClassWithGenericSuperMethod1
 
     }
 }
+
+class TestNative {
+    public native void finalize();
+}

src/xdocs/config_coding.xml

@@ -1564,8 +1564,9 @@ if ("something".equals(x))
     <section name="SuperClone">
       <subsection name="Description">
         <p>
-          Checks that an overriding <code>clone()</code> method
-          invokes <code>super.clone()</code>.
+          Checks that an overriding <code>clone()</code> method invokes
+          <code>super.clone()</code>. Does not check native methods, as
+          they have no possible java defined implementation.
         </p>
 
         <p>
@@ -1599,8 +1600,9 @@ if (&quot;something&quot;.equals(x))
     <section name="SuperFinalize">
       <subsection name="Description">
         <p>
-          Checks that an overriding <code>finalize()</code>
-          method invokes <code>super.finalize()</code>.
+          Checks that an overriding <code>finalize()</code> method invokes
+          <code>super.finalize()</code>. Does not check native methods, as
+          they have no possible java defined implementation.
         </p>
 
         <p>