EQEmulator Forums - Compile Error in REV 661

EQEmulator Forums (https://www.eqemulator.org/forums/index.php)

- Development::Bug Reports (https://www.eqemulator.org/forums/forumdisplay.php?f=591)

- - Compile Error in REV 661 (https://www.eqemulator.org/forums/showthread.php?t=28575)

leslamarch

06-14-2009 08:09 PM

Compile Error in REV 661

Just figured I would post this. Compiled under VS 2008

Code:

Error    144    error C2664: 'command_add' : cannot convert parameter 4 from 'void (__cdecl *)(Client *)' to 'CmdFuncPtr'    \zone\command.cpp    349

trevius

06-14-2009 08:23 PM

Hmm, it compiled fine under Linux for me. Maybe removing the "Seperator *sep" causes a problem with Windows compiles. You can try changing this:

/zone/command.cpp line 5218

Code:

void command_randomfeatures(Client *c)

to this:
/zone/command.cpp line 5218

Code:

void command_randomfeatures(Client *c, const Seperator *sep)

And also changing this:
/zone/command.h line 222

Code:

void command_randomfeatures(Client *c);

to this:
/zone/command.h line 222

Code:

void command_randomfeatures(Client *c, const Seperator *sep);

If that fixes it and the command still works properly, then I can correct that on the next SVN update I do unless someone wants to add it in before that.

leslamarch

06-14-2009 09:11 PM

no go, that leads to another error

Code:

Error 5 error C2447: '{' : missing function header (old-style formal list?) \zone\command.cpp 5219

If I'm the only one having problems with this I'll work on this when I have time, Kinda strange I'm the only one though. Thanks for the suggestion as always its appreciated.

Domel

06-14-2009 09:27 PM

I'm getting the same error.../nod

realityincarnate

06-14-2009 10:17 PM

Remove the semi-colon from the end of line 5218 in Trevius' modifications and it should work.

Domel

06-14-2009 11:07 PM

Ahh, right on ! That got it :cool:

trevius

06-14-2009 11:09 PM

Yeah, my bad, I corrected the stuff to change in the original post I made here.

I will get it changed tonight on the SVN if I remember. I didn't think much about it since it compiled and runs ok on Linux. Didn't think it needed the sep stuff since the command isn't taking an argument. I guess the fact that all other commands are set that way should have tipped me off that it was required :P

All times are GMT -4. The time now is 05:56 AM.