[make] static declaration follows non-static e.a.

[Ericston]

Wanneer ik PHP 4.1.2 compile met GD 1.8.4 ( zonder GD ging het daarvoor feilloos ) krijg ik deze foutmelding:

In file included from gd.c:36:
php_gd.h:69: warning: static declaration for `gdImageColorResolve' follows non-static
gd.c:92: conflicting types for `gdIOCtx'
/usr/local/gd/include/gd_io.h:18: previous declaration of `gdIOCtx'
make[3]: *** [gd.lo] Error 1
make[3]: Leaving directory `/root/src/php-4.1.2/ext/gd'
make[2]: *** [all-recursive] Error 1
make[2]: Leaving directory `/root/src/php-4.1.2/ext/gd'
make[1]: *** [all-recursive] Error 1
make[1]: Leaving directory `/root/src/php-4.1.2/ext'
make: *** [all-recursive] Error 1

Nu heb ik gezocht op Google en is de enige oplossing die gegeven wordt deze:

i've had the same problem since 4.0x...eventually i just gave up trying to
compile with gd support.If anyone has a better step by step instruction page
than the ones i've tried please let us know
bron: http://news.php.net/article.php?group=php.install&article=5706

Beetje jammer dus.

Daarnaast helpt dit een eind op weg:

Compiling PHP3 now gives the following warning (using gd 1.5):

functions/gdttf.c:165: warning: static declaration for `gdImageColorResolve' follows non-static

because the prototype in line 153 says

int gdImageColorResolve(...)

and the function declaration in line 163 says

static int gdImageColorResolve(...)

Another thought: Why is this function present in gdttf.c as well as in
gd.c? Isn't gd.c always used with gdttf.c?
bron: http://www.phpbuilder.com/mail/php-developer-list/199911/0798.php

Effe de betrokken bestanden greppen op de genoemde functies en variabelen:

GD includes:
gd.h:

code:

[..]
int gdImageColorResolve(gdImagePtr im, int r, int g, int b);
[..]

gd_io.h:

code:

[..]
typedef struct gdIOCtx {
      int     (*getC)(struct gdIOCtx*);
      int     (*getBuf)(struct gdIOCtx*, void*, int);
      void     (*putC)(struct gdIOCtx*, int);
      int     (*putBuf)(struct gdIOCtx*, const void*, int);
      int     (*seek)(struct gdIOCtx*, const int);
      long    (*tell)(struct gdIOCtx*);
      void    (*free)(struct gdIOCtx*);
} gdIOCtx;
[..]

PHP stuff:
php_gd.h:

code:

[..]
static int gdImageColorResolve(gdImagePtr, int, int, int);
[..]

gd.c:

code:

[..]
typedef FILE gdIOCtx;
[..]

Zoals jullie zien worden gdImageColorResolve() en gdIOCtx dubbel en verschillend gedeclareerd.

Nu snap ik dat er hier iets moet gebeuren, dus wat zouden jullie doen, wat moet aan wat worden aangepast?

Subtiele hints of complete codefragmenten zijn welkom.

[stylee]

Die typedef is volkomen legaal in C hoor

code:

static int gdImageColorResolve(gdImagePtr, int, int, int);

* stylee 's C kennis is een beetje roestig...

Kan iig niet bedenken waar die static voor zou moeten dienen

Zet er anders gewoon eens een static voor, kijk wat er gebeurt.

[Ericston]

Op zaterdag 23 maart 2002 22:33 schreef stylee het volgende:
Die typedef is volkomen legaal in C hoor

Nee, want even daarvoor (of daarna, heb de Makefile nog niet helemaal uitgepluist) wordt gdIOCtx zo gedeclareerd:

code:

typedef struct gdIOCtx {
      int     (*getC)(struct gdIOCtx*);
      int     (*getBuf)(struct gdIOCtx*, void*, int);
      void     (*putC)(struct gdIOCtx*, int);
      int     (*putBuf)(struct gdIOCtx*, const void*, int);
      int     (*seek)(struct gdIOCtx*, const int);
      long    (*tell)(struct gdIOCtx*);
      void    (*free)(struct gdIOCtx*);
} gdIOCtx;

code:

static int gdImageColorResolve(gdImagePtr, int, int, int);

Kan iig niet bedenken waar die static voor zou moeten dienen

Zet er anders gewoon eens een static voor, kijk wat er gebeurt.

Ik pas die PHP code dan maar aan aan de GD code ( die static weghalen dus ), en dan post ik zo wel mijn bevindingen.

[stylee]

Ericston: Nee, want even daarvoor (of daarna, heb de Makefile nog niet helemaal uitgepluist) wordt gdIOCtx zo gedeclareerd...

Een typedef geeft alleen maar een 'alias' aan voor een bepaalde type.

Als je eerst een struct had gedeclareerd van het type gdIOCtx en je wilt nu deze definiëren (of was het eerst definiëren en dan declareren.. raak hier altijd mee in de knoop ) dan zou je zoiets schrijven:

code:

gdIOCtx a;

Nu is gdIOCtx niet echt leuk om te schrijven maar FILE weer wel, dus maken ze er een alias van.

code:

typedef FILE gdIOCtx; // je zou bijv ook kunnen schrijven
                // typedef FILE int;
FILE a;        // zelfde als 
                // gdIOCtx a;

Lang geleden dat ik C heb gedaan maar zal zo ongeveer wel kloppen, voor vragen enzo maar ff aankloppen in het gezellige /14

[Ericston]

Op zaterdag 23 maart 2002 22:52 schreef stylee het volgende:

code:

typedef FILE gdIOCtx; // je zou bijv ook kunnen schrijven
                // typedef FILE int;
FILE a;        // zelfde als 
                // gdIOCtx a;

Lang geleden dat ik C heb gedaan maar zal zo ongeveer wel kloppen, voor vragen enzo maar ff aankloppen in het gezellige /14

Hmm, volgens K&R is het precies andersom.

Ik heb nu van...

code:

static int gdImageColorResolve(gdImagePtr, int, int, int);

...in php_gd.h, dit gemaakt...

code:

int gdImageColorResolve(gdImagePtr, int, int, int);

En bij...

code:

typedef FILE gdIOCtx

...stond dit...

code:

#if HAVE_LIBGD15
/* it's >= 1.5, i.e. has IOCtx */
#define USE_GD_IOCTX 1
#else
#undef USE_GD_IOCTX
#endif

#ifdef USE_GD_IOCTX
#include "gd_ctx.c"
#else
#define gdImageCreateFromGdCtx  NULL
#define gdImageCreateFromGd2Ctx     NULL
#define gdImageCreateFromGd2partCtx NULL
#define gdImageCreateFromGifCtx     NULL
#define gdImageCreateFromJpegCtx    NULL
#define gdImageCreateFromPngCtx     NULL
#define gdImageCreateFromWBMPCtx    NULL
typedef FILE gdIOCtx;
#define CTX_PUTC(c, fp) fputc(c, fp)
#endif

...in gd.c. Nu heb ik dit gemaakt...

code:

#if HAVE_LIBGD15
/* it's >= 1.5, i.e. has IOCtx */
#define USE_GD_IOCTX 1
#else
#undef USE_GD_IOCTX
#endif

#include "gd_ctx.c"

...omdat ik GD 1.8.3 heb, en die typedef eigenlijk niet mag gebeuren doordat er een onduidelijke fout in die #if stuff zit. Waarschijnlijk is HAVE_LIDGD15 ten onrechte false.

Iig, het compileert...
...ship it.

* Ericston weet trouwens hoe gezellig /14 is.

Was deze thread eigenlijk wel nodig? Naja, het staat in ieder geval op het net nu.