Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Include QString #1298

Merged
merged 1 commit into from
Nov 13, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions rviz_common/include/rviz_common/display.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -36,6 +36,7 @@

#include <QIcon> // NOLINT: cpplint is unable to handle the include order here
#include <QSet> // NOLINT: cpplint is unable to handle the include order here
#include <QString> // NOLINT: cpplint is unable to handle the include order here

#include "rclcpp/time.hpp"

Expand Down
2 changes: 2 additions & 0 deletions rviz_common/include/rviz_common/display_group.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -32,6 +32,8 @@
#ifndef RVIZ_COMMON__DISPLAY_GROUP_HPP_
#define RVIZ_COMMON__DISPLAY_GROUP_HPP_

#include <QString>

#include "rviz_common/display.hpp"
#include "rviz_common/visibility_control.hpp"

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -32,6 +32,8 @@
#ifndef RVIZ_COMMON__FACTORY__CLASS_ID_RECORDING_FACTORY_HPP_
#define RVIZ_COMMON__FACTORY__CLASS_ID_RECORDING_FACTORY_HPP_

#include <QString>

#include "./factory.hpp"

namespace rviz_common
Expand Down
2 changes: 2 additions & 0 deletions rviz_common/include/rviz_common/message_filter_display.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -35,6 +35,8 @@
#include <memory>
#include <string>

#include <QString> // NOLINT: cpplint is unable to handle the include order here

#include <message_filters/subscriber.hpp>

#include "rviz_common/ros_topic_display.hpp"
Expand Down
1 change: 1 addition & 0 deletions rviz_common/include/rviz_common/panel.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -33,6 +33,7 @@
#ifndef RVIZ_COMMON__PANEL_HPP_
#define RVIZ_COMMON__PANEL_HPP_

#include <QString>
#include <QWidget>

#include "rviz_common/config.hpp"
Expand Down
1 change: 1 addition & 0 deletions rviz_common/include/rviz_common/panel_dock_widget.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -34,6 +34,7 @@

#include <QDockWidget>
#include <QLabel>
#include <QString>

#include "rviz_common/config.hpp"
#include "rviz_common/visibility_control.hpp"
Expand Down
2 changes: 2 additions & 0 deletions rviz_common/include/rviz_common/properties/bool_property.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -32,6 +32,8 @@
#ifndef RVIZ_COMMON__PROPERTIES__BOOL_PROPERTY_HPP_
#define RVIZ_COMMON__PROPERTIES__BOOL_PROPERTY_HPP_

#include <QString>

#include "rviz_common/properties/property.hpp"
#include "rviz_common/visibility_control.hpp"

Expand Down
1 change: 1 addition & 0 deletions rviz_common/include/rviz_common/properties/combo_box.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -31,6 +31,7 @@
#define RVIZ_COMMON__PROPERTIES__COMBO_BOX_HPP_

#include <QComboBox>
#include <QString>

#include "rviz_common/visibility_control.hpp"

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -38,6 +38,7 @@
#include <OgreColourValue.h>

#include <QColor> // NOLINT cpplint cannot handle include order here
#include <QString> // NOLINT cpplint cannot handle include order here

#include "rviz_common/properties/bool_property.hpp"
#include "rviz_common/visibility_control.hpp"
Expand Down
5 changes: 3 additions & 2 deletions rviz_common/include/rviz_common/properties/enum_property.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -32,10 +32,11 @@
#ifndef RVIZ_COMMON__PROPERTIES__ENUM_PROPERTY_HPP_
#define RVIZ_COMMON__PROPERTIES__ENUM_PROPERTY_HPP_

#include <QStringList>

#include <string>

#include <QString> // NOLINT: cpplint is unable to handle the include order here
#include <QStringList> // NOLINT: cpplint is unable to handle the include order here

#include "rviz_common/properties/string_property.hpp"
#include "rviz_common/visibility_control.hpp"

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -31,6 +31,8 @@
#ifndef RVIZ_COMMON__PROPERTIES__FILE_PICKER_PROPERTY_HPP_
#define RVIZ_COMMON__PROPERTIES__FILE_PICKER_PROPERTY_HPP_

#include <QString>

#include "rviz_common/properties/string_property.hpp"
#include "rviz_common/visibility_control.hpp"

Expand Down
2 changes: 2 additions & 0 deletions rviz_common/include/rviz_common/properties/float_property.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -32,6 +32,8 @@
#ifndef RVIZ_COMMON__PROPERTIES__FLOAT_PROPERTY_HPP_
#define RVIZ_COMMON__PROPERTIES__FLOAT_PROPERTY_HPP_

#include <QString>

#include "rviz_common/properties/property.hpp"
#include "rviz_common/visibility_control.hpp"

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -33,6 +33,8 @@

#include <memory>

#include <QString> // NOLINT: cpplint is unable to handle the include order here

#include "rviz_common/properties/bool_property.hpp"

namespace rviz_common
Expand Down
2 changes: 2 additions & 0 deletions rviz_common/include/rviz_common/properties/int_property.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -32,6 +32,8 @@
#ifndef RVIZ_COMMON__PROPERTIES__INT_PROPERTY_HPP_
#define RVIZ_COMMON__PROPERTIES__INT_PROPERTY_HPP_

#include <QString>

#include "rviz_common/properties/property.hpp"
#include "rviz_common/visibility_control.hpp"

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -33,6 +33,7 @@
#define RVIZ_COMMON__PROPERTIES__PROPERTY_TREE_MODEL_HPP_

#include <QAbstractItemModel>
#include <QString>

#include "rviz_common/visibility_control.hpp"

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -33,6 +33,7 @@
#define RVIZ_COMMON__PROPERTIES__PROPERTY_TREE_WIDGET_HPP_

#include <QTreeView>
#include <QString>

#include "./property_tree_model.hpp"
#include "rviz_common/config.hpp"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -33,6 +33,8 @@

#include <OgreQuaternion.h>

#include <QString>

#include "rviz_common/properties/property.hpp"
#include "rviz_common/visibility_control.hpp"

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -30,14 +30,15 @@
#ifndef RVIZ_COMMON__PROPERTIES__REGEX_FILTER_PROPERTY_HPP_
#define RVIZ_COMMON__PROPERTIES__REGEX_FILTER_PROPERTY_HPP_

#include <QValidator>
#include <QLineEdit>
#include <QToolTip>
#include <QWidget>

#include <regex>
#include <string>

#include <QValidator> // NOLINT: cpplint is unable to handle the include order here
#include <QLineEdit> // NOLINT: cpplint is unable to handle the include order here
#include <QString> // NOLINT: cpplint is unable to handle the include order here
#include <QToolTip> // NOLINT: cpplint is unable to handle the include order here
#include <QWidget> // NOLINT: cpplint is unable to handle the include order here

#include "rviz_common/properties/string_property.hpp"
#include "rviz_common/visibility_control.hpp"

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -32,6 +32,8 @@

#include <string>

#include <QString> // NOLINT: cpplint is unable to handle the include order here

#include "rviz_common/properties/editable_enum_property.hpp"
#include "rviz_common/ros_integration/ros_node_abstraction_iface.hpp"
#include "rviz_common/visibility_control.hpp"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -33,6 +33,7 @@
#define RVIZ_COMMON__PROPERTIES__STATUS_PROPERTY_HPP_

#include <QIcon>
#include <QString>

#include "rviz_common/properties/property.hpp"
#include "rviz_common/visibility_control.hpp"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -34,6 +34,8 @@

#include <string>

#include <QString> // NOLINT: cpplint is unable to handle the include order here

#include "rviz_common/properties/property.hpp"
#include "rviz_common/visibility_control.hpp"

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -33,6 +33,9 @@
#define RVIZ_COMMON__PROPERTIES__TF_FRAME_PROPERTY_HPP_

#include <string>

#include <QString> // NOLINT: cpplint is unable to handle the include order here

#include "rviz_common/frame_manager_iface.hpp"

#include "./editable_enum_property.hpp"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -34,6 +34,8 @@

#include <OgreVector.h>

#include <QString>

#include "rviz_common/properties/property.hpp"
#include "rviz_common/visibility_control.hpp"

Expand Down
2 changes: 2 additions & 0 deletions rviz_common/include/rviz_common/ros_topic_display.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -40,6 +40,8 @@
#include <OgreSceneNode.h>
#include <OgreSceneManager.h>

#include <QString> // NOLINT: cpplint is unable to handle the include order here

#endif

#include "rclcpp/qos.hpp"
Expand Down
1 change: 1 addition & 0 deletions rviz_common/include/rviz_common/tool_manager.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -37,6 +37,7 @@

#include <QList> // NOLINT: cpplint is unable to handle the include order here
#include <QObject> // NOLINT: cpplint is unable to handle the include order here
#include <QString> // NOLINT: cpplint is unable to handle the include order here
#include <QStringList> // NOLINT: cpplint is unable to handle the include order here

#include "rviz_common/factory/pluginlib_factory.hpp"
Expand Down
1 change: 1 addition & 0 deletions rviz_common/include/rviz_common/view_manager.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -37,6 +37,7 @@

#include <QList> // NOLINT: cpplint is unable to handle the include order here
#include <QObject> // NOLINT: cpplint is unable to handle the include order here
#include <QString> // NOLINT: cpplint is unable to handle the include order here
#include <QStringList> // NOLINT: cpplint is unable to handle the include order here

#include "rviz_common/properties/property.hpp"
Expand Down
2 changes: 2 additions & 0 deletions rviz_common/include/rviz_common/visualization_manager.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -36,6 +36,8 @@
#include <deque>
#include <memory>

#include <QString> // NOLINT: cpplint is unable to handle the include order here

#include "rclcpp/clock.hpp"
#include "rclcpp/time.hpp"
#include "tf2_ros/transform_listener.h"
Expand Down
1 change: 1 addition & 0 deletions rviz_common/include/rviz_common/visualizer_app.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -36,6 +36,7 @@

#include <QApplication> // NOLINT: cpplint is unable to handle the include order here
#include <QObject> // NOLINT: cpplint is unable to handle the include order here
#include <QString> // NOLINT: cpplint is unable to handle the include order here

#include "rviz_common/ros_integration/ros_client_abstraction_iface.hpp"
#include "rviz_common/visibility_control.hpp"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -33,6 +33,7 @@
#define RVIZ_COMMON__WINDOW_MANAGER_INTERFACE_HPP_

#include <Qt>
#include <QString>

class QWidget;
class QString;
Expand Down
2 changes: 2 additions & 0 deletions rviz_common/include/rviz_common/yaml_config_reader.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -34,6 +34,8 @@

#include <istream>

#include <QString> // NOLINT: cpplint is unable to handle the include order here

#include "rviz_common/config.hpp"
#include "rviz_common/visibility_control.hpp"

Expand Down
2 changes: 2 additions & 0 deletions rviz_common/include/rviz_common/yaml_config_writer.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -34,6 +34,8 @@

#include <ostream>

#include <QString> // NOLINT: cpplint is unable to handle the include order here

#include "rviz_common/config.hpp"
#include "rviz_common/visibility_control.hpp"

Expand Down
1 change: 1 addition & 0 deletions rviz_common/src/rviz_common/add_display_dialog.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -47,6 +47,7 @@
#include <QLabel> // NOLINT: cpplint is unable to handle the include order here
#include <QLineEdit> // NOLINT: cpplint is unable to handle the include order here
#include <QPushButton> // NOLINT: cpplint is unable to handle the include order here
#include <QString> // NOLINT: cpplint is unable to handle the include order here
#include <QTabWidget> // NOLINT: cpplint is unable to handle the include order here
#include <QTextBrowser> // NOLINT: cpplint is unable to handle the include order here
#include <QVBoxLayout> // NOLINT: cpplint is unable to handle the include order here
Expand Down
1 change: 1 addition & 0 deletions rviz_common/src/rviz_common/add_display_dialog.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -39,6 +39,7 @@
#include <QComboBox> // NOLINT: cpplint is unable to handle the include order here
#include <QDialog> // NOLINT: cpplint is unable to handle the include order here
#include <QMultiMap> // NOLINT: cpplint is unable to handle the include order here
#include <QString> // NOLINT: cpplint is unable to handle the include order here
#include <QTreeWidget> // NOLINT: cpplint is unable to handle the include order here

#include "rviz_common/factory/factory.hpp"
Expand Down
1 change: 1 addition & 0 deletions rviz_common/src/rviz_common/config.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -32,6 +32,7 @@
#include "rviz_common/config.hpp"

#include <QLocale>
#include <QString>

namespace rviz_common
{
Expand Down
1 change: 1 addition & 0 deletions rviz_common/src/rviz_common/display.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -43,6 +43,7 @@
#include <QFont> // NOLINT: cpplint is unable to handle the include order here
#include <QMetaObject> // NOLINT: cpplint is unable to handle the include order here
#include <QWidget> // NOLINT: cpplint is unable to handle the include order here
#include <QString> // NOLINT: cpplint is unable to handle the include order here

#include "rclcpp/time.hpp"
#include "rclcpp/rclcpp.hpp"
Expand Down
2 changes: 2 additions & 0 deletions rviz_common/src/rviz_common/display_factory.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -35,6 +35,8 @@

#include <tinyxml2.h> // NOLINT: cpplint is unable to handle the include order here

#include <QString> // NOLINT: cpplint is unable to handle the include order here

#include "rviz_common/display_group.hpp"
#include "rviz_common/logging.hpp"

Expand Down
1 change: 1 addition & 0 deletions rviz_common/src/rviz_common/display_group.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -35,6 +35,7 @@
#include <map>

#include <QColor> // NOLINT: cpplint is unable to handle the include order here
#include <QString> // NOLINT: cpplint is unable to handle the include order here

#include "./display_factory.hpp"
#include "./failed_display.hpp"
Expand Down
1 change: 1 addition & 0 deletions rviz_common/src/rviz_common/displays_panel.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -39,6 +39,7 @@
#include <QInputDialog> // NOLINT: cpplint is unable to handle the include order here
#include <QProgressDialog> // NOLINT: cpplint is unable to handle the include order here
#include <QPushButton> // NOLINT: cpplint is unable to handle the include order here
#include <QString> // NOLINT: cpplint is unable to handle the include order here
#include <QTimer> // NOLINT: cpplint is unable to handle the include order here
#include <QVBoxLayout> // NOLINT: cpplint is unable to handle the include order here

Expand Down
1 change: 1 addition & 0 deletions rviz_common/src/rviz_common/failed_display.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -32,6 +32,7 @@
#include "failed_display.hpp"

#include <QColor> // NOLINT: cpplint is unable to handle the include order here
#include <QString> // NOLINT: cpplint is unable to handle the include order here

#include "rviz_common/load_resource.hpp"
#include "rviz_common/properties/status_property.hpp"
Expand Down
2 changes: 2 additions & 0 deletions rviz_common/src/rviz_common/failed_display.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -32,6 +32,8 @@
#ifndef RVIZ_COMMON__FAILED_DISPLAY_HPP_
#define RVIZ_COMMON__FAILED_DISPLAY_HPP_

#include <QString>

#include "rviz_common/display.hpp"

namespace rviz_common
Expand Down
1 change: 1 addition & 0 deletions rviz_common/src/rviz_common/failed_panel.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -34,6 +34,7 @@
#include <cstdio>

#include <QHBoxLayout> // NOLINT: cpplint is unable to handle the include order here
#include <QString> // NOLINT: cpplint is unable to handle the include order here
#include <QTextBrowser> // NOLINT: cpplint is unable to handle the include order here

namespace rviz_common
Expand Down
2 changes: 2 additions & 0 deletions rviz_common/src/rviz_common/failed_panel.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -34,6 +34,8 @@

#include <memory>

#include <QString> // NOLINT: cpplint is unable to handle the include order here

#include "rviz_common/panel.hpp"

namespace rviz_common
Expand Down
1 change: 1 addition & 0 deletions rviz_common/src/rviz_common/failed_tool.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -32,6 +32,7 @@
#include "failed_tool.hpp"

#include <QMessageBox>
#include <QString>

#include "rviz_common/window_manager_interface.hpp"
#include "rviz_common/display_context.hpp"
Expand Down
1 change: 1 addition & 0 deletions rviz_common/src/rviz_common/failed_view_controller.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -32,6 +32,7 @@
#include "failed_view_controller.hpp"

#include <QMessageBox>
#include <QString>

#include "rviz_common/display_context.hpp"
#include "rviz_common/window_manager_interface.hpp"
Expand Down
Loading