From 0c8b5b5c4d62110515b29cca76d1d847b69e7f91 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?R=C3=A9mi=20Verschelde?= Date: Tue, 4 May 2021 16:40:33 +0200 Subject: [PATCH] Style: Remove redundant void argument lists Using clang-tidy's `modernize-redundant-void-arg`. https://clang.llvm.org/extra/clang-tidy/checks/modernize-redundant-void-arg.html --- .clang-tidy | 2 +- main/tests/test_astar.cpp | 2 +- main/tests/test_ordered_hash_map.cpp | 2 +- main/tests/test_string.cpp | 2 +- .../include/videodecoder/godot_videodecoder.h | 2 +- modules/theora/video_stream_theora.cpp | 2 +- modules/theora/video_stream_theora.h | 2 +- scene/3d/arvr_nodes.cpp | 8 ++++---- scene/3d/arvr_nodes.h | 12 ++++++------ scene/resources/environment.cpp | 2 +- scene/resources/environment.h | 2 +- scene/resources/theme.cpp | 2 +- scene/resources/theme.h | 2 +- servers/physics/joints/generic_6dof_joint_sw.cpp | 2 +- servers/physics/joints/generic_6dof_joint_sw.h | 2 +- servers/physics/joints/slider_joint_sw.cpp | 10 +++++----- servers/physics/joints/slider_joint_sw.h | 10 +++++----- 17 files changed, 33 insertions(+), 33 deletions(-) diff --git a/.clang-tidy b/.clang-tidy index 2906e66c8dd..49250ce037e 100644 --- a/.clang-tidy +++ b/.clang-tidy @@ -1,5 +1,5 @@ --- -Checks: 'clang-diagnostic-*,clang-analyzer-*,-*,modernize-use-bool-literals,modernize-use-nullptr' +Checks: 'clang-diagnostic-*,clang-analyzer-*,-*,modernize-redundant-void-arg,modernize-use-bool-literals,modernize-use-nullptr' WarningsAsErrors: '' HeaderFilterRegex: '.*' AnalyzeTemporaryDtors: false diff --git a/main/tests/test_astar.cpp b/main/tests/test_astar.cpp index 3f77648e882..deadbaa6090 100644 --- a/main/tests/test_astar.cpp +++ b/main/tests/test_astar.cpp @@ -352,7 +352,7 @@ bool test_solutions() { return true; } -typedef bool (*TestFunc)(void); +typedef bool (*TestFunc)(); TestFunc test_funcs[] = { test_abc, diff --git a/main/tests/test_ordered_hash_map.cpp b/main/tests/test_ordered_hash_map.cpp index 301843d51be..176bd37022b 100644 --- a/main/tests/test_ordered_hash_map.cpp +++ b/main/tests/test_ordered_hash_map.cpp @@ -130,7 +130,7 @@ bool test_const_iteration() { return test_const_iteration(map); } -typedef bool (*TestFunc)(void); +typedef bool (*TestFunc)(); TestFunc test_funcs[] = { diff --git a/main/tests/test_string.cpp b/main/tests/test_string.cpp index c2b145320cd..558cdc694a7 100644 --- a/main/tests/test_string.cpp +++ b/main/tests/test_string.cpp @@ -1094,7 +1094,7 @@ bool test_35() { return state; } -typedef bool (*TestFunc)(void); +typedef bool (*TestFunc)(); TestFunc test_funcs[] = { diff --git a/modules/gdnative/include/videodecoder/godot_videodecoder.h b/modules/gdnative/include/videodecoder/godot_videodecoder.h index f226114f5b5..bb85ae0df4e 100644 --- a/modules/gdnative/include/videodecoder/godot_videodecoder.h +++ b/modules/gdnative/include/videodecoder/godot_videodecoder.h @@ -46,7 +46,7 @@ typedef struct void *next; void *(*constructor)(godot_object *); void (*destructor)(void *); - const char *(*get_plugin_name)(void); + const char *(*get_plugin_name)(); const char **(*get_supported_extensions)(int *count); godot_bool (*open_file)(void *, void *); // data struct, and a FileAccess pointer godot_real (*get_length)(const void *); diff --git a/modules/theora/video_stream_theora.cpp b/modules/theora/video_stream_theora.cpp index dd2105f0b2c..f6ac9c35bd7 100644 --- a/modules/theora/video_stream_theora.cpp +++ b/modules/theora/video_stream_theora.cpp @@ -79,7 +79,7 @@ int VideoStreamPlaybackTheora::queue_page(ogg_page *page) { return 0; } -void VideoStreamPlaybackTheora::video_write(void) { +void VideoStreamPlaybackTheora::video_write() { th_ycbcr_buffer yuv; th_decode_ycbcr_out(td, yuv); diff --git a/modules/theora/video_stream_theora.h b/modules/theora/video_stream_theora.h index bd915566b1e..fe4691ea295 100644 --- a/modules/theora/video_stream_theora.h +++ b/modules/theora/video_stream_theora.h @@ -63,7 +63,7 @@ class VideoStreamPlaybackTheora : public VideoStreamPlayback { int buffer_data(); int queue_page(ogg_page *page); - void video_write(void); + void video_write(); float get_time() const; bool theora_eos; diff --git a/scene/3d/arvr_nodes.cpp b/scene/3d/arvr_nodes.cpp index 355a62f9374..f41cc63ff2a 100644 --- a/scene/3d/arvr_nodes.cpp +++ b/scene/3d/arvr_nodes.cpp @@ -273,11 +273,11 @@ void ARVRController::set_controller_id(int p_controller_id) { update_configuration_warning(); }; -int ARVRController::get_controller_id(void) const { +int ARVRController::get_controller_id() const { return controller_id; }; -String ARVRController::get_controller_name(void) const { +String ARVRController::get_controller_name() const { // get our ARVRServer ARVRServer *arvr_server = ARVRServer::get_singleton(); ERR_FAIL_NULL_V(arvr_server, String()); @@ -475,7 +475,7 @@ void ARVRAnchor::set_anchor_id(int p_anchor_id) { update_configuration_warning(); }; -int ARVRAnchor::get_anchor_id(void) const { +int ARVRAnchor::get_anchor_id() const { return anchor_id; }; @@ -483,7 +483,7 @@ Vector3 ARVRAnchor::get_size() const { return size; }; -String ARVRAnchor::get_anchor_name(void) const { +String ARVRAnchor::get_anchor_name() const { // get our ARVRServer ARVRServer *arvr_server = ARVRServer::get_singleton(); ERR_FAIL_NULL_V(arvr_server, String()); diff --git a/scene/3d/arvr_nodes.h b/scene/3d/arvr_nodes.h index ca868901280..c5a511a94ba 100644 --- a/scene/3d/arvr_nodes.h +++ b/scene/3d/arvr_nodes.h @@ -82,8 +82,8 @@ protected: public: void set_controller_id(int p_controller_id); - int get_controller_id(void) const; - String get_controller_name(void) const; + int get_controller_id() const; + String get_controller_name() const; int get_joystick_id() const; int is_button_pressed(int p_button) const; @@ -95,7 +95,7 @@ public: bool get_is_active() const; ARVRPositionalTracker::TrackerHand get_hand() const; - Ref get_mesh(void) const; + Ref get_mesh() const; String get_configuration_warning() const; @@ -123,15 +123,15 @@ protected: public: void set_anchor_id(int p_anchor_id); - int get_anchor_id(void) const; - String get_anchor_name(void) const; + int get_anchor_id() const; + String get_anchor_name() const; bool get_is_active() const; Vector3 get_size() const; Plane get_plane() const; - Ref get_mesh(void) const; + Ref get_mesh() const; String get_configuration_warning() const; diff --git a/scene/resources/environment.cpp b/scene/resources/environment.cpp index 365dafa2237..a3bab0e6c55 100644 --- a/scene/resources/environment.cpp +++ b/scene/resources/environment.cpp @@ -146,7 +146,7 @@ float Environment::get_ambient_light_energy() const { float Environment::get_ambient_light_sky_contribution() const { return ambient_sky_contribution; } -int Environment::get_camera_feed_id(void) const { +int Environment::get_camera_feed_id() const { return camera_feed_id; } diff --git a/scene/resources/environment.h b/scene/resources/environment.h index 672f603e8d2..f4fb001da3e 100644 --- a/scene/resources/environment.h +++ b/scene/resources/environment.h @@ -207,7 +207,7 @@ public: Color get_ambient_light_color() const; float get_ambient_light_energy() const; float get_ambient_light_sky_contribution() const; - int get_camera_feed_id(void) const; + int get_camera_feed_id() const; void set_tonemapper(ToneMapper p_tone_mapper); ToneMapper get_tonemapper() const; diff --git a/scene/resources/theme.cpp b/scene/resources/theme.cpp index c75b09a9a8b..62bddf0a37d 100644 --- a/scene/resources/theme.cpp +++ b/scene/resources/theme.cpp @@ -66,7 +66,7 @@ PoolVector Theme::_get_stylebox_list(const String &p_node_type) const { return ilret; } -PoolVector Theme::_get_stylebox_types(void) const { +PoolVector Theme::_get_stylebox_types() const { PoolVector ilret; List il; diff --git a/scene/resources/theme.h b/scene/resources/theme.h index 95db1cd7e3b..33d3517d641 100644 --- a/scene/resources/theme.h +++ b/scene/resources/theme.h @@ -53,7 +53,7 @@ class Theme : public Resource { PoolVector _get_icon_list(const String &p_node_type) const; PoolVector _get_stylebox_list(const String &p_node_type) const; - PoolVector _get_stylebox_types(void) const; + PoolVector _get_stylebox_types() const; PoolVector _get_font_list(const String &p_node_type) const; PoolVector _get_color_list(const String &p_node_type) const; PoolVector _get_constant_list(const String &p_node_type) const; diff --git a/servers/physics/joints/generic_6dof_joint_sw.cpp b/servers/physics/joints/generic_6dof_joint_sw.cpp index b2ceaf1ad79..a86109192e1 100644 --- a/servers/physics/joints/generic_6dof_joint_sw.cpp +++ b/servers/physics/joints/generic_6dof_joint_sw.cpp @@ -412,7 +412,7 @@ real_t Generic6DOFJointSW::getAngle(int axis_index) const { return m_calculatedAxisAngleDiff[axis_index]; } -void Generic6DOFJointSW::calcAnchorPos(void) { +void Generic6DOFJointSW::calcAnchorPos() { real_t imA = A->get_inv_mass(); real_t imB = B->get_inv_mass(); real_t weight; diff --git a/servers/physics/joints/generic_6dof_joint_sw.h b/servers/physics/joints/generic_6dof_joint_sw.h index 4a6c1d53290..52db3ba4734 100644 --- a/servers/physics/joints/generic_6dof_joint_sw.h +++ b/servers/physics/joints/generic_6dof_joint_sw.h @@ -363,7 +363,7 @@ public: return B; } - virtual void calcAnchorPos(void); // overridable + virtual void calcAnchorPos(); // overridable void set_param(Vector3::Axis p_axis, PhysicsServer::G6DOFJointAxisParam p_param, real_t p_value); real_t get_param(Vector3::Axis p_axis, PhysicsServer::G6DOFJointAxisParam p_param) const; diff --git a/servers/physics/joints/slider_joint_sw.cpp b/servers/physics/joints/slider_joint_sw.cpp index 8defb19d901..d4dc403f02f 100644 --- a/servers/physics/joints/slider_joint_sw.cpp +++ b/servers/physics/joints/slider_joint_sw.cpp @@ -305,7 +305,7 @@ void SliderJointSW::solve(real_t p_step) { //----------------------------------------------------------------------------- -void SliderJointSW::calculateTransforms(void) { +void SliderJointSW::calculateTransforms() { m_calculatedTransformA = A->get_transform() * m_frameInA; m_calculatedTransformB = B->get_transform() * m_frameInB; m_realPivotAInW = m_calculatedTransformA.origin; @@ -324,7 +324,7 @@ void SliderJointSW::calculateTransforms(void) { //----------------------------------------------------------------------------- -void SliderJointSW::testLinLimits(void) { +void SliderJointSW::testLinLimits() { m_solveLinLim = false; m_linPos = m_depth[0]; if (m_lowerLinLimit <= m_upperLinLimit) { @@ -344,7 +344,7 @@ void SliderJointSW::testLinLimits(void) { //----------------------------------------------------------------------------- -void SliderJointSW::testAngLimits(void) { +void SliderJointSW::testAngLimits() { m_angDepth = real_t(0.); m_solveAngLim = false; if (m_lowerAngLimit <= m_upperAngLimit) { @@ -364,7 +364,7 @@ void SliderJointSW::testAngLimits(void) { //----------------------------------------------------------------------------- -Vector3 SliderJointSW::getAncorInA(void) { +Vector3 SliderJointSW::getAncorInA() { Vector3 ancorInA; ancorInA = m_realPivotAInW + (m_lowerLinLimit + m_upperLinLimit) * real_t(0.5) * m_sliderAxis; ancorInA = A->get_transform().inverse().xform(ancorInA); @@ -373,7 +373,7 @@ Vector3 SliderJointSW::getAncorInA(void) { //----------------------------------------------------------------------------- -Vector3 SliderJointSW::getAncorInB(void) { +Vector3 SliderJointSW::getAncorInB() { Vector3 ancorInB; ancorInB = m_frameInB.origin; return ancorInB; diff --git a/servers/physics/joints/slider_joint_sw.h b/servers/physics/joints/slider_joint_sw.h index aa658d88cd6..d139a7bef34 100644 --- a/servers/physics/joints/slider_joint_sw.h +++ b/servers/physics/joints/slider_joint_sw.h @@ -230,12 +230,12 @@ public: bool getSolveAngLimit() { return m_solveAngLim; } real_t getAngDepth() { return m_angDepth; } // shared code used by ODE solver - void calculateTransforms(void); - void testLinLimits(void); - void testAngLimits(void); + void calculateTransforms(); + void testLinLimits(); + void testAngLimits(); // access for PE Solver - Vector3 getAncorInA(void); - Vector3 getAncorInB(void); + Vector3 getAncorInA(); + Vector3 getAncorInB(); void set_param(PhysicsServer::SliderJointParam p_param, real_t p_value); real_t get_param(PhysicsServer::SliderJointParam p_param) const;