Skip to content

Commit 6507d42

Browse files
committed
fix: new clang-tidy errors
1 parent f5fadcb commit 6507d42

6 files changed

Lines changed: 4 additions & 8 deletions

File tree

src/streamers/image_transport_streamer.cpp

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -54,6 +54,7 @@
5454
#include "image_transport/image_transport.hpp"
5555
#include "rclcpp/node.hpp"
5656
#include "rclcpp/logging.hpp"
57+
#include "rclcpp/qos.hpp"
5758
#include "rmw/qos_profiles.h"
5859
#include "sensor_msgs/msg/image.hpp"
5960

src/streamers/jpeg_streamers.cpp

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,6 @@
3333
#include <chrono>
3434
#include <cstdint>
3535
#include <cstdio>
36-
#include <cstring>
3736
#include <memory>
3837
#include <mutex>
3938
#include <string>

src/streamers/png_streamers.cpp

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,6 @@
3232
#include <chrono>
3333
#include <cstdint>
3434
#include <cstdio>
35-
#include <cstring>
3635
#include <memory>
3736
#include <mutex>
3837
#include <string>

src/streamers/vp8_streamer.cpp

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,6 @@ extern "C"
3636
#include <libavutil/opt.h>
3737
}
3838

39-
#include <cstring>
4039
#include <map>
4140
#include <memory>
4241

src/streamers/vp9_streamer.cpp

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,6 @@ extern "C"
3535
#include <libavutil/opt.h>
3636
}
3737

38-
#include <cstring>
3938
#include <memory>
4039

4140
#include "async_web_server_cpp/http_connection.hpp"

src/web_video_server.cpp

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,6 @@
3232

3333
#include <algorithm>
3434
#include <chrono>
35-
#include <cstring>
3635
#include <exception>
3736
#include <map>
3837
#include <memory>
@@ -205,7 +204,7 @@ bool WebVideoServer::handle_stream(
205204
const char * end)
206205
{
207206
const std::string type = request.get_query_param_value_or_default("type", default_stream_type_);
208-
if (streamer_factories_.find(type) != streamer_factories_.end()) {
207+
if (streamer_factories_.contains(type)) {
209208
const std::shared_ptr<StreamerInterface> streamer = streamer_factories_[type]->create_streamer(
210209
request, connection, weak_from_this());
211210
streamer->start();
@@ -224,7 +223,7 @@ bool WebVideoServer::handle_snapshot(
224223
const char * end)
225224
{
226225
const std::string type = request.get_query_param_value_or_default("type", default_snapshot_type_);
227-
if (snapshot_streamer_factories_.find(type) != snapshot_streamer_factories_.end()) {
226+
if (snapshot_streamer_factories_.contains(type)) {
228227
const std::shared_ptr<StreamerInterface> streamer =
229228
snapshot_streamer_factories_[type]->create_streamer(
230229
request, connection, weak_from_this());
@@ -244,7 +243,7 @@ bool WebVideoServer::handle_stream_viewer(
244243
const char * end)
245244
{
246245
const std::string type = request.get_query_param_value_or_default("type", default_stream_type_);
247-
if (streamer_factories_.find(type) != streamer_factories_.end()) {
246+
if (streamer_factories_.contains(type)) {
248247
const std::string topic = request.get_query_param_value_or_default("topic", "");
249248

250249
async_web_server_cpp::HttpReply::builder(async_web_server_cpp::HttpReply::ok)

0 commit comments

Comments
 (0)