Fix more "Attempt to push null on operand stack" variantsHEADmaster

This change fixes more cases in the *Declaration classes that generate code.
Both #336 and #337 are due to the same piece of code not correctly computing
the resolved position because of copying it from a source method when it needs
recomputing for the method being generated. In particular primitive types
and/or double slot types (like long or double) cause the code to miscompute
the resolved positions or set the wrong type of expected type on the stack.

Fixes #336
Fixes #337

1 files changed, 17 insertions, 0 deletions

diff --git a/tests/bugs1924/336/Bang.java b/tests/bugs1924/336/Bang.java
new file mode 100644
index 000000000..276484c51
--- /dev/null
+++ b/tests/bugs1924/336/Bang.java
@@ -0,0 +1,17 @@
+public class Bang {
+
+public static void main(String[] argv) {
+  new Bang().m("a",1,"b");
+}
+
+  public int m(String a, int i, String b) {
+    return 42;
+  }
+
+}
+
+aspect X {
+  int around(String a, int b, String d): execution(* m(..)) && args(a,b,d) {
+	return proceed(a,b,d);
+  }
+}