> > if ( !str_cmp(word,"Clan") )
> > {
> >     char name[MIL] = fread_string(fp);
> >     ch->clan = clan_lookup(name);
> >     fMatch = TRUE;
> >     break;
> > }

Hello,
Wouldn't you be using MSL instead of MIL? And from that block of code,
going by how I've written clan stuff before, it's not close to how i'd
do it or how I've ever seen it done.

Clan lookup would normally be an arg: Name would = argument

->clan = clan_lookup(argument);

Then you free it with fread_string, Clan lookup by name is complete,
now free it.

->clan = clan_lookup(fread_string(fp));

That is just off the top of my head because without knowing where you
are calling that and how it's being used. I can't even begin to think
where it would be or why it's written like that.

Just my thoughts,
Chris

Reply via email to