Re: cvs: php4 /ext/mbstring mbfilter.c mbfilter_ja.c
| From: | Derick Rethans | 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
---------------------------------------------------------------------