[joropito]

I have a really dumb question but I need the answer.

If I use MF_GetAmxString to get the pointer of the parameter as string. It's necessary to free it at function end?

And what happens if I want to keep using that pointer during the game?

[claudiuhks]

Try to free it. If testing server will crash, freeing is not required. Otherwise, it's okay.
That's my opinion.

[micapat]

It's certainly AMXX Module who manages this.
Just look how works "entity_set_string" for instance

If you want to use the string during the game, copy it in another one, or call ALLOC_STRING / STRING.

[joropito]

Originally Posted by micapat It's certainly AMXX Module who manages this. Just look how works "entity_set_string" for instance If you want to use the string during the game, copy it in another one, or call ALLOC_STRING / STRING.

Yes. Currently I'm copying the string to a newly allocated pointer to be safe.
Thanks

[Arkshine]

He's talking about MF_GetAmxString, where you have such code :

...
register char* dest = buffor[id];
char* start = dest;
...
return start;

AFAIK, register tries to keep the same pointer, which means it would not be a good idea to free it.
And since he needs to keep using the same pointer during the game, the answer is he should make a copy of the string.

EDIT: Ninja'd.

[gladius]

Use free() when pointer does point to a block of memory previously allocated with malloc, calloc or realloc. Otherwise, if pointer does not point to a block of memory allocated with the above functions, the behavior is undefined. And if is a NULL pointer, the function does nothing.

Also, if you wanna keep your pointer during the game, copy it and never change original/header Pointer.

Ex(Linked list)
Calling & setting Header...

Node *head;
head = new Node(value);

On another function...
Calling & Setting Linked list...

Node *aux = head;
while(aux)
{
         aux = aux->next;
}
aux = new Node(value);

On another function...
Calling & Displaying Linked list...

Node *aux = head;
while(aux)
{
         cout << aux->GetValue() << endl;
         aux = aux->next;
}

And I will never lose my first datum. (works like your question)
Not the best example but I wanna show you how to keep a pointer and not lose its value.