Skip to content
This repository was archived by the owner on Mar 8, 2020. It is now read-only.

Commit acbb016

Browse files
authored
Merge pull request #119 from bzz/fix-104-method-args-modifers
Fix #104 method args modifers
2 parents 785cd50 + 031a1fc commit acbb016

5 files changed

+1217
-10
lines changed

Diff for: driver/normalizer/normalizer.go

+10-10
Original file line numberDiff line numberDiff line change
@@ -143,16 +143,16 @@ var Normalizers = []Mapping{
143143
"modifiers": Var("ann"), // TODO: it's an array, we should expand it somewhere
144144
"name": Var("name"),
145145
"body": Var("body"),
146-
"parameters": Each("args", Obj{
147-
uast.KeyType: String("SingleVariableDeclaration"),
148-
uast.KeyPos: Var("apos"),
149-
"extraDimensions2": Is(nil),
150-
"initializer": Var("ainit"),
151-
"modifiers": Is(nil),
152-
"name": Var("aname"),
153-
"type": Var("atype"),
154-
"varargs": Cases("varg", String("false"), String("true")),
155-
"varargsAnnotations": Is(nil),
146+
"parameters": Each("args", Fields{
147+
{Name: uast.KeyType, Op: String("SingleVariableDeclaration")},
148+
{Name: uast.KeyPos, Op: Var("apos")},
149+
{Name: "extraDimensions2", Op: Is(nil)},
150+
{Name: "initializer", Op: Var("ainit")},
151+
{Name: "modifiers", Op: Any(), Drop: true}, // FIXME: preserve this array
152+
{Name: "name", Op: Var("aname")},
153+
{Name: "type", Op: Var("atype")},
154+
{Name: "varargs", Op: Cases("varg", String("false"), String("true"))},
155+
{Name: "varargsAnnotations", Op: Is(nil)},
156156
}),
157157
"receiverQualifier": Var("recv_name"),
158158
"receiverType": Cases("recv",

Diff for: fixtures/issue104_final_args.java

+6
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,6 @@
1+
package pkg;
2+
3+
class C {
4+
public void m1(final String str1) {}
5+
public void m2(String str2) {}
6+
}

0 commit comments

Comments
 (0)