[Yashas]

Quote:

Originally Posted by Nero_3D No idea why you added the cmdid parameter to the command Makes it harder to change the command processor Isn't even needed, you just could use cid_commandname

You need it for reassigned commands.

Код:

//abc is not an alternate command to xyz but has it's command function reassigned to that of xyz
CMD:xyz(cmdid, playerid, params[])
{
      if(cmdid == cid_xyz) //directly using variables is faster than using GetCommandID
      {
              //player used /xyz
      }
      else if(cmdid == cid_abc) //cid_xyz and cid_abc are defined by SmartCMD
      {
             //player used /abc
      }
      return CMD_SUCCESS;
}

I can get rid of cmdid and introduce a new function, GetCurrentCommandID but this would dirty the code.

I will have to think about it.

Quote:

Originally Posted by Nero_3D [*]Benchmark script is outdated (on first page) some functions are undefined[/LIST]

Will update soon.

[QUOTE=Nero_3D;3811977][*]You should change the prefix and macro names, makes it impossible to test in one go
Do it like in ycmd, only add a compatibility macro while using your own namespace[/LIST]
I have no clue how YCMD works nor did I understand what you tried to say.

Quote:

Originally Posted by Nero_3D I know therefor I said "add a compatibility macro" then I could undef CMD and it still works without interfering The other reason was about the prefix "cmd_", if you change something and it isn't compatible to the old format you should change the prefix too

Will think about it.