> for(i=0;guilds[i].class != -1;i++) > if (GET_CLASS(ch) == guilds[i].class && world[ch->in_room].number == > guilds[i].guild_room) { > found = TRUE; > continue; > }else > i++; you're double incrementing it now :)..the i++ in the for loop takes care of it, so you can get rid of the else part..also, you can probably break out of the for loop once you've found the room and class, since logically a person can't be in two rooms at once..and most likely won't have more than one class.. that's about all that i saw was incorrecxt, though i'd do a couple things different that i think i already told you about.. siv +------------------------------------------------------------+ | Ensure that you have read the CircleMUD Mailing List FAQ: | | http://democracy.queensu.ca/~fletcher/Circle/list-faq.html | +------------------------------------------------------------+
This archive was generated by hypermail 2b30 : 12/08/00 PST