Error on DoMethod()

[Jettah]

Hi,

While designing a simple program I encountered an error and a warning when using DoMethod().

Here is a part of the offending source:

struct Window *Win = NULL;
if (Win = (struct Window *)DoMethod(&WinObj, WM_OPEN, NULL))
{
//anything usefull here
}

The warning is: cast does not match function type
The error sounds: Win has an incomplete type

Can anyone shed some light on this matter?

Regards,

Jettah

[Jettah]

@Piru

Stands corrected. My keyboard is a bit sticky on that particular key.

The source code however does contain that parenthesis.

Thanks anyway

Jettah

[Jettah]

@DaveP

"Check you have all the intuition includes required in order to use it."

struct Window is defined in intuition/intuition.h, which is included.

Thanks

Jettah

[Jettah]

@Thomas


Up to now I have this piece of source code (relevant part only):

{                                                 // level 0
Object *WinObj = NULL;
if (WinObj = )
{                                                 // level 1
struct Window *Win = NULL;
if (Win = struct Window *)DoMethod(WinObj, WM_OPEN, NULL))
{                                                 // level 2
// Try to do the usefull stuff here...

DoMethod(WInObj, WM_CLOSE, NULL);
Win    = NULL;
WinSig = 0L;
}                                                 // level 2
DisposeObject((Object *)WinObj);
WinObj = NULL;
}                                                 // level 1
}                                                 // level 0

I now get the WARNING: cast does not match funktion type
intuition protos are included explicitly:
#include

Sytem used: A1200 OS3.9 StormC4 Gcc

Why, oh why, do I get a warning about a cast? IMHO I'm doing it the correct way, but I can be wrong of course.

Thanks,

Jettah

[Jettah]

@Thomas

On OS3.9 you have to include clib/alib_protos.h for the DoMethod prototype.

This file has been included (was included all the way).

Changed the offending line.
FROM:
if (Win = (struct Window *)DoMethod(WinObj, WM_OPEN, NULL))

TO:
if ((ULONG)Win = DoMethod(WinObj, WM_OPEN, NULL))

The WARNING has now gone, but I do have a feeling that it is not a correct way of doing things. Am I wrong in this respect?

EDIT:
The program compiles and links well. It executes fine, so I assume it is correct now!

Thanks, for all your remarks, wisdom and knowledge. I'm very satisfied.
To me, this topic is closed.

Jettah