Opened 16 years ago

Closed 16 years ago

Last modified 7 months 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
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 Changed 16 years ago by fingolfin

Owner: set to SF/olki

comment:2 Changed 16 years ago by fingolfin

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 Changed 16 years ago by SF/olki

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

comment:4 Changed 16 years ago by fingolfin

Status: newpending

comment:5 Changed 16 years ago by fingolfin

Chris, did olki's change help?

comment:6 Changed 16 years ago by SF/kreeblah

Status: pendingnew

comment:7 Changed 16 years ago by SF/kreeblah

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

comment:8 Changed 16 years ago by SF/olki

Status: newclosed

comment:9 Changed 7 months ago by digitall

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