*groan* in Developer Help

ID:262918
 
Jan 20 2006, 2:36 pm
This simple proc I tried to use to create a list of players. It not work!!!
proc
    PlayerList()
        var/list/List
        for(var/mob/M in world)
            if(ismob(M))
                if(M.client)
                    List += "[M.name]"
        List += "Cancel"
        return List

It, at runtime, yells at me, calling M null at the if(M.client) line. WHY??? What am I doing wrong this time?

--Vito
#1 Jan 20 2006, 2:54 pm
Try changing if(ismob(M)) to if(M&&ismob(M)).

Edit:
Also, I think you need to change your var/list/L line to var/list/L = new().

Hiead
#2 Jan 20 2006, 2:57 pm
In response to Hiead (#1)
Yeah, got it working. I gotta tweak something else now in my all-day tinker spree.


--Vito