Ctests fixes

Author: Mauro Passerino

rclcpp/include/rclcpp/any_subscription_callback.hpp

@@ -187,7 +187,7 @@ class AnySubscriptionCallback
     TRACEPOINT(callback_start, static_cast<const void *>(this), true);
 
     // If the message is not valid, return.
-    if(!message) {
+    if (!message) {
       return;
     }
 
@@ -216,7 +216,7 @@ class AnySubscriptionCallback
     TRACEPOINT(callback_start, static_cast<const void *>(this), true);
 
     // If the message is not valid, return.
-    if(!message) {
+    if (!message) {
       return;
     }
 

rclcpp/include/rclcpp/experimental/buffers/ring_buffer_implementation.hpp

@@ -93,9 +93,9 @@ class RingBufferImplementation : public BufferImplementationBase<BufferT>
       RCLCPP_ERROR(rclcpp::get_logger("rclcpp"), "Calling dequeue on empty intra-process buffer");
       // This situation can happen on the EventsExecutor if we have more events in the queue
       // than messages in the history cache (set by the qos_policies.depth of the subscription)
-      // For example if we set depth=1 and we get 2 messages really fast (so no time for processing),
-      // we could end up with 2 events in the queue but only 1 msg is actually stored on the buffer.
-      // In this case we return an empty buffer.
+      // For example if we set depth=1 and we get 2 messages really fast (so no time for
+      // processing), we could end up with 2 events in the queue but only 1 msg is
+      // actually stored on the buffer. In this case we return an empty buffer.
       return BufferT();
     }
 

rclcpp/test/rclcpp/executors/test_events_queue.cpp

@@ -11,11 +11,12 @@
 // WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 // See the License for the specific language governing permissions and
 // limitations under the License.
+#include <gtest/gtest.h>
+
 #include <chrono>
+#include <memory>
 #include <queue>
 
-#include <gtest/gtest.h>
-
 #include "rclcpp/experimental/buffers/simple_events_queue.hpp"
 
 using namespace std::chrono_literals;