diff options
| author | Ranjit Mathew <rmathew@hotmail.com> | 2005-01-10 18:01:04 +0000 |
|---|---|---|
| committer | Ranjit Mathew <rmathew@gcc.gnu.org> | 2005-01-10 18:01:04 +0000 |
| commit | 5bf8aebc12015bb509e144a6962b29979e9acd2a (patch) | |
| tree | 18c2ff35f5a3bb1a432f042a7cd2a8fff92a19b4 /gcc/java/check-init.c | |
| parent | 3e636396c84f821afdcf405e88fe448228b96901 (diff) | |
| download | gcc-5bf8aebc12015bb509e144a6962b29979e9acd2a.zip gcc-5bf8aebc12015bb509e144a6962b29979e9acd2a.tar.gz gcc-5bf8aebc12015bb509e144a6962b29979e9acd2a.tar.bz2 | |
re PR java/19277 (allows array.length++ although it is final)
PR java/19277
* check-init.c (check_init): Take care of references that do not
have an explicit final variable declaration (e.g. array length
access) for pre/post in/de-crement operators.
From-SVN: r93144
Diffstat (limited to 'gcc/java/check-init.c')
| -rw-r--r-- | gcc/java/check-init.c | 12 |
1 files changed, 10 insertions, 2 deletions
diff --git a/gcc/java/check-init.c b/gcc/java/check-init.c index e124ffc..6dc0dee 100644 --- a/gcc/java/check-init.c +++ b/gcc/java/check-init.c @@ -1,5 +1,6 @@ /* Code to test for "definitive [un]assignment". - Copyright (C) 1999, 2000, 2001, 2003, 2004 Free Software Foundation, Inc. + Copyright (C) 1999, 2000, 2001, 2003, 2004, 2005 Free Software Foundation, + Inc. This file is part of GCC. @@ -832,7 +833,14 @@ check_init (tree exp, words before) case POSTINCREMENT_EXPR: tmp = get_variable_decl (TREE_OPERAND (exp, 0)); if (tmp != NULL_TREE && DECL_FINAL (tmp)) - final_assign_error (DECL_NAME (tmp)); + final_assign_error (DECL_NAME (tmp)); + else if (TREE_CODE (tmp = TREE_OPERAND (exp, 0)) == COMPONENT_REF) + { + /* Take care of array length accesses too. */ + tree decl = TREE_OPERAND (tmp, 1); + if (DECL_FINAL (decl)) + final_assign_error (DECL_NAME (decl)); + } /* Avoid needless recursion. */ exp = TREE_OPERAND (exp, 0); |