* Replace "nstr = malloc(strlen(str) + 1); strcpy(nstr, str);" by using "nstr = strdup(str);" instead
* Remove a non-sensical self-assignment ("pIdent = pIdent;")
* In scriptAddVariable:
* "Overallocate" for the new VAR_SYMBOL so that we can store the identifier in that single chunk of memory
* This allows deallocating the entire structure with a single free() call (which prevents problems with the constness change to VAR_SYMBOL mentioned above)
* Don't take ownership of the memory pointed to by psVarIdent->pIdent, just copy its contents instead
* Make FREE_VARIDENTDECL a function instead: freeVARIDENTDECL
* In this function also free psDcl->pIdent
git-svn-id: svn+ssh://svn.gna.org/svn/warzone/trunk@4696 4a71c877-e1ca-e34f-864e-861f7616d084