diff options
author | Rémi Verschelde <rverschelde@gmail.com> | 2020-05-14 16:41:43 +0200 |
---|---|---|
committer | Rémi Verschelde <rverschelde@gmail.com> | 2020-05-14 21:57:34 +0200 |
commit | 0ee0fa42e6639b6fa474b7cf6afc6b1a78142185 (patch) | |
tree | 198d4ff7665d89307f6ca2469fa38620a9eb1672 /editor/debugger/editor_debugger_tree.cpp | |
parent | 07bc4e2f96f8f47991339654ff4ab16acc19d44f (diff) | |
download | redot-engine-0ee0fa42e6639b6fa474b7cf6afc6b1a78142185.tar.gz |
Style: Enforce braces around if blocks and loops
Using clang-tidy's `readability-braces-around-statements`.
https://clang.llvm.org/extra/clang-tidy/checks/readability-braces-around-statements.html
Diffstat (limited to 'editor/debugger/editor_debugger_tree.cpp')
-rw-r--r-- | editor/debugger/editor_debugger_tree.cpp | 18 |
1 files changed, 12 insertions, 6 deletions
diff --git a/editor/debugger/editor_debugger_tree.cpp b/editor/debugger/editor_debugger_tree.cpp index ecffa0f05d..0b5f865a98 100644 --- a/editor/debugger/editor_debugger_tree.cpp +++ b/editor/debugger/editor_debugger_tree.cpp @@ -84,8 +84,9 @@ void EditorDebuggerTree::_scene_tree_folded(Object *p_obj) { } TreeItem *item = Object::cast_to<TreeItem>(p_obj); - if (!item) + if (!item) { return; + } ObjectID id = ObjectID(uint64_t(item->get_metadata(0))); if (unfold_cache.has(id)) { @@ -97,8 +98,9 @@ void EditorDebuggerTree::_scene_tree_folded(Object *p_obj) { void EditorDebuggerTree::_scene_tree_rmb_selected(const Vector2 &p_position) { TreeItem *item = get_item_at_position(p_position); - if (!item) + if (!item) { return; + } item->select(0); @@ -176,12 +178,14 @@ void EditorDebuggerTree::update_scene_tree(const SceneDebuggerTree *p_tree, int // Apply filters. while (parent) { const bool had_siblings = item->get_prev() || item->get_next(); - if (filter.is_subsequence_ofi(item->get_text(0))) + if (filter.is_subsequence_ofi(item->get_text(0))) { break; // Filter matches, must survive. + } parent->remove_child(item); memdelete(item); - if (had_siblings) + if (had_siblings) { break; // Parent must survive. + } item = parent; parent = item->get_parent(); // Check if parent expects more children. @@ -199,8 +203,9 @@ void EditorDebuggerTree::update_scene_tree(const SceneDebuggerTree *p_tree, int } String EditorDebuggerTree::get_selected_path() { - if (!get_selected()) + if (!get_selected()) { return ""; + } return _get_path(get_selected()); } @@ -256,7 +261,8 @@ void EditorDebuggerTree::_item_menu_id_pressed(int p_option) { } void EditorDebuggerTree::_file_selected(const String &p_file) { - if (inspected_object_id.is_null()) + if (inspected_object_id.is_null()) { return; + } emit_signal("save_node", inspected_object_id, p_file, debugger_id); } |