summaryrefslogtreecommitdiff
path: root/core/list.h
diff options
context:
space:
mode:
authorRémi Verschelde <rverschelde@gmail.com>2020-05-12 17:01:17 +0200
committerRémi Verschelde <rverschelde@gmail.com>2020-05-14 10:01:56 +0200
commit1f6f364a56319eabd02c050746fe7df3f55ffee3 (patch)
tree8bebdce946466ce8e9476ccd46c9dba62c323938 /core/list.h
parente7c9d818766a119089873e4941e4865fb36883ec (diff)
Port member initialization from constructor to declaration (C++11)
Using `clang-tidy`'s `modernize-use-default-member-init` check and manual review of the changes, and some extra manual changes that `clang-tidy` failed to do. Also went manually through all of `core` to find occurrences that `clang-tidy` couldn't handle, especially all initializations done in a constructor without using initializer lists.
Diffstat (limited to 'core/list.h')
-rw-r--r--core/list.h20
1 files changed, 7 insertions, 13 deletions
diff --git a/core/list.h b/core/list.h
index 65d45ffdfc..eb74fa7917 100644
--- a/core/list.h
+++ b/core/list.h
@@ -54,9 +54,9 @@ public:
friend class List<T, A>;
T value;
- Element *next_ptr;
- Element *prev_ptr;
- _Data *data;
+ Element *next_ptr = nullptr;
+ Element *prev_ptr = nullptr;
+ _Data *data = nullptr;
public:
/**
@@ -139,11 +139,7 @@ public:
data->erase(this);
}
- _FORCE_INLINE_ Element() {
- next_ptr = 0;
- prev_ptr = 0;
- data = nullptr;
- };
+ _FORCE_INLINE_ Element() {}
};
private:
@@ -178,7 +174,7 @@ private:
}
};
- _Data *_data;
+ _Data *_data = nullptr;
public:
/**
@@ -687,7 +683,6 @@ public:
*/
List(const List &p_list) {
- _data = nullptr;
const Element *it = p_list.front();
while (it) {
@@ -696,9 +691,8 @@ public:
}
}
- List() {
- _data = nullptr;
- };
+ List() {}
+
~List() {
clear();
if (_data) {