Remove implicit this in abstract methods, not the last argument

closes #266

CHANGELOG.md

@@ -1,6 +1,7 @@
 ### 1.3.1 (to be released)
 
 - changed the order in which fields are shown (first static, then instance fields)
+- fixed last argument of abstract methods being removed instead of implicit `this` ([#266](https://github.com/HaxeFoundation/dox/pull/266))
 
 ### 1.3.0 (March 11, 2020)
 

src/dox/Processor.hx

@@ -89,7 +89,7 @@ class Processor {
 								else
 									switch (cf.type) {
 										case CFunction(args, _):
-											args.pop();
+											args.shift();
 										case _:
 									}
 								fields.push(cf);

test/TestClass.hx

@@ -24,7 +24,6 @@
  * 	var foo = bar;
  * ```
  */
-
 class TestClass {
 	static function main() {}
 
@@ -147,6 +146,7 @@ typedef TestTypedef = {
 		And a Second Line `"type":"ItemDetail"`
 	**/
 	var myField:String;
+
 	@:dox(hide) var hiddenField:Bool;
 }
 
@@ -186,6 +186,13 @@ abstract TestEnumAbstract(Int) {
 	var value = 0;
 }
 
+abstract TestAbstract(Int) {
+	/**
+		Method should have `a:Int` and `b:String` arguments, but the implicit `this` argument shouldn't show up.
+	**/
+	public function foo(a:Int, b:String) {}
+}
+
 /**
 	Properties should be sorted alphabetically.
 **/
@@ -265,6 +272,7 @@ class TestMetaData {
 	@:deprecated("Use `red` instead")
 	@:metaWithoutParams
 	public static var deprecatedRed = 0xFF0000;
+
 	@:meta
 	@:to
 	public static var red = 12;
@@ -275,6 +283,7 @@ class TestMetaData {
 	This should not be visible on the index page.
 **/
 class Index {}
+
 @:generic class GenericClass<T> {}
 class ChildOfGeneric extends GenericClass<String> {}