On Fri, Dec 13, 2013 at 19:42, Philip Guenther wrote: > +static __inline int > +process_is_multithreaded(struct process *pr) > +{ > + struct proc *p = TAILQ_FIRST(&pr->ps_threads); > + return (p != NULL && TAILQ_NEXT(p, p_thr_link) != NULL);
This list will never be empty, so you can drop the p != null check.