sem: drop MethodDeclaration->parameters->modifiers

Instead of failing whole trasformation by assuming it's always nil, when it's not. Signed-off-by: Alexander Bezzubov <[email protected]>
bblfsh · May 6, 2019 · f737e60 · f737e60
1 parent f8753dc
commit f737e60
Show file tree

Hide file tree

Showing 5 changed files with 1,208 additions and 1 deletion.
diff --git a/driver/normalizer/normalizer.go b/driver/normalizer/normalizer.go
@@ -148,7 +148,7 @@ var Normalizers = []Mapping{
 				{Name: uast.KeyPos, Op: Var("apos")},
 				{Name: "extraDimensions2", Op: Is(nil)},
 				{Name: "initializer", Op: Var("ainit")},
-				{Name: "modifiers", Op: Is(nil)},
+				{Name: "modifiers", Op: Any(), Drop: true}, //TODO: preserve this array
 				{Name: "name", Op: Var("aname")},
 				{Name: "type", Op: Var("atype")},
 				{Name: "varargs", Op: Cases("varg", String("false"), String("true"))},

diff --git a/fixtures/issue104_final_args.java b/fixtures/issue104_final_args.java
@@ -0,0 +1,6 @@
+package pkg;
+
+class C {
+  public void m1(final String str1) {}
+  public void m2(String str2) {}
+}