Opened 21 years ago

Closed 21 years ago

Last modified 6 years ago

#884 closed defect

BeOS compilation problems w/sky module in current CVS tree

Reported by: SF/kreeblah Owned by: SF/olki
Priority: normal Component: Port: BeOS
Version: Keywords:
Cc: Game:

Description

This has happened before later stopped (only reporting this as a warning, for some reason . . .), but I'm getting the following error when compiling the current CVS tree.

g++ -Wp,-MMD,"sky/.deps/text.d2" -g -O -Wall -Wstrict- prototypes -Wuninitialized -Wno-long-long -Wno-multichar -Wno- unknown-pragmas -DHAVE_CONFIG_H -DUNIX -I. -Icommon `sdl- config --cflags` -c sky/text.cpp -o sky/text.o /boot/home/ScummVMCVS/1306032256/scummvm/sky/text.cpp: In method `struct displayText_t SkyText::displayText(char *, uint8 *, bool, short unsigned int, unsigned char)': /boot/home/ScummVMCVS/1306032256/scummvm/sky/text.cpp:316: initializing non-const `uint8 *&' with `byte *' will use a temporary make: *** [sky/text.o] Error 1

I still don't understand what the difference between an 8-bit object and a byte, but it's generating an error because of it.

As a side note, should I be submitting reports somewhere about compilation warnings (the forums, maybe?) or just compilation errors?

Thanks.

Ticket imported from: #754378. Ticket imported from: bugs/884.

Change History (9)

comment:1 by fingolfin, 21 years ago

Owner: set to SF/olki

comment:2 by fingolfin, 21 years ago

That this is marked as an error (or even a warning) is clearly a bug in your compiler.

However, it is also a sign of inconsistent use of the byte datatype vs. uint8 - even though they are, on the machine level, identical. Olki, you hopefully know which of you guys is responsible for that code, whoever is should try to fix this.

comment:3 by SF/olki, 21 years ago

commited a fix. Hope that works. Otherwise assign to joostp

comment:4 by fingolfin, 21 years ago

Status: newpending

comment:5 by fingolfin, 21 years ago

Chris, did olki's change help?

comment:6 by SF/kreeblah, 21 years ago

Status: pendingnew

comment:7 by SF/kreeblah, 21 years ago

Yes, it did. The Sky module compiles fine now. Thanks, olki.

comment:8 by SF/olki, 21 years ago

Status: newclosed

comment:9 by digitall, 6 years ago

Component: --Unset--Port: BeOS
Note: See TracTickets for help on using tickets.