Re: cvs: php4 /ext/mbstring mbfilter.c mbfilter_ja.c

From: Date: Thu, 28 Mar 2002 07:29:04 +0000
Subject: Re: cvs: php4 /ext/mbstring mbfilter.c mbfilter_ja.c
References: 1  Groups: php.qa 
Request: Send a blank email to php-qa+get-4784@lists.php.net to get a copy of this message
Hello Sascha, On Mon, 3 Dec 2001, Sascha Schumann wrote: > sas Mon Dec 3 10:32:14 2001 EDT > > Modified files: > /php4/ext/mbstring mbfilter.c mbfilter_ja.c > Log: > Fix declarations of static variables. Builds on AIX now. But it broke the build on Tru64 Unix: > cc: Error: mbfilter_ja.c, line 86: The static declaration of \"cp932ext3_eucjp_ta > ble\" is a tentative definition and specifies an incomplete type. (incompstat) > static const unsigned short cp932ext3_eucjp_table[]; > > Replaced static with extern for all lines concerned (mbfilter.c as well), as it was in the > previous version, and it works. Derick > > > Index: php4/ext/mbstring/mbfilter.c > diff -u php4/ext/mbstring/mbfilter.c:1.16 php4/ext/mbstring/mbfilter.c:1.17 > --- php4/ext/mbstring/mbfilter.c:1.16 Wed Aug 29 07:29:52 2001 > +++ php4/ext/mbstring/mbfilter.c Mon Dec 3 10:32:12 2001 > @@ -79,7 +79,7 @@ > * > */ > > -/* $Id: mbfilter.c,v 1.16 2001/08/29 11:29:52 wez Exp $ */ > +/* $Id: mbfilter.c,v 1.17 2001/12/03 15:32:12 sas Exp $ */ > > > #include <stdlib.h> > @@ -101,19 +101,19 @@ > #define mbfl_pfree free > > /* unicode table */ > -extern const unsigned short cp1252_ucs_table[]; > -extern const unsigned short iso8859_2_ucs_table[]; > -extern const unsigned short iso8859_3_ucs_table[]; > -extern const unsigned short iso8859_4_ucs_table[]; > -extern const unsigned short iso8859_5_ucs_table[]; > -extern const unsigned short iso8859_6_ucs_table[]; > -extern const unsigned short iso8859_7_ucs_table[]; > -extern const unsigned short iso8859_8_ucs_table[]; > -extern const unsigned short iso8859_9_ucs_table[]; > -extern const unsigned short iso8859_10_ucs_table[]; > -extern const unsigned short iso8859_13_ucs_table[]; > -extern const unsigned short iso8859_14_ucs_table[]; > -extern const unsigned short iso8859_15_ucs_table[]; > +static const unsigned short cp1252_ucs_table[]; > +static const unsigned short iso8859_2_ucs_table[]; > +static const unsigned short iso8859_3_ucs_table[]; > +static const unsigned short iso8859_4_ucs_table[]; > +static const unsigned short iso8859_5_ucs_table[]; > +static const unsigned short iso8859_6_ucs_table[]; > +static const unsigned short iso8859_7_ucs_table[]; > +static const unsigned short iso8859_8_ucs_table[]; > +static const unsigned short iso8859_9_ucs_table[]; > +static const unsigned short iso8859_10_ucs_table[]; > +static const unsigned short iso8859_13_ucs_table[]; > +static const unsigned short iso8859_14_ucs_table[]; > +static const unsigned short iso8859_15_ucs_table[]; > > /* charactor property table */ > #define MBFL_CHP_CTL 0x01 > @@ -123,7 +123,7 @@ > #define MBFL_CHP_MMHQENC 0x10 /* must Q-encoding in MIME Header encoded-word */ > #define MBFL_CHP_MSPECIAL 0x20 /* RFC822 Special characters */ > > -extern const unsigned char mbfl_charprop_table[]; > +static const unsigned char mbfl_charprop_table[]; > > > /* language structure */ > Index: php4/ext/mbstring/mbfilter_ja.c > diff -u php4/ext/mbstring/mbfilter_ja.c:1.2 php4/ext/mbstring/mbfilter_ja.c:1.3 > --- php4/ext/mbstring/mbfilter_ja.c:1.2 Mon May 7 09:57:59 2001 > +++ php4/ext/mbstring/mbfilter_ja.c Mon Dec 3 10:32:12 2001 > @@ -77,41 +77,41 @@ > * > */ > > -/* $Id: mbfilter_ja.c,v 1.2 2001/05/07 13:57:59 hirokawa Exp $ */ > +/* $Id: mbfilter_ja.c,v 1.3 2001/12/03 15:32:12 sas Exp $ */ > > #include "mbfilter.h" > #include "mbfilter_ja.h" > > /* cp932 table */ > -extern const unsigned short cp932ext3_eucjp_table[]; > -extern const int cp932ext3_eucjp_table_size; > +static const unsigned short cp932ext3_eucjp_table[]; > +static const int cp932ext3_eucjp_table_size; > > /* unicode table */ > -extern const unsigned short jisx0208_ucs_table[]; > -extern const unsigned short jisx0212_ucs_table[]; > -extern const unsigned short cp932ext1_ucs_table[]; > -extern const unsigned short cp932ext2_ucs_table[]; > -extern const unsigned short cp932ext3_ucs_table[]; > -extern const unsigned short ucs_a1_jis_table[]; > -extern const unsigned short ucs_a2_jis_table[]; > -extern const unsigned short ucs_i_jis_table[]; > -extern const unsigned short ucs_r_jis_table[]; > -extern const int jisx0208_ucs_table_size; > -extern const int jisx0212_ucs_table_size; > -extern const int cp932ext1_ucs_table_min; > -extern const int cp932ext1_ucs_table_max; > -extern const int cp932ext2_ucs_table_min; > -extern const int cp932ext2_ucs_table_max; > -extern const int cp932ext3_ucs_table_min; > -extern const int cp932ext3_ucs_table_max; > -extern const int ucs_a1_jis_table_min; > -extern const int ucs_a1_jis_table_max; > -extern const int ucs_a2_jis_table_min; > -extern const int ucs_a2_jis_table_max; > -extern const int ucs_i_jis_table_min; > -extern const int ucs_i_jis_table_max; > -extern const int ucs_r_jis_table_min; > -extern const int ucs_r_jis_table_max; > +static const unsigned short jisx0208_ucs_table[]; > +static const unsigned short jisx0212_ucs_table[]; > +static const unsigned short cp932ext1_ucs_table[]; > +static const unsigned short cp932ext2_ucs_table[]; > +static const unsigned short cp932ext3_ucs_table[]; > +static const unsigned short ucs_a1_jis_table[]; > +static const unsigned short ucs_a2_jis_table[]; > +static const unsigned short ucs_i_jis_table[]; > +static const unsigned short ucs_r_jis_table[]; > +static const int jisx0208_ucs_table_size; > +static const int jisx0212_ucs_table_size; > +static const int cp932ext1_ucs_table_min; > +static const int cp932ext1_ucs_table_max; > +static const int cp932ext2_ucs_table_min; > +static const int cp932ext2_ucs_table_max; > +static const int cp932ext3_ucs_table_min; > +static const int cp932ext3_ucs_table_max; > +static const int ucs_a1_jis_table_min; > +static const int ucs_a1_jis_table_max; > +static const int ucs_a2_jis_table_min; > +static const int ucs_a2_jis_table_max; > +static const int ucs_i_jis_table_min; > +static const int ucs_i_jis_table_max; > +static const int ucs_r_jis_table_min; > +static const int ucs_r_jis_table_max; > > #define CK(statement) do { if ((statement) < 0) return (-1); } while (0) Derick Rethans --------------------------------------------------------------------- PHP: Scripting the Web - www.php.net - derick@php.net All your branches are belong to me! SRM: Script Running Machine - www.vl-srm.net ---------------------------------------------------------------------

« previous php.qa (#4784) next »