godotengine
/
godot
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

More codespell fixes, do more changes from previous ignore list

This commit is contained in:
Rémi Verschelde 2023-02-01 10:49:05 +01:00
parent c40020513a
commit e52213e2fa
No known key found for this signature in database
GPG Key ID: C3336907360768E1
16 changed files with 64 additions and 64 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
core/config/project_settings.cpp
View File

@ -301,13 +301,13 @@ bool ProjectSettings::_set(const StringName &p_name, const Variant &p_value) {
for (int i = 1; i < s.size(); i++) { for (int i = 1; i < s.size(); i++) {
String feature = s[i].strip_edges(); String feature = s[i].strip_edges();
Pair<StringName, StringName> fo(feature, p_name); Pair<StringName, StringName> feature_override(feature, p_name);
if (!feature_overrides.has(s[0])) { if (!feature_overrides.has(s[0])) {
feature_overrides[s[0]] = LocalVector<Pair<StringName, StringName>>(); feature_overrides[s[0]] = LocalVector<Pair<StringName, StringName>>();
} }
feature_overrides[s[0]].push_back(fo); feature_overrides[s[0]].push_back(feature_override);
} }
} }
} }

10
core/math/quick_hull.cpp
View File

@ -383,15 +383,15 @@ Error QuickHull::build(const Vector<Vector3> &p_points, Geometry3D::MeshData &r_
if (O->get().plane.is_equal_approx(f.plane)) { if (O->get().plane.is_equal_approx(f.plane)) {
//merge and delete edge and contiguous face, while repointing edges (uuugh!) //merge and delete edge and contiguous face, while repointing edges (uuugh!)
int ois = O->get().indices.size(); int o_index_size = O->get().indices.size();
for (int j = 0; j < ois; j++) { for (int j = 0; j < o_index_size; j++) {
//search a //search a
if (O->get().indices[j] == a) { if (O->get().indices[j] == a) {
//append the rest //append the rest
for (int k = 0; k < ois; k++) { for (int k = 0; k < o_index_size; k++) {
int idx = O->get().indices[(k + j) % ois]; int idx = O->get().indices[(k + j) % o_index_size];
int idxn = O->get().indices[(k + j + 1) % ois]; int idxn = O->get().indices[(k + j + 1) % o_index_size];
if (idx == b && idxn == a) { //already have b! if (idx == b && idxn == a) { //already have b!
break; break;
} }

8
core/math/transform_2d.cpp
View File

@ -151,7 +151,7 @@ void Transform2D::orthonormalize() {
Vector2 y = columns[1]; Vector2 y = columns[1];
x.normalize(); x.normalize();
y = (y - x * (x.dot(y))); y = y - x * x.dot(y);
y.normalize(); y.normalize();
columns[0] = x; columns[0] = x;
@ -159,9 +159,9 @@ void Transform2D::orthonormalize() {
} }
Transform2D Transform2D::orthonormalized() const { Transform2D Transform2D::orthonormalized() const {
Transform2D on = *this; Transform2D ortho = *this;
on.orthonormalize(); ortho.orthonormalize();
return on; return ortho;
} }
bool Transform2D::is_equal_approx(const Transform2D &p_transform) const { bool Transform2D::is_equal_approx(const Transform2D &p_transform) const {

4
core/object/worker_thread_pool.h
View File

@ -173,8 +173,8 @@ public:
template <class C, class M, class U> template <class C, class M, class U>
GroupID add_template_group_task(C *p_instance, M p_method, U p_userdata, int p_elements, int p_tasks = -1, bool p_high_priority = false, const String &p_description = String()) { GroupID add_template_group_task(C *p_instance, M p_method, U p_userdata, int p_elements, int p_tasks = -1, bool p_high_priority = false, const String &p_description = String()) {
typedef GroupUserData<C, M, U> GUD; typedef GroupUserData<C, M, U> GroupUD;
GUD *ud = memnew(GUD); GroupUD *ud = memnew(GroupUD);
ud->instance = p_instance; ud->instance = p_instance;
ud->method = p_method; ud->method = p_method;
ud->userdata = p_userdata; ud->userdata = p_userdata;

4
doc/classes/OS.xml
View File

@ -553,11 +553,11 @@
[b]Note:[/b] If the user has disabled the recycle bin on their system, the file will be permanently deleted instead. [b]Note:[/b] If the user has disabled the recycle bin on their system, the file will be permanently deleted instead.
[codeblocks] [codeblocks]
[gdscript] [gdscript]
var file_to_remove = "user://slot1.sav" var file_to_remove = "user://slot1.save"
OS.move_to_trash(ProjectSettings.globalize_path(file_to_remove)) OS.move_to_trash(ProjectSettings.globalize_path(file_to_remove))
[/gdscript] [/gdscript]
[csharp] [csharp]
var fileToRemove = "user://slot1.sav"; var fileToRemove = "user://slot1.save";
OS.MoveToTrash(ProjectSettings.GlobalizePath(fileToRemove)); OS.MoveToTrash(ProjectSettings.GlobalizePath(fileToRemove));
[/csharp] [/csharp]
[/codeblocks] [/codeblocks]

24
editor/filesystem_dock.cpp
View File

@ -241,8 +241,8 @@ void FileSystemDock::_update_tree(const Vector<String> &p_uncollapsed_paths, boo
} }
for (int i = 0; i < favorite_paths.size(); i++) { for (int i = 0; i < favorite_paths.size(); i++) {
String fave = favorite_paths[i]; String favorite = favorite_paths[i];
if (!fave.begins_with("res://")) { if (!favorite.begins_with("res://")) {
continue; continue;
} }
@ -252,18 +252,18 @@ void FileSystemDock::_update_tree(const Vector<String> &p_uncollapsed_paths, boo
String text; String text;
Ref<Texture2D> icon; Ref<Texture2D> icon;
Color color; Color color;
if (fave == "res://") { if (favorite == "res://") {
text = "/"; text = "/";
icon = folder_icon; icon = folder_icon;
color = folder_color; color = folder_color;
} else if (fave.ends_with("/")) { } else if (favorite.ends_with("/")) {
text = fave.substr(0, fave.length() - 1).get_file(); text = favorite.substr(0, favorite.length() - 1).get_file();
icon = folder_icon; icon = folder_icon;
color = folder_color; color = folder_color;
} else { } else {
text = fave.get_file(); text = favorite.get_file();
int index; int index;
EditorFileSystemDirectory *dir = EditorFileSystem::get_singleton()->find_file(fave, &index); EditorFileSystemDirectory *dir = EditorFileSystem::get_singleton()->find_file(favorite, &index);
if (dir) { if (dir) {
icon = _get_tree_item_icon(dir->get_file_import_is_valid(index), dir->get_file_type(index)); icon = _get_tree_item_icon(dir->get_file_import_is_valid(index), dir->get_file_type(index));
} else { } else {
@ -277,18 +277,18 @@ void FileSystemDock::_update_tree(const Vector<String> &p_uncollapsed_paths, boo
ti->set_text(0, text); ti->set_text(0, text);
ti->set_icon(0, icon); ti->set_icon(0, icon);
ti->set_icon_modulate(0, color); ti->set_icon_modulate(0, color);
ti->set_tooltip_text(0, fave); ti->set_tooltip_text(0, favorite);
ti->set_selectable(0, true); ti->set_selectable(0, true);
ti->set_metadata(0, fave); ti->set_metadata(0, favorite);
if (p_select_in_favorites && fave == path) { if (p_select_in_favorites && favorite == path) {
ti->select(0); ti->select(0);
ti->set_as_cursor(0); ti->set_as_cursor(0);
} }
if (!fave.ends_with("/")) { if (!favorite.ends_with("/")) {
Array udata; Array udata;
udata.push_back(tree_update_id); udata.push_back(tree_update_id);
udata.push_back(ti); udata.push_back(ti);
EditorResourcePreview::get_singleton()->queue_resource_preview(fave, this, "_tree_thumbnail_done", udata); EditorResourcePreview::get_singleton()->queue_resource_preview(favorite, this, "_tree_thumbnail_done", udata);
} }
} }
} }

6
misc/scripts/codespell.sh
View File

@ -1,8 +1,8 @@
#!/bin/sh #!/bin/sh
SKIP_LIST="./.*,./bin,./thirdparty,*.desktop,*.gen.*,*.po,*.pot,*.rc,./AUTHORS.md,./COPYRIGHT.txt,./DONORS.md," SKIP_LIST="./.*,./**/.*,./bin,./thirdparty,*.desktop,*.gen.*,*.po,*.pot,*.rc,./AUTHORS.md,./COPYRIGHT.txt,./DONORS.md,"
SKIP_LIST+="./core/string/locales.h,./editor/project_converter_3_to_4.cpp,./misc/scripts/codespell.sh," SKIP_LIST+="./core/input/gamecontrollerdb.txt,./core/string/locales.h,./editor/project_converter_3_to_4.cpp,./misc/scripts/codespell.sh,"
SKIP_LIST+="./platform/android/java/lib/src/com,./platform/web/node_modules,./platform/web/package-lock.json," SKIP_LIST+="./platform/android/java/lib/src/com,./platform/web/node_modules,./platform/web/package-lock.json,"
IGNORE_LIST="alo,ba,complies,curvelinear,doubleclick,expct,fave,findn,gird,gud,inout,lod,nd,numer,ois,readded,ro,sav,statics,te,varius,varn,wan" IGNORE_LIST="curvelinear,doubleclick,expct,findn,gird,hel,inout,lod,nd,numer,ot,te"
codespell -w -q 3 -S "${SKIP_LIST}" -L "${IGNORE_LIST}" --builtin "clear,rare,en-GB_to_en-US" codespell -w -q 3 -S "${SKIP_LIST}" -L "${IGNORE_LIST}" --builtin "clear,rare,en-GB_to_en-US"

18
modules/mono/glue/GodotSharp/GodotSharp/Core/Transform2D.cs
View File

@ -247,19 +247,19 @@ namespace Godot
/// <returns>The orthonormalized transform.</returns> /// <returns>The orthonormalized transform.</returns>
public readonly Transform2D Orthonormalized() public readonly Transform2D Orthonormalized()
{ {
Transform2D on = this; Transform2D ortho = this;
Vector2 onX = on.X; Vector2 orthoX = ortho.X;
Vector2 onY = on.Y; Vector2 orthoY = ortho.Y;
onX.Normalize(); orthoX.Normalize();
onY = onY - (onX * onX.Dot(onY)); orthoY = orthoY - orthoX * orthoX.Dot(orthoY);
onY.Normalize(); orthoY.Normalize();
on.X = onX; ortho.X = orthoX;
on.Y = onY; ortho.Y = orthoY;
return on; return ortho;
} }
/// <summary> /// <summary>

2
platform/android/java/lib/src/org/godotengine/godot/FullScreenGodotApp.java
View File

@ -99,7 +99,7 @@ public abstract class FullScreenGodotApp extends FragmentActivity implements God
// from scratch. Therefore, we need to kill the whole app process and relaunch it. // from scratch. Therefore, we need to kill the whole app process and relaunch it.
// //
// Restarting only the activity, wouldn't be enough unless it did proper cleanup (including // Restarting only the activity, wouldn't be enough unless it did proper cleanup (including
// releasing and reloading native libs or resetting their state somehow and clearing statics). // releasing and reloading native libs or resetting their state somehow and clearing static data).
Log.v(TAG, "Restarting Godot instance..."); Log.v(TAG, "Restarting Godot instance...");
ProcessPhoenix.triggerRebirth(FullScreenGodotApp.this); ProcessPhoenix.triggerRebirth(FullScreenGodotApp.this);
} }

2
scene/2d/navigation_agent_2d.cpp
View File

@ -140,7 +140,7 @@ void NavigationAgent2D::_notification(int p_what) {
switch (p_what) { switch (p_what) {
case NOTIFICATION_POST_ENTER_TREE: { case NOTIFICATION_POST_ENTER_TREE: {
// need to use POST_ENTER_TREE cause with normal ENTER_TREE not all required Nodes are ready. // need to use POST_ENTER_TREE cause with normal ENTER_TREE not all required Nodes are ready.
// cannot use READY as ready does not get called if Node is readded to SceneTree // cannot use READY as ready does not get called if Node is re-added to SceneTree
set_agent_parent(get_parent()); set_agent_parent(get_parent());
set_physics_process_internal(true); set_physics_process_internal(true);

2
scene/3d/navigation_agent_3d.cpp
View File

@ -143,7 +143,7 @@ void NavigationAgent3D::_notification(int p_what) {
switch (p_what) { switch (p_what) {
case NOTIFICATION_POST_ENTER_TREE: { case NOTIFICATION_POST_ENTER_TREE: {
// need to use POST_ENTER_TREE cause with normal ENTER_TREE not all required Nodes are ready. // need to use POST_ENTER_TREE cause with normal ENTER_TREE not all required Nodes are ready.
// cannot use READY as ready does not get called if Node is readded to SceneTree // cannot use READY as ready does not get called if Node is re-added to SceneTree
set_agent_parent(get_parent()); set_agent_parent(get_parent());
set_physics_process_internal(true); set_physics_process_internal(true);

6
scene/main/node.cpp
View File

@ -2135,13 +2135,13 @@ Node *Node::_duplicate(int p_flags, HashMap<const Node *, Node *> *r_duplimap) c
} else if ((p_flags & DUPLICATE_USE_INSTANTIATION) && !get_scene_file_path().is_empty()) { } else if ((p_flags & DUPLICATE_USE_INSTANTIATION) && !get_scene_file_path().is_empty()) {
Ref<PackedScene> res = ResourceLoader::load(get_scene_file_path()); Ref<PackedScene> res = ResourceLoader::load(get_scene_file_path());
ERR_FAIL_COND_V(res.is_null(), nullptr); ERR_FAIL_COND_V(res.is_null(), nullptr);
PackedScene::GenEditState ges = PackedScene::GEN_EDIT_STATE_DISABLED; PackedScene::GenEditState edit_state = PackedScene::GEN_EDIT_STATE_DISABLED;
#ifdef TOOLS_ENABLED #ifdef TOOLS_ENABLED
if (p_flags & DUPLICATE_FROM_EDITOR) { if (p_flags & DUPLICATE_FROM_EDITOR) {
ges = PackedScene::GEN_EDIT_STATE_INSTANCE; edit_state = PackedScene::GEN_EDIT_STATE_INSTANCE;
} }
#endif #endif
node = res->instantiate(ges); node = res->instantiate(edit_state);
ERR_FAIL_COND_V(!node, nullptr); ERR_FAIL_COND_V(!node, nullptr);
node->set_scene_instance_load_placeholder(get_scene_instance_load_placeholder()); node->set_scene_instance_load_placeholder(get_scene_instance_load_placeholder());

6
scene/resources/resource_format_text.cpp
View File

@ -2379,15 +2379,15 @@ Error ResourceFormatSaverText::set_uid(const String &p_path, ResourceUID::ID p_u
String local_path = ProjectSettings::get_singleton()->localize_path(p_path); String local_path = ProjectSettings::get_singleton()->localize_path(p_path);
Error err = OK; Error err = OK;
{ {
Ref<FileAccess> fo = FileAccess::open(p_path, FileAccess::READ); Ref<FileAccess> file = FileAccess::open(p_path, FileAccess::READ);
if (fo.is_null()) { if (file.is_null()) {
ERR_FAIL_V(ERR_CANT_OPEN); ERR_FAIL_V(ERR_CANT_OPEN);
} }
ResourceLoaderText loader; ResourceLoaderText loader;
loader.local_path = local_path; loader.local_path = local_path;
loader.res_path = loader.local_path; loader.res_path = loader.local_path;
err = loader.set_uid(fo, p_uid); err = loader.set_uid(file, p_uid);
} }
if (err == OK) { if (err == OK) {

10
servers/rendering/renderer_rd/storage_rd/material_storage.h
View File

@ -323,13 +323,13 @@ public:
// http://andrewthall.org/papers/df64_qf128.pdf // http://andrewthall.org/papers/df64_qf128.pdf
#ifdef REAL_T_IS_DOUBLE #ifdef REAL_T_IS_DOUBLE
static _FORCE_INLINE_ void split_double(double a, float *ahi, float *alo) { static _FORCE_INLINE_ void split_double(double a, float *a_hi, float *a_lo) {
const double SPLITTER = (1 << 29) + 1; const double SPLITTER = (1 << 29) + 1;
double t = a * SPLITTER; double t = a * SPLITTER;
double thi = t - (t - a); double t_hi = t - (t - a);
double tlo = a - thi; double t_lo = a - t_hi;
*ahi = (float)thi; *a_hi = (float)t_hi;
*alo = (float)tlo; *a_lo = (float)t_lo;
} }
#endif #endif

8
servers/rendering/shader_language.cpp
View File

@ -4530,14 +4530,14 @@ bool ShaderLanguage::_check_node_constness(const Node *p_node) const {
case Node::TYPE_CONSTANT: case Node::TYPE_CONSTANT:
break; break;
case Node::TYPE_VARIABLE: { case Node::TYPE_VARIABLE: {
const VariableNode *varn = static_cast<const VariableNode *>(p_node); const VariableNode *var_node = static_cast<const VariableNode *>(p_node);
if (!varn->is_const) { if (!var_node->is_const) {
return false; return false;
} }
} break; } break;
case Node::TYPE_ARRAY: { case Node::TYPE_ARRAY: {
const ArrayNode *arrn = static_cast<const ArrayNode *>(p_node); const ArrayNode *arr_node = static_cast<const ArrayNode *>(p_node);
if (!arrn->is_const) { if (!arr_node->is_const) {
return false; return false;
} }
} break; } break;

14
tests/scene/test_text_edit.h
View File

@ -3086,7 +3086,7 @@ TEST_CASE("[SceneTree][TextEdit] context menu") {
text_edit->get_viewport()->set_embedding_subwindows(true); // Bypass display server for drop handling. text_edit->get_viewport()->set_embedding_subwindows(true); // Bypass display server for drop handling.
text_edit->set_size(Size2(800, 200)); text_edit->set_size(Size2(800, 200));
text_edit->set_line(0, "Lorem ipsum dolor sit amet, consectetur adipiscing elit. Donec varius mattis leo, sed porta ex lacinia bibendum. Nunc bibendum pellentesque."); text_edit->set_line(0, "Lorem ipsum dolor sit amet, consectetur adipiscing elit. Donec vasius mattis leo, sed porta ex lacinia bibendum. Nunc bibendum pellentesque.");
MessageQueue::get_singleton()->flush(); MessageQueue::get_singleton()->flush();
text_edit->set_context_menu_enabled(false); text_edit->set_context_menu_enabled(false);
@ -3227,7 +3227,7 @@ TEST_CASE("[SceneTree][TextEdit] mouse") {
SceneTree::get_singleton()->get_root()->add_child(text_edit); SceneTree::get_singleton()->get_root()->add_child(text_edit);
text_edit->set_size(Size2(800, 200)); text_edit->set_size(Size2(800, 200));
text_edit->set_line(0, "Lorem ipsum dolor sit amet, consectetur adipiscing elit. Donec varius mattis leo, sed porta ex lacinia bibendum. Nunc bibendum pellentesque."); text_edit->set_line(0, "Lorem ipsum dolor sit amet, consectetur adipiscing elit. Donec vasius mattis leo, sed porta ex lacinia bibendum. Nunc bibendum pellentesque.");
MessageQueue::get_singleton()->flush(); MessageQueue::get_singleton()->flush();
CHECK(text_edit->get_word_at_pos(text_edit->get_pos_at_line_column(0, 1)) == "Lorem"); CHECK(text_edit->get_word_at_pos(text_edit->get_pos_at_line_column(0, 1)) == "Lorem");
@ -3305,9 +3305,9 @@ TEST_CASE("[SceneTree][TextEdit] caret") {
SEND_GUI_ACTION(text_edit, "ui_text_caret_left"); SEND_GUI_ACTION(text_edit, "ui_text_caret_left");
CHECK(text_edit->get_caret_column() == 0); CHECK(text_edit->get_caret_column() == 0);
text_edit->set_line(0, "Lorem ipsum dolor sit amet, consectetur adipiscing elit. Donec varius mattis leo, sed porta ex lacinia bibendum. Nunc bibendum pellentesque."); text_edit->set_line(0, "Lorem ipsum dolor sit amet, consectetur adipiscing elit. Donec vasius mattis leo, sed porta ex lacinia bibendum. Nunc bibendum pellentesque.");
for (int i = 0; i < 3; i++) { for (int i = 0; i < 3; i++) {
text_edit->insert_line_at(0, "Lorem ipsum dolor sit amet, consectetur adipiscing elit. Donec varius mattis leo, sed porta ex lacinia bibendum. Nunc bibendum pellentesque."); text_edit->insert_line_at(0, "Lorem ipsum dolor sit amet, consectetur adipiscing elit. Donec vasius mattis leo, sed porta ex lacinia bibendum. Nunc bibendum pellentesque.");
} }
MessageQueue::get_singleton()->flush(); MessageQueue::get_singleton()->flush();
@ -3519,7 +3519,7 @@ TEST_CASE("[SceneTree][TextEdit] line wrapping") {
// Set size for boundary. // Set size for boundary.
text_edit->set_size(Size2(800, 200)); text_edit->set_size(Size2(800, 200));
text_edit->set_line(0, "Lorem ipsum dolor sit amet, consectetur adipiscing elit. Donec varius mattis leo, sed porta ex lacinia bibendum. Nunc bibendum pellentesque."); text_edit->set_line(0, "Lorem ipsum dolor sit amet, consectetur adipiscing elit. Donec vasius mattis leo, sed porta ex lacinia bibendum. Nunc bibendum pellentesque.");
CHECK_FALSE(text_edit->is_line_wrapped(0)); CHECK_FALSE(text_edit->is_line_wrapped(0));
CHECK(text_edit->get_line_wrap_count(0) == 0); CHECK(text_edit->get_line_wrap_count(0) == 0);
CHECK(text_edit->get_line_wrap_index_at_column(0, 130) == 0); CHECK(text_edit->get_line_wrap_index_at_column(0, 130) == 0);
@ -3569,7 +3569,7 @@ TEST_CASE("[SceneTree][TextEdit] viewport") {
// No subcases here for performance. // No subcases here for performance.
text_edit->set_size(Size2(800, 600)); text_edit->set_size(Size2(800, 600));
for (int i = 0; i < 50; i++) { for (int i = 0; i < 50; i++) {
text_edit->insert_line_at(0, "Lorem ipsum dolor sit amet, consectetur adipiscing elit. Donec varius mattis leo, sed porta ex lacinia bibendum. Nunc bibendum pellentesque."); text_edit->insert_line_at(0, "Lorem ipsum dolor sit amet, consectetur adipiscing elit. Donec vasius mattis leo, sed porta ex lacinia bibendum. Nunc bibendum pellentesque.");
} }
MessageQueue::get_singleton()->flush(); MessageQueue::get_singleton()->flush();
@ -3897,7 +3897,7 @@ TEST_CASE("[SceneTree][TextEdit] viewport") {
CHECK(text_edit->get_h_scroll() == 0); CHECK(text_edit->get_h_scroll() == 0);
text_edit->set_h_scroll(10000000); text_edit->set_h_scroll(10000000);
CHECK(text_edit->get_h_scroll() == 313); CHECK(text_edit->get_h_scroll() == 314);
text_edit->set_h_scroll(-100); text_edit->set_h_scroll(-100);
CHECK(text_edit->get_h_scroll() == 0); CHECK(text_edit->get_h_scroll() == 0);