🔒 Locked

trouble here...

Started by GameDev.net Dec 3, 1999 at 6:50 PM 3 replies 650+ views

Original Post

Dec 3, 1999 at 6:50 PM

Hi!

I've converted an old game project from MSVC 5.0 to 6.0....when i try to compile the whole thing i get an error on this line of code:

winclass.hbrBackground = GetStockObject(DKGRAY_BRUSH);

the error is:
cannot convert from 'void *' to 'struct HBRUSH__ *'

what is the problem???

Thanks in advance.

mason

Dec 3, 1999 at 5:03 PM

Not sure why you got this in 6 and not 5, but you should just be able to cast it:

winclass.hbrBackground = reinterpret_cast<HBRUSH *>(GetStockObject(DKGRAY_BRUSH));

Mason McCuskey
Spin Studios
www.spin-studios.com

[This message has been edited by mason (edited December 03, 1999).]

Gecko

Dec 3, 1999 at 5:38 PM

i don't think you need all that reinterpret_cast<> stuff, all you need to do is

winclass.hbrBackground = (HBRUSH)GetStockObject(DKGRAY_BRUSH);

GetStockObject just returns a void pointer to some memory, and you need to tell the compiler to cast it into the HBRUSH class for the background of your window

------------------

_________________Gecko___

Myopic Rhino

Dec 3, 1999 at 6:03 PM

Actually, reinterpret_cast and (its siblings) is more consistent with proper C++ style.

Atavist

Dec 3, 1999 at 6:50 PM

Thats one of the big differences between vs5 and vs6. 5 was more forgiving with mixing types, which didnt really follow standards.

Misuse of the reinterpret_cast operator can easily be unsafe. (and not as easy to read) Unless the desired conversion is inherently low-level, you should use one of the other cast operators.

Topic Locked

This topic has been locked by a moderator. New replies are not allowed.