Merge pull request #161 from ros2/fix_import_symbols

Fix cpplint and windows build

rclcpp/include/rclcpp/node.hpp

@@ -172,7 +172,6 @@ public:
    * \param[in] group Callback group to execute this timer's callback in.
    */
   template<typename CallbackType>
-  RCLCPP_PUBLIC
   typename rclcpp::timer::WallTimer<CallbackType>::SharedPtr
   create_wall_timer(
     std::chrono::nanoseconds period,

rclcpp/include/rclcpp/timer.hpp

@@ -41,7 +41,7 @@ public:
   RCLCPP_SMART_PTR_DEFINITIONS_NOT_COPYABLE(TimerBase);
 
   RCLCPP_PUBLIC
-  TimerBase(std::chrono::nanoseconds period);
+  explicit TimerBase(std::chrono::nanoseconds period);
 
   RCLCPP_PUBLIC
   virtual ~TimerBase();
@@ -142,7 +142,6 @@ public:
   void
   execute_callback_delegate()
   {
-    //callback_(std::move(std::shared_ptr<TimerBase>(this)));
     callback_(*this);
   }