get rid of unrelated changes.

Signed-off-by: Tomoya.Fujita <[email protected]>

Author: fujitatomoya

rclcpp/src/rclcpp/init_options.cpp

@@ -44,7 +44,6 @@ InitOptions::InitOptions(const InitOptions & other)
 : InitOptions(*other.get_rcl_init_options())
 {
   shutdown_on_sigint = other.shutdown_on_sigint;
-  initialize_logging_ = other.initialize_logging_;
 }
 
 bool
@@ -71,7 +70,6 @@ InitOptions::operator=(const InitOptions & other)
       rclcpp::exceptions::throw_from_rcl_error(ret, "failed to copy rcl init options");
     }
     this->shutdown_on_sigint = other.shutdown_on_sigint;
-    this->initialize_logging_ = other.initialize_logging_;
   }
   return *this;
 }
@@ -106,7 +104,6 @@ InitOptions::finalize_init_options_impl()
 const rcl_init_options_t *
 InitOptions::get_rcl_init_options() const
 {
-  std::lock_guard<std::mutex> init_options_lock(init_options_mutex_);
   return init_options_.get();
 }
 

rclcpp/test/rclcpp/test_init_options.cpp

@@ -45,38 +45,9 @@ TEST(TestInitOptions, test_construction) {
   }
 }
 
-TEST(TestInitOptions, test_initialize_logging) {
-  {
-    auto options = rclcpp::InitOptions();
-    EXPECT_TRUE(options.auto_initialize_logging());
-    const rcl_init_options_t * rcl_options = options.get_rcl_init_options();
-    ASSERT_TRUE(rcl_options != nullptr);
-    ASSERT_TRUE(rcl_options->impl != nullptr);
-  }
-
-  {
-    auto options = rclcpp::InitOptions().auto_initialize_logging(true);
-    EXPECT_TRUE(options.auto_initialize_logging());
-    const rcl_init_options_t * rcl_options = options.get_rcl_init_options();
-    ASSERT_TRUE(rcl_options != nullptr);
-    ASSERT_TRUE(rcl_options->impl != nullptr);
-  }
-
-  {
-    auto options = rclcpp::InitOptions().auto_initialize_logging(false);
-    EXPECT_FALSE(options.auto_initialize_logging());
-    const rcl_init_options_t * rcl_options = options.get_rcl_init_options();
-    ASSERT_TRUE(rcl_options != nullptr);
-    ASSERT_TRUE(rcl_options->impl != nullptr);
-  }
-}
-
 TEST(TestInitOptions, test_domain_id) {
   rcl_allocator_t allocator = rcl_get_default_allocator();
   auto options = rclcpp::InitOptions(allocator);
-  const rcl_init_options_t * rcl_options = options.get_rcl_init_options();
-  ASSERT_TRUE(rcl_options != nullptr);
-  ASSERT_TRUE(rcl_options->impl != nullptr);
 
   options.use_default_domain_id();
   EXPECT_EQ(RCL_DEFAULT_DOMAIN_ID, options.get_domain_id());