public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
* [Bug c++/11957] New: wrong "warning: statement has no effect"
@ 2003-08-17 19:18 ritzert at t-online dot de
2003-08-17 19:21 ` [Bug c++/11957] " pinskia at gcc dot gnu dot org
` (6 more replies)
0 siblings, 7 replies; 8+ messages in thread
From: ritzert at t-online dot de @ 2003-08-17 19:18 UTC (permalink / raw)
To: gcc-bugs
PLEASE REPLY TO gcc-bugzilla@gcc.gnu.org ONLY, *NOT* gcc-bugs@gcc.gnu.org.
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=11957
Summary: wrong "warning: statement has no effect"
Product: gcc
Version: 3.4
Status: UNCONFIRMED
Severity: normal
Priority: P2
Component: c++
AssignedTo: unassigned at gcc dot gnu dot org
ReportedBy: ritzert at t-online dot de
CC: gcc-bugs at gcc dot gnu dot org
GCC build triplet: i686-pc-linux-gnu
GCC host triplet: i686-pc-linux-gnu
GCC target triplet: i686-pc-linux-gnu
# cat x.cpp
#include <string>
const std::string str[ 1 ] = { "a" };
const std::wstring wstr[ 1 ] = { L"a" };
# c++ -Wall -c x.cpp
x.cpp:3: warning: statement has no effect
x.cpp:4: warning: statement has no effect
Should be self-explanatory.
^ permalink raw reply [flat|nested] 8+ messages in thread
* [Bug c++/11957] wrong "warning: statement has no effect"
2003-08-17 19:18 [Bug c++/11957] New: wrong "warning: statement has no effect" ritzert at t-online dot de
@ 2003-08-17 19:21 ` pinskia at gcc dot gnu dot org
2003-08-17 19:30 ` nathan at gcc dot gnu dot org
` (5 subsequent siblings)
6 siblings, 0 replies; 8+ messages in thread
From: pinskia at gcc dot gnu dot org @ 2003-08-17 19:21 UTC (permalink / raw)
To: gcc-bugs
PLEASE REPLY TO gcc-bugzilla@gcc.gnu.org ONLY, *NOT* gcc-bugs@gcc.gnu.org.
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=11957
pinskia at gcc dot gnu dot org changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|UNCONFIRMED |WAITING
------- Additional Comments From pinskia at gcc dot gnu dot org 2003-08-17 19:21 -------
I could not reproduce it on the mainline (20030814)?
Can you provide the output of gcc -v?
^ permalink raw reply [flat|nested] 8+ messages in thread
* [Bug c++/11957] wrong "warning: statement has no effect"
2003-08-17 19:18 [Bug c++/11957] New: wrong "warning: statement has no effect" ritzert at t-online dot de
2003-08-17 19:21 ` [Bug c++/11957] " pinskia at gcc dot gnu dot org
@ 2003-08-17 19:30 ` nathan at gcc dot gnu dot org
2003-08-17 19:46 ` ritzert at t-online dot de
` (4 subsequent siblings)
6 siblings, 0 replies; 8+ messages in thread
From: nathan at gcc dot gnu dot org @ 2003-08-17 19:30 UTC (permalink / raw)
To: gcc-bugs
PLEASE REPLY TO gcc-bugzilla@gcc.gnu.org ONLY, *NOT* gcc-bugs@gcc.gnu.org.
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=11957
nathan at gcc dot gnu dot org changed:
What |Removed |Added
----------------------------------------------------------------------------
AssignedTo|unassigned at gcc dot gnu |nathan at gcc dot gnu dot
|dot org |org
Status|WAITING |ASSIGNED
------- Additional Comments From nathan at gcc dot gnu dot org 2003-08-17 19:30 -------
someting with my 11512 fix
^ permalink raw reply [flat|nested] 8+ messages in thread
* [Bug c++/11957] wrong "warning: statement has no effect"
2003-08-17 19:18 [Bug c++/11957] New: wrong "warning: statement has no effect" ritzert at t-online dot de
2003-08-17 19:21 ` [Bug c++/11957] " pinskia at gcc dot gnu dot org
2003-08-17 19:30 ` nathan at gcc dot gnu dot org
@ 2003-08-17 19:46 ` ritzert at t-online dot de
2003-08-18 12:55 ` nathan at gcc dot gnu dot org
` (3 subsequent siblings)
6 siblings, 0 replies; 8+ messages in thread
From: ritzert at t-online dot de @ 2003-08-17 19:46 UTC (permalink / raw)
To: gcc-bugs
PLEASE REPLY TO gcc-bugzilla@gcc.gnu.org ONLY, *NOT* gcc-bugs@gcc.gnu.org.
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=11957
------- Additional Comments From ritzert at t-online dot de 2003-08-17 19:46 -------
Subject: Re: wrong "warning: statement has no effect"
pinskia at gcc dot gnu dot org schrieb:
> http://gcc.gnu.org/bugzilla/show_bug.cgi?id=11957
> ------- Additional Comments From pinskia at gcc dot gnu
> dot org 2003-08-17 19:21 -------
> I could not reproduce it on the mainline (20030814)?
> Can you provide the output of gcc -v?
Sure. I'm using today's snapshot:
gcc version 3.4 20030817 (experimental)
Michael
^ permalink raw reply [flat|nested] 8+ messages in thread
* [Bug c++/11957] wrong "warning: statement has no effect"
2003-08-17 19:18 [Bug c++/11957] New: wrong "warning: statement has no effect" ritzert at t-online dot de
` (2 preceding siblings ...)
2003-08-17 19:46 ` ritzert at t-online dot de
@ 2003-08-18 12:55 ` nathan at gcc dot gnu dot org
2003-08-18 12:55 ` cvs-commit at gcc dot gnu dot org
` (2 subsequent siblings)
6 siblings, 0 replies; 8+ messages in thread
From: nathan at gcc dot gnu dot org @ 2003-08-18 12:55 UTC (permalink / raw)
To: gcc-bugs
PLEASE REPLY TO gcc-bugzilla@gcc.gnu.org ONLY, *NOT* gcc-bugs@gcc.gnu.org.
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=11957
nathan at gcc dot gnu dot org changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|ASSIGNED |RESOLVED
Resolution| |FIXED
------- Additional Comments From nathan at gcc dot gnu dot org 2003-08-18 12:55 -------
2003-08-18 Nathan Sidwell <nathan@codesourcery.com>
PR c++/11957
* cp-tree.h (finish_stmt_expr): Add bool parameter.
* init.c (finish_init_stmts): Pass true to finish_stmt_expr. Don't
adjust the stmt_expr here.
(build_vec_init): Use finish_stmt_expr_expr, convert result to
array type.
* parser.c (cp_parser_primar_expression): Adjust finish_stmt_expr
call.
* pt.c (tsubst_copy): Likewise.
* semantics.c (finish_stmt_expr): Add parameter.
^ permalink raw reply [flat|nested] 8+ messages in thread
* [Bug c++/11957] wrong "warning: statement has no effect"
2003-08-17 19:18 [Bug c++/11957] New: wrong "warning: statement has no effect" ritzert at t-online dot de
` (3 preceding siblings ...)
2003-08-18 12:55 ` nathan at gcc dot gnu dot org
@ 2003-08-18 12:55 ` cvs-commit at gcc dot gnu dot org
2003-08-18 14:30 ` nathan at gcc dot gnu dot org
2005-02-22 12:21 ` [Bug c++/11957] [3.4 Regression] " cvs-commit at gcc dot gnu dot org
6 siblings, 0 replies; 8+ messages in thread
From: cvs-commit at gcc dot gnu dot org @ 2003-08-18 12:55 UTC (permalink / raw)
To: gcc-bugs
PLEASE REPLY TO gcc-bugzilla@gcc.gnu.org ONLY, *NOT* gcc-bugs@gcc.gnu.org.
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=11957
------- Additional Comments From cvs-commit at gcc dot gnu dot org 2003-08-18 12:55 -------
Subject: Bug 11957
CVSROOT: /cvs/gcc
Module name: gcc
Changes by: nathan@gcc.gnu.org 2003-08-18 12:55:04
Modified files:
gcc/cp : ChangeLog cp-tree.h init.c parser.c pt.c
semantics.c
Log message:
cp:
PR c++/11957
* cp-tree.h (finish_stmt_expr): Add bool parameter.
* init.c (finish_init_stmts): Pass true to finish_stmt_expr. Don't
adjust the stmt_expr here.
(build_vec_init): Use finish_stmt_expr_expr, convert result to
array type.
* parser.c (cp_parser_primar_expression): Adjust finish_stmt_expr
call.
* pt.c (tsubst_copy): Likewise.
* semantics.c (finish_stmt_expr): Add parameter.
testsuite:
PR c++/11957
* g++.dg/warn/noeffect1.C: New test.
Patches:
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/cp/ChangeLog.diff?cvsroot=gcc&r1=1.3606&r2=1.3607
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/cp/cp-tree.h.diff?cvsroot=gcc&r1=1.902&r2=1.903
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/cp/init.c.diff?cvsroot=gcc&r1=1.338&r2=1.339
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/cp/parser.c.diff?cvsroot=gcc&r1=1.100&r2=1.101
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/cp/pt.c.diff?cvsroot=gcc&r1=1.759&r2=1.760
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/cp/semantics.c.diff?cvsroot=gcc&r1=1.347&r2=1.348
^ permalink raw reply [flat|nested] 8+ messages in thread
* [Bug c++/11957] wrong "warning: statement has no effect"
2003-08-17 19:18 [Bug c++/11957] New: wrong "warning: statement has no effect" ritzert at t-online dot de
` (4 preceding siblings ...)
2003-08-18 12:55 ` cvs-commit at gcc dot gnu dot org
@ 2003-08-18 14:30 ` nathan at gcc dot gnu dot org
2005-02-22 12:21 ` [Bug c++/11957] [3.4 Regression] " cvs-commit at gcc dot gnu dot org
6 siblings, 0 replies; 8+ messages in thread
From: nathan at gcc dot gnu dot org @ 2003-08-18 14:30 UTC (permalink / raw)
To: gcc-bugs
PLEASE REPLY TO gcc-bugzilla@gcc.gnu.org ONLY, *NOT* gcc-bugs@gcc.gnu.org.
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=11957
nathan at gcc dot gnu dot org changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |gerald at pfeifer dot com
------- Additional Comments From nathan at gcc dot gnu dot org 2003-08-18 14:30 -------
*** Bug 11964 has been marked as a duplicate of this bug. ***
^ permalink raw reply [flat|nested] 8+ messages in thread
* [Bug c++/11957] [3.4 Regression] wrong "warning: statement has no effect"
2003-08-17 19:18 [Bug c++/11957] New: wrong "warning: statement has no effect" ritzert at t-online dot de
` (5 preceding siblings ...)
2003-08-18 14:30 ` nathan at gcc dot gnu dot org
@ 2005-02-22 12:21 ` cvs-commit at gcc dot gnu dot org
6 siblings, 0 replies; 8+ messages in thread
From: cvs-commit at gcc dot gnu dot org @ 2005-02-22 12:21 UTC (permalink / raw)
To: gcc-bugs
------- Additional Comments From cvs-commit at gcc dot gnu dot org 2005-02-22 03:13 -------
Subject: Bug 11957
CVSROOT: /cvs/gcc
Module name: gcc
Changes by: bryce@gcc.gnu.org 2005-02-22 03:13:35
Modified files:
libjava : ChangeLog
libjava/gcj : method.h
libjava/java/io: ObjectInputStream.java ObjectStreamField.java
VMObjectStreamClass.java
natObjectInputStream.cc
natVMObjectStreamClass.cc
ObjectOutputStream.java
libjava/java/lang/reflect: Field.java natField.cc
Log message:
2005-02-21 Bryce McKinlay <mckinlay@redhat.com>
Merge serialization from GNU Classpath.
* gcj/method.h: Add missing #includes.
* java/io/ObjectInputStream.java (readClassDescriptor): Check for
primitive class IDs on the stream here...
(resolveClass): ...not here.
* java/io/ObjectStreamField.java: Use VMObjectStream class calls to set
fields.
* java/io/VMObjectStreamClass.java (setDoubleNative, setFloatNative,
setLongNative, setIntNative, setShortNative, setCharNative,
setByteNative, setBooleanNative, setObjectNative): New native methods.
* java/io/natVMObjectStreamClass.java (setDoubleNative, setFloatNative,
setLongNative, setIntNative, setShortNative, setCharNative,
setByteNative, setBooleanNative, setObjectNative): Implement them.
* java/io/natObjectInputStream.cc (allocateObject): Add new parameters
from Classpath's version. Use _Jv_FromReflectedConstructor(). Call
the constructor here.
(callConstructor): Removed.
(getCallersClassLoader): Removed.
* java/lang/reflect/Field.java (setByte, setShort, setInt, setLong,
setFloat, setDouble, setChar, setBoolean): Add 'checkFinal' parameter
to control whether setting final field values is permitted. Call
getAddr() with checkFinal parameter instead of setAddr().
* java/lang/reflect/natField.cc (getType): Lookup and resolve field
only if not done already.
(getAddr): Add checkFinal parameter. Do the final field check only if
checkFinal is set.
(setAddr): Removed.
2005-02-21 Mark Wielaard <mark@klomp.org>
# Fixes bug #11957
* java/io/ObjectInputStream.java (resolveClass): Don't check "void"
twice.
2005-02-21 Mark Wielaard <mark@klomp.org>
Fixes bug #11618.
* java/io/ObjectInputStream.java (readClassDescriptor): Handle classes
without a super class and us ObjectStreamClass.lookupForClassObject().
(resolveClass): Check for primitive types.
(lookupClass): Return null when argument is null.
2005-02-21 Jeroen Frijters <jeroen@frijters.net>
* java/io/ObjectInputStream.java
(readObject): Fix to consume TC_ENDBLOCKDATA after readExternal.
2005-02-21 Jeroen Frijters <jeroen@frijters.net>
* java/io/ObjectOutputStream.java
(writeObject, callWriteMethod): Replaced reflection with accessing
cached info in ObjectStreamClass.
(getMethod): Removed.
* java/io/ObjectStreamClass.java
(findMethod): Added check to make sure the method found has the
right modifiers.
(cacheMethods): Added writeReplace and writeObject methods.
(setFlags): Look at new writeObjectMethod field instead of doing
reflection again.
(writeReplaceMethod): New field.
(writeObjectMethod): New field.
2005-02-21 Guilhem Lavaux <guilhem@kaffe.org>
Jeroen Frijters <jeroen@frijters.net>
* java/io/ObjectInputStream.java
(newObject): Changed prototype. Get a constructor reflect object
directly.
(callConstructor): Removed.
(allocateObject): Changed prototype.
(readClassDescriptor): Build the constructor reflection directly.
(readObject): Invoke newObject using the new prototype.
* java/io/ObjectStreamClass.java
(firstNonSerializableParent): Removed.
(firstNonSerializableParentConstructor): Added.
Patches:
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/libjava/ChangeLog.diff?cvsroot=gcc&r1=1.3371&r2=1.3372
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/libjava/gcj/method.h.diff?cvsroot=gcc&r1=1.6&r2=1.7
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/libjava/java/io/ObjectInputStream.java.diff?cvsroot=gcc&r1=1.40&r2=1.41
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/libjava/java/io/ObjectStreamField.java.diff?cvsroot=gcc&r1=1.12&r2=1.13
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/libjava/java/io/VMObjectStreamClass.java.diff?cvsroot=gcc&r1=1.1&r2=1.2
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/libjava/java/io/natObjectInputStream.cc.diff?cvsroot=gcc&r1=1.8&r2=1.9
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/libjava/java/io/natVMObjectStreamClass.cc.diff?cvsroot=gcc&r1=1.1&r2=1.2
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/libjava/java/io/ObjectOutputStream.java.diff?cvsroot=gcc&r1=1.31&r2=1.32
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/libjava/java/lang/reflect/Field.java.diff?cvsroot=gcc&r1=1.9&r2=1.10
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/libjava/java/lang/reflect/natField.cc.diff?cvsroot=gcc&r1=1.16&r2=1.17
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=11957
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2005-02-22 3:14 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2003-08-17 19:18 [Bug c++/11957] New: wrong "warning: statement has no effect" ritzert at t-online dot de
2003-08-17 19:21 ` [Bug c++/11957] " pinskia at gcc dot gnu dot org
2003-08-17 19:30 ` nathan at gcc dot gnu dot org
2003-08-17 19:46 ` ritzert at t-online dot de
2003-08-18 12:55 ` nathan at gcc dot gnu dot org
2003-08-18 12:55 ` cvs-commit at gcc dot gnu dot org
2003-08-18 14:30 ` nathan at gcc dot gnu dot org
2005-02-22 12:21 ` [Bug c++/11957] [3.4 Regression] " cvs-commit at gcc dot gnu dot org
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).