Browse Source

Remove dead test.

Brian Aker 14 years ago
parent
commit
9fdefac14c
4 changed files with 3 additions and 6 deletions
  1. 0 1
      libgearman/connection.hpp
  2. 3 0
      libgearman/function/base.hpp
  3. 0 3
      libgearman/worker.cc
  4. 0 2
      tests/internals.cc

+ 0 - 1
libgearman/connection.hpp

@@ -45,7 +45,6 @@ struct gearman_connection_st
   struct {
     bool ready;
     bool packet_in_use;
-    bool external_fd;
     bool ignore_lost_connection;
     bool close_after_flush;
   } options;

+ 3 - 0
libgearman/function/base.hpp

@@ -100,6 +100,9 @@ struct _worker_function_st
 
   virtual ~_worker_function_st()
   {
+    if (options.packet_in_use)
+      gearman_packet_free(&packet);
+
     delete [] function_name;
   }
 };

+ 0 - 3
libgearman/worker.cc

@@ -1160,9 +1160,6 @@ static void _worker_function_free(gearman_worker_st *worker,
     function->next->prev= function->prev;
   worker->function_count--;
 
-  if (function->options.packet_in_use)
-    gearman_packet_free(&(function->packet));
-
   delete function;
 }
 

+ 0 - 2
tests/internals.cc

@@ -250,7 +250,6 @@ static test_return_t connection_init_test(void *)
 
   test_false(connection_ptr->options.ready);
   test_false(connection_ptr->options.packet_in_use);
-  test_false(connection_ptr->options.external_fd);
   test_false(connection_ptr->options.ignore_lost_connection);
   test_false(connection_ptr->options.close_after_flush);
 
@@ -270,7 +269,6 @@ static test_return_t connection_alloc_test(void *)
 
   test_false(connection_ptr->options.ready);
   test_false(connection_ptr->options.packet_in_use);
-  test_false(connection_ptr->options.external_fd);
   test_false(connection_ptr->options.ignore_lost_connection);
   test_false(connection_ptr->options.close_after_flush);