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/array_property_edit.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/array_property_edit.cpp')
-rw-r--r-- | editor/array_property_edit.cpp | 21 |
1 files changed, 14 insertions, 7 deletions
diff --git a/editor/array_property_edit.cpp b/editor/array_property_edit.cpp index 3832a93360..20f947e707 100644 --- a/editor/array_property_edit.cpp +++ b/editor/array_property_edit.cpp @@ -37,8 +37,9 @@ Variant ArrayPropertyEdit::get_array() const { Object *o = ObjectDB::get_instance(obj); - if (!o) + if (!o) { return Array(); + } Variant arr = o->get(property); if (!arr.is_array()) { Callable::CallError ce; @@ -59,8 +60,9 @@ void ArrayPropertyEdit::_set_size(int p_size) { Variant arr = get_array(); arr.call("resize", p_size); Object *o = ObjectDB::get_instance(obj); - if (!o) + if (!o) { return; + } o->set(property, arr); } @@ -69,8 +71,9 @@ void ArrayPropertyEdit::_set_value(int p_idx, const Variant &p_value) { Variant arr = get_array(); arr.set(p_idx, p_value); Object *o = ObjectDB::get_instance(obj); - if (!o) + if (!o) { return; + } o->set(property, arr); } @@ -84,8 +87,9 @@ bool ArrayPropertyEdit::_set(const StringName &p_name, const Variant &p_value) { int size = arr.call("size"); int newsize = p_value; - if (newsize == size) + if (newsize == size) { return true; + } UndoRedo *ur = EditorNode::get_undo_redo(); ur->create_action(TTR("Resize Array")); @@ -184,8 +188,9 @@ bool ArrayPropertyEdit::_get(const StringName &p_name, Variant &r_ret) const { int idx = pn.get_slicec('/', 1).get_slicec('_', 0).to_int(); bool valid; r_ret = arr.get(idx, &valid); - if (valid) + if (valid) { r_ret = r_ret.get_type(); + } return valid; } else { @@ -210,8 +215,9 @@ void ArrayPropertyEdit::_get_property_list(List<PropertyInfo> *p_list) const { p_list->push_back(PropertyInfo(Variant::INT, "array/size", PROPERTY_HINT_RANGE, "0,100000,1")); int pages = size / ITEMS_PER_PAGE; - if (pages > 0) + if (pages > 0) { p_list->push_back(PropertyInfo(Variant::INT, "array/page", PROPERTY_HINT_RANGE, "0," + itos(pages) + ",1")); + } int offset = page * ITEMS_PER_PAGE; @@ -288,8 +294,9 @@ void ArrayPropertyEdit::_bind_methods() { ArrayPropertyEdit::ArrayPropertyEdit() { page = 0; for (int i = 0; i < Variant::VARIANT_MAX; i++) { - if (i > 0) + if (i > 0) { vtypes += ","; + } vtypes += Variant::get_type_name(Variant::Type(i)); } default_type = Variant::NIL; |