for ( ch = char_list; ch != NULL; ch = ch->next )
{
ch_next = ch->next;
for (obj = mud->f_obj ; obj ; obj = mud->update_obj)
{
mud->update_obj = obj->next;
void extract_obj( OBJ_DATA *obj )
{
if (mud->update_obj == obj)
{
mud->update_obj = obj->next;
}
I have an old bug report from the ROM list where someone had trouble with this while making corpses. Seems another function modified the obj_next and caused a crash. The posted fix was this:
Now, I have 32 occurrences of the first version, and I'm wondering if there's a reason for writing it that way, or if it's okay to replace them all with the second version, just for consistency (and possible bug-proofing against future changes).