commit 64018324d5e5071eea3b3f72f939d91dc7aef029
author Jorge Boncompte [DTI2] <jorge@dti2.net> Mon May 07 16:53:13 2012 +0000
committer David Lamparter <equinox@opensourcerouting.org> Tue May 22 20:35:51 2012 +0200
tree faa7378fe45c6a11cf5198ba182f9bf54f3022f5
parent 41af338ecd56b9891d996f0fc850e02318b0c01a

lib: micro-op for thread_get()

thread_trim_head() already checks that the list is not empty.

Signed-off-by: Jorge Boncompte [DTI2] <jorge@dti2.net>
Signed-off-by: David Lamparter <equinox@opensourcerouting.org>

lib/thread.c

diff --git a/lib/thread.c b/lib/thread.c
index 784b29c..3740147 100644
--- a/lib/thread.c
+++ b/lib/thread.c
@@ -667,11 +667,10 @@
 thread_get (struct thread_master *m, u_char type,
 	    int (*func) (struct thread *), void *arg, const char* funcname)
 {
-  struct thread *thread;
+  struct thread *thread = thread_trim_head (&m->unuse);
 
-  if (!thread_empty (&m->unuse))
+  if (thread)
     {
-      thread = thread_trim_head (&m->unuse);
       if (thread->funcname)
         XFREE(MTYPE_THREAD_FUNCNAME, thread->funcname);
     }