std::string is wrapped to hell, so when you call operator= (mystring = somecharptr), it's actually copying what's at that pointer to it's own buffer.
Anything with * by it is a pointer, meaning that the variable holds nothing but an address, so a char* is just the address of a single character. A null-terminated string is a sequence of characters terminated by a null (char = to 0), but since you can't hold a whole string in a handy little 32 bit package, you put it somewhere in memory and reference it [the beginning of it] by it's address through a pointer.
When you have char* shit = engine->Cmd_Argv(2), you are retrieving the address of a string in the engine's memory containing the second argument. After your command is executed, lord only knows wtf the engine will do with the memory at that address. You need to copy what's there, when it's good, into something you control (your std::string, or a char array of your own).
As for
Code:
Delayed Delayeds[40];
/* for (int x = 0; x <= 40; x++)
Delayeds[x].used = 0; // Delete all unterminated*/
// Crashs the server :|
That's because Delayeds has 40 elements, indexed 0 - 39. C[++] has no bounds checking (thank God), so Delayeds[40] will reference the memory just after the last Delayed, which in this case is bad mojo. Change <= to <.
For this, however, I recommend you learn std::vector. It kicks ass:
Code:
typedef struct {
std::string strCmd;
float flTime;
} DelayedCmd_s;
std::vector<DelayedCmd_s*> g_vDelayedCmds;
void AddDelayedCmd(const char* lpcCmd, float flTime){
DelayedCmd_s* Cmd = new DelayedCmd_s;
Cmd->strCmd = lpcCmd;
Cmd->flTime = flTime;
g_vDelayedCmds.push_back(Cmd);
}
void UpdateCmds(){
float flNow = engine->Time();
for(unsigned int i = 0; i < g_vDelayedCmds.size(); i++){
if(g_vDelayedCmds[i]->flTime <= flNow){
engine->ServerCommand(g_vDelayedCmds[i]->strCmd.c_str());
delete g_vDelayedCmds[i];
g_vDelayedCmds.erase(g_vDelayedCmds.begin() + i);
i--;
}
}
}
or some shit
purists will say use std::list and iterators. they can go fuck themselves
)
Linear Mode
