re PR java/6085 (gcj SEGVs on complicated inner class structures)

Fix for PR java/6085:
        * parse.y (patch_method_invocation): Always use build_access_to_thisn
        to get enclosing "this" argument for inner-class constructor
        invocation. Pass correct arguments to build_access_to_thisn.

From-SVN: r52318
This commit is contained in:
Bryce McKinlay 2002-04-15 10:25:29 +01:00
parent fe55a037f6
commit 10aa83c7df
2 changed files with 13 additions and 18 deletions

View File

@ -1,3 +1,10 @@
2002-04-15 Bryce McKinlay <bryce@waitaki.otago.ac.nz>
Fix for PR java/6085:
* parse.y (patch_method_invocation): Always use build_access_to_thisn
to get enclosing "this" argument for inner-class constructor
invocation. Pass correct arguments to build_access_to_thisn.
2002-04-10 Andreas Jaeger <aj@suse.de>
* gcj.texi (Input Options): Fix extdirs patch.
@ -26,6 +33,7 @@
* Make-lang.in: Compile jcf-path with version info for use in
identifying the appropriate libgcj.jar.
>>>>>>> 1.982
2002-04-08 Tom Tromey <tromey@redhat.com>
For PR libgcj/5303:

View File

@ -105,7 +105,6 @@ static void check_inner_class_access PARAMS ((tree, tree, tree));
static int check_pkg_class_access PARAMS ((tree, tree, bool));
static void register_package PARAMS ((tree));
static tree resolve_package PARAMS ((tree, tree *, tree *));
static tree lookup_package_type PARAMS ((const char *, int));
static tree resolve_class PARAMS ((tree, tree, tree, tree));
static void declare_local_variables PARAMS ((int, tree, tree));
static void dump_java_tree PARAMS ((enum tree_dump_index, tree));
@ -10466,27 +10465,15 @@ patch_method_invocation (patch, primary, where, from_super,
/* This handles the situation where a constructor invocation needs
to have an enclosing context passed as a second parameter (the
constructor is one of an inner class. We extract it from the
current function. */
constructor is one of an inner class). */
if ((is_super_init ||
(TREE_CODE (patch) == CALL_EXPR && name == this_identifier_node))
&& PURE_INNER_CLASS_TYPE_P (DECL_CONTEXT (list)))
{
tree enclosing_decl = DECL_CONTEXT (TYPE_NAME (current_class));
tree extra_arg;
if (ANONYMOUS_CLASS_P (current_class) || !DECL_CONTEXT (enclosing_decl))
{
extra_arg = DECL_FUNCTION_BODY (current_function_decl);
extra_arg = TREE_CHAIN (BLOCK_EXPR_DECLS (extra_arg));
}
else
{
tree dest = TREE_TYPE (DECL_CONTEXT (enclosing_decl));
extra_arg =
build_access_to_thisn (TREE_TYPE (enclosing_decl), dest, 0);
extra_arg = java_complete_tree (extra_arg);
}
tree dest = TYPE_NAME (DECL_CONTEXT (list));
tree extra_arg =
build_access_to_thisn (current_class, DECL_CONTEXT (dest), 0);
extra_arg = java_complete_tree (extra_arg);
args = tree_cons (NULL_TREE, extra_arg, args);
}