diff options
author | apbianco <apbianco@138bc75d-0d04-0410-961f-82ee72b054a4> | 2001-07-13 19:43:54 +0000 |
---|---|---|
committer | apbianco <apbianco@138bc75d-0d04-0410-961f-82ee72b054a4> | 2001-07-13 19:43:54 +0000 |
commit | 754a6f79884b27f2ff3c6b919beda0190665a9ce (patch) | |
tree | 1a2ea0dfc79181825aa29dc546dfd81b7294ee46 | |
parent | dd5cfeaff31ed0afa5b7a7d39ec3fc127c2d70e9 (diff) | |
download | gcc-754a6f79884b27f2ff3c6b919beda0190665a9ce.tar.gz |
2001-07-12 Alexandre Petit-Bianco <apbianco@redhat.com>
* parse.y (patch_method_invocation): Don't override primary if one
is already provided, but let this$<n> be built. Fixed comment.
2001-07-12 Alexandre Petit-Bianco <apbianco@redhat.com>
* parse.y (empty_statement:): Report empty statement error only
when found at class declaration level.
Fixes PR java/3635
(http://gcc.gnu.org/ml/gcc-patches/2001-07/msg00885.html )
git-svn-id: svn+ssh://gcc.gnu.org/svn/gcc/trunk@43993 138bc75d-0d04-0410-961f-82ee72b054a4
-rw-r--r-- | gcc/java/ChangeLog | 11 | ||||
-rw-r--r-- | gcc/java/parse.y | 13 |
2 files changed, 21 insertions, 3 deletions
diff --git a/gcc/java/ChangeLog b/gcc/java/ChangeLog index 4648f54f0eb..63015155428 100644 --- a/gcc/java/ChangeLog +++ b/gcc/java/ChangeLog @@ -1,3 +1,14 @@ +2001-07-12 Alexandre Petit-Bianco <apbianco@redhat.com> + + * parse.y (patch_method_invocation): Don't override primary if one + is already provided, but let this$<n> be built. Fixed comment. + +2001-07-12 Alexandre Petit-Bianco <apbianco@redhat.com> + + * parse.y (empty_statement:): Report empty statement error only + when found at class declaration level. + Fixes PR java/3635 + 2001-07-12 Tom Tromey <tromey@redhat.com> * expr.c (expand_load_internal): New function. diff --git a/gcc/java/parse.y b/gcc/java/parse.y index b5cdbdb04ef..03a540c06e0 100644 --- a/gcc/java/parse.y +++ b/gcc/java/parse.y @@ -1453,7 +1453,13 @@ statement_without_trailing_substatement: empty_statement: SC_TK { - if (flag_extraneous_semicolon) + if (flag_extraneous_semicolon + && ! current_static_block + && (! current_function_decl || + /* Verify we're not in a inner class declaration */ + (GET_CPC () != TYPE_NAME + (DECL_CONTEXT (current_function_decl))))) + { EXPR_WFL_SET_LINECOL (wfl_operator, lineno, -1); parse_warning_context (wfl_operator, "An empty declaration is a deprecated feature that should not be used"); @@ -10030,7 +10036,7 @@ patch_method_invocation (patch, primary, where, from_super, - LIST is non static. It's invocation is transformed from x(a1,....,an) into access$<n>(this$<n>,a1,...,an). - LIST is static. It's invocation is transformed from - x(a1,....,an) into TYPEOF(this$<n>).x(a1,....an). + x(a1,....,an) into TYPE_OF(this$<n>).x(a1,....an). Of course, this$<n> can be abitrary complex, ranging from this$0 (the immediate outer context) to @@ -10040,7 +10046,8 @@ patch_method_invocation (patch, primary, where, from_super, this_arg has to be moved into the (then generated) stub argument list. In the meantime, the selected function might have be replaced by a generated stub. */ - if (maybe_use_access_method (is_super_init, &list, &this_arg)) + if (!primary && + maybe_use_access_method (is_super_init, &list, &this_arg)) { args = tree_cons (NULL_TREE, this_arg, args); this_arg = NULL_TREE; /* So it doesn't get chained twice */ |