stiv commented

Similar issues with texture.c. Beware 32 vs 64 bit issues between different architectures.

texture.c:258: error: format ‘%u’ expects type ‘unsigned int’, but argument 7 has type ‘long unsigned int’

texture.c:263: error: format ‘%u’ expects type ‘unsigned int’, but argument 8 has type ‘long unsigned int’

stiv commented

Same story in 2.3 branch. 32 bit linux

../../../2.3/lib/ivis_opengl/pieblitfunc.c:280: error: passing argument 2 of ‘glDeleteTextures’ from incompatible pointer type

../../2.3/src/texture.c:253: error: format ‘%u’ expects type ‘unsigned int’, but argument 7 has type ‘long unsigned int’

../../2.3/src/texture.c:258: error: format ‘%u’ expects type ‘unsigned int’, but argument 8 has type ‘long unsigned int’

Zarel commented

Well, better to cast a long to an int, than an int ptr to a long ptr...

I'd like someone with Linux to go fix these errors, since we'd have to repeat this many times to get all of them. Stiv, can you write a patch?

jaakan commented

I think changing

glDeleteTextures(1, &_TEX_PAGE[radarTexture].id);
to
glDeleteTextures(1, (GLuint *) &_TEX_PAGE[radarTexture].id);

takes care of pieblitfunc.c:280.... error

Would something like that fix the following?

texture.c:258: error: format ‘%u’ expects type ‘unsigned int’, but argument 7 has type ‘long unsigned int’
texture.c:263: error: format ‘%u’ expects type ‘unsigned int’, but argument 8 has type ‘long unsigned int’

like

_TEX_PAGE[texPage].id 
to
(GLuint)_TEX_PAGE[texPage].id

in texture.c

these changes do help it to fully build again for me at least.

Zarel changed status from new to closed

Zarel set resolution to fixed

Zarel commented

Fixed in [8590]/[8591].