public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
* [Bug testsuite/33775] Memset
[not found] <bug-33775-15216@http.gcc.gnu.org/bugzilla/>
@ 2007-10-15 11:14 ` pinskia at gcc dot gnu dot org
2007-11-09 13:13 ` johan dot walles at gmail dot com
2007-11-09 18:17 ` manu at gcc dot gnu dot org
2 siblings, 0 replies; 3+ messages in thread
From: pinskia at gcc dot gnu dot org @ 2007-10-15 11:14 UTC (permalink / raw)
To: gcc-bugs
------- Comment #1 from pinskia at gcc dot gnu dot org 2007-10-15 11:14 -------
This code written as a[(char*)b] is valid even C code, in C, d[e] is the same
as *(d+e) so d[e] is the same as e[d]. So the code is valid and does the
correct thing. There is no bug here. Unless you can explain exactly why you
want the change, I am going to close this as invalid.
--
pinskia at gcc dot gnu dot org changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |gcc-bugs at gcc dot gnu dot
| |org
Component|classpath |testsuite
Product|classpath |gcc
Version|unspecified |unknown
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=33775
^ permalink raw reply [flat|nested] 3+ messages in thread
* [Bug testsuite/33775] Memset
[not found] <bug-33775-15216@http.gcc.gnu.org/bugzilla/>
2007-10-15 11:14 ` [Bug testsuite/33775] Memset pinskia at gcc dot gnu dot org
@ 2007-11-09 13:13 ` johan dot walles at gmail dot com
2007-11-09 18:17 ` manu at gcc dot gnu dot org
2 siblings, 0 replies; 3+ messages in thread
From: johan dot walles at gmail dot com @ 2007-11-09 13:13 UTC (permalink / raw)
To: gcc-bugs
------- Comment #2 from johan dot walles at gmail dot com 2007-11-09 13:13 -------
IMO the proposed change would make the code more readable.
--
johan dot walles at gmail dot com changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |johan dot walles at gmail
| |dot com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=33775
^ permalink raw reply [flat|nested] 3+ messages in thread
* [Bug testsuite/33775] Memset
[not found] <bug-33775-15216@http.gcc.gnu.org/bugzilla/>
2007-10-15 11:14 ` [Bug testsuite/33775] Memset pinskia at gcc dot gnu dot org
2007-11-09 13:13 ` johan dot walles at gmail dot com
@ 2007-11-09 18:17 ` manu at gcc dot gnu dot org
2 siblings, 0 replies; 3+ messages in thread
From: manu at gcc dot gnu dot org @ 2007-11-09 18:17 UTC (permalink / raw)
To: gcc-bugs
------- Comment #3 from manu at gcc dot gnu dot org 2007-11-09 18:16 -------
(In reply to comment #2)
> IMO the proposed change would make the code more readable.
Johan, the code you quote is in a file from the testsuite. Such code is used to
test for some correct behavior under various circumstances. Therefore, there is
typically a very good reason for the code to be written like that. Moreover, in
the testsuite you will find not only unreadable code but also code that is
broken on purpose to test that the compiler can handle such code.
--
manu at gcc dot gnu dot org changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |manu at gcc dot gnu dot org
Status|UNCONFIRMED |RESOLVED
Resolution| |INVALID
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=33775
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2007-11-09 18:17 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <bug-33775-15216@http.gcc.gnu.org/bugzilla/>
2007-10-15 11:14 ` [Bug testsuite/33775] Memset pinskia at gcc dot gnu dot org
2007-11-09 13:13 ` johan dot walles at gmail dot com
2007-11-09 18:17 ` manu 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).