* iconvdata: Get rid of lvalue casts
@ 2004-03-07 7:29 Andreas Jaeger
2004-03-07 7:37 ` Ulrich Drepper
0 siblings, 1 reply; 2+ messages in thread
From: Andreas Jaeger @ 2004-03-07 7:29 UTC (permalink / raw)
To: Glibc hackers
[-- Attachment #1: Type: text/plain, Size: 5396 bytes --]
Here's one more patch to get rid of lvalues to make GCC 3.5 happy. It
generates the same code as before on my machine and passes the
testsuite.
Ok to commit?
Andreas
2004-03-07 Andreas Jaeger <aj@suse.de>
* iconvdata/iso-2022-cn-ext.c (BODY): Remove cast used as lvalue.
* iconvdata/tcvn5712-1.c (EMIT_SHIFT_TO_INIT): Likewise.
* iconvdata/euc-jisx0213.c (EMIT_SHIFT_TO_INIT): Likewise.
* iconvdata/shift_jisx0213.c (EMIT_SHIFT_TO_INIT): Likewise.
* iconvdata/tscii.c (EMIT_SHIFT_TO_INIT): Likewise.
============================================================
Index: iconvdata/tscii.c
--- iconvdata/tscii.c 24 Sep 2002 03:39:45 -0000 1.1
+++ iconvdata/tscii.c 7 Mar 2004 07:10:46 -0000
@@ -1,5 +1,5 @@
/* Conversion from and to TSCII.
- Copyright (C) 2002 Free Software Foundation, Inc.
+ Copyright (C) 2002, 2004 Free Software Foundation, Inc.
This file is part of the GNU C Library.
Contributed by Bruno Haible <bruno@clisp.org>, 2002.
@@ -98,7 +98,8 @@
break; \
} \
/* Write out the pending character. */ \
- *((uint32_t *) outbuf)++ = data->__statep->__count >> 8; \
+ *((uint32_t *) outbuf) = data->__statep->__count >> 8; \
+ outbuf += sizeof (uint32_t); \
/* Retrieve the successor state. */ \
data->__statep->__count = \
tscii_next_state[(data->__statep->__count >> 4) & 0x0f]; \
============================================================
Index: iconvdata/iso-2022-cn-ext.c
--- iconvdata/iso-2022-cn-ext.c 28 Jun 2002 21:13:14 -0000 1.12
+++ iconvdata/iso-2022-cn-ext.c 7 Mar 2004 07:10:47 -0000
@@ -1,5 +1,5 @@
/* Conversion module for ISO-2022-CN-EXT.
- Copyright (C) 2000-2002 Free Software Foundation, Inc.
+ Copyright (C) 2000-2002, 2004 Free Software Foundation, Inc.
This file is part of the GNU C Library.
Contributed by Ulrich Drepper <drepper@cygnus.com>, 2000.
@@ -377,7 +377,8 @@ enum
} \
} \
\
- *((uint32_t *) outptr)++ = ch; \
+ *((uint32_t *) outptr) = ch; \
+ outptr += sizeof (uint32_t); \
}
#define EXTRA_LOOP_DECLS , int *setp
#define INIT_PARAMS int set = (*setp >> 3) & CURRENT_MASK; \
============================================================
Index: iconvdata/tcvn5712-1.c
--- iconvdata/tcvn5712-1.c 2 Dec 2002 22:07:52 -0000 1.7
+++ iconvdata/tcvn5712-1.c 7 Mar 2004 07:10:47 -0000
@@ -1,5 +1,5 @@
/* Conversion to and from TCVN5712-1.
- Copyright (C) 2001, 2002 Free Software Foundation, Inc.
+ Copyright (C) 2001, 2002, 2004 Free Software Foundation, Inc.
This file is part of the GNU C Library.
Contributed by Ulrich Drepper <drepper@redhat.com>, 2001.
@@ -68,7 +68,8 @@
if (__builtin_expect (outbuf + 4 <= outend, 1)) \
{ \
/* Write out the last character. */ \
- *((uint32_t *) outbuf)++ = data->__statep->__count >> 3; \
+ *((uint32_t *) outbuf) = data->__statep->__count >> 3; \
+ outbuf += sizeof (uint32_t); \
data->__statep->__count = 0; \
} \
else \
============================================================
Index: iconvdata/euc-jisx0213.c
--- iconvdata/euc-jisx0213.c 2 Dec 2002 22:07:54 -0000 1.8
+++ iconvdata/euc-jisx0213.c 7 Mar 2004 07:10:47 -0000
@@ -1,5 +1,5 @@
/* Conversion from and to EUC-JISX0213.
- Copyright (C) 2002 Free Software Foundation, Inc.
+ Copyright (C) 2002, 2004 Free Software Foundation, Inc.
This file is part of the GNU C Library.
Contributed by Bruno Haible <bruno@clisp.org>, 2002.
@@ -83,7 +83,8 @@
if (__builtin_expect (outbuf + 4 <= outend, 1)) \
{ \
/* Write out the last character. */ \
- *((uint32_t *) outbuf)++ = data->__statep->__count >> 3; \
+ *((uint32_t *) outbuf) = data->__statep->__count >> 3; \
+ outbuf += sizeof (uint32_t); \
data->__statep->__count = 0; \
} \
else \
============================================================
Index: iconvdata/shift_jisx0213.c
--- iconvdata/shift_jisx0213.c 2 Dec 2002 22:07:56 -0000 1.8
+++ iconvdata/shift_jisx0213.c 7 Mar 2004 07:10:47 -0000
@@ -1,5 +1,5 @@
/* Conversion from and to Shift_JISX0213.
- Copyright (C) 2002 Free Software Foundation, Inc.
+ Copyright (C) 2002, 2004 Free Software Foundation, Inc.
This file is part of the GNU C Library.
Contributed by Bruno Haible <bruno@clisp.org>, 2002.
@@ -83,7 +83,8 @@
if (__builtin_expect (outbuf + 4 <= outend, 1)) \
{ \
/* Write out the last character. */ \
- *((uint32_t *) outbuf)++ = data->__statep->__count >> 3; \
+ *((uint32_t *) outbuf) = data->__statep->__count >> 3; \
+ outbuf += sizeof (uint32_t); \
data->__statep->__count = 0; \
} \
else \
--
Andreas Jaeger, aj@suse.de, http://www.suse.de/~aj
SuSE Linux AG, Maxfeldstr. 5, 90409 Nürnberg, Germany
GPG fingerprint = 93A3 365E CE47 B889 DF7F FED1 389A 563C C272 A126
[-- Attachment #2: Type: application/pgp-signature, Size: 188 bytes --]
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: iconvdata: Get rid of lvalue casts
2004-03-07 7:29 iconvdata: Get rid of lvalue casts Andreas Jaeger
@ 2004-03-07 7:37 ` Ulrich Drepper
0 siblings, 0 replies; 2+ messages in thread
From: Ulrich Drepper @ 2004-03-07 7:37 UTC (permalink / raw)
To: Andreas Jaeger; +Cc: Glibc hackers
Andreas Jaeger wrote:
> Ok to commit?
Yes.
--
➧ Ulrich Drepper ➧ Red Hat, Inc. ➧ 444 Castro St ➧ Mountain View, CA â–
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2004-03-07 7:37 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2004-03-07 7:29 iconvdata: Get rid of lvalue casts Andreas Jaeger
2004-03-07 7:37 ` Ulrich Drepper
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).