diff options
| author | Jakub Jelinek <jakub@redhat.com> | 2020-04-17 16:59:57 +0200 |
|---|---|---|
| committer | Jakub Jelinek <jakub@redhat.com> | 2020-04-17 16:59:57 +0200 |
| commit | 2e3897490e0f99b22a2813cfb34d59a1ea71ff68 (patch) | |
| tree | 694c8b8b270f33ea3477a22df070bd0366ebf6fb /gcc/cp/call.c | |
| parent | 1dfc50232dcb703454db4f54c538042a32be2138 (diff) | |
| download | gcc-2e3897490e0f99b22a2813cfb34d59a1ea71ff68.zip gcc-2e3897490e0f99b22a2813cfb34d59a1ea71ff68.tar.gz gcc-2e3897490e0f99b22a2813cfb34d59a1ea71ff68.tar.bz2 | |
c, c++: Fix two redundantAssignment warnings [PR94629]
This change fixes two obvious redundant assignments reported by cppcheck:
trunk.git/gcc/c/c-parser.c:16969:2: style: Variable 'data.clauses' is reassigned a value before the old one has been used. [redundantAssignment]
trunk.git/gcc/cp/call.c:5116:9: style: Variable 'arg2' is reassigned a value before the old one has been used. [redundantAssignment]
2020-04-17 Jakub Jelinek <jakub@redhat.com>
PR other/94629
* c-parser.c (c_parser_oacc_routine): Remove redundant assignment
to data.clauses.
* call.c (build_conditional_expr_1): Remove redundant assignment to
arg2.
Diffstat (limited to 'gcc/cp/call.c')
| -rw-r--r-- | gcc/cp/call.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/gcc/cp/call.c b/gcc/cp/call.c index 1f3d9d2..c582316 100644 --- a/gcc/cp/call.c +++ b/gcc/cp/call.c @@ -5112,7 +5112,7 @@ build_conditional_expr_1 (const op_location_t &loc, /* Make sure that lvalues remain lvalues. See g++.oliva/ext1.C. */ if (glvalue_p (arg1)) { - arg2 = arg1 = cp_stabilize_reference (arg1); + arg1 = cp_stabilize_reference (arg1); arg2 = arg1 = prevent_lifetime_extension (arg1); } else |