Compile Error in REV 661

leslamarch · #1 06-14-2009, 08:09 PM

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** · #2 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 · #3 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 · #4 06-14-2009, 09:27 PM

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

**realityincarnate** · #5 06-14-2009, 10:17 PM

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

Domel · #6 06-14-2009, 11:07 PM

Ahh, right on ! That got it

**trevius** · #7 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