New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Cpp11 initializers: last src root changeset #6022
Conversation
Finish to migrate all src root folder files to C++11 constructor initializers
/* | ||
The block from which the object was loaded from, and in which | ||
a copy of the static data resides. | ||
*/ | ||
v3s16 m_static_block; | ||
v3s16 m_static_block = v3s16(1337,1337,1337); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What is the significance of 1337? Just a magic number?
src/subgame.h
Outdated
@@ -38,14 +36,14 @@ struct SubgameSpec | |||
std::string menuicon_path; | |||
|
|||
SubgameSpec(const std::string &id_="", | |||
const std::string &path_="", | |||
const std::string &path_="", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we don't pad with spaces around '=' anymore?
bool m_move_has_really_moved = false; | ||
s64 m_move_downtime = 0; | ||
bool m_move_sent_as_mouse_event = false; | ||
v2s32 m_move_downlocation = v2s32(-10000, -10000); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why -10000, -10000?
src/voxel.h
Outdated
v3s16 MaxEdge; | ||
}; | ||
|
||
// unused | ||
// unused | ||
#define VOXELFLAG_UNUSED (1<<0) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
padding around << operator?
src/voxel.h
Outdated
v3s16 MaxEdge; | ||
}; | ||
|
||
// unused | ||
// unused | ||
#define VOXELFLAG_UNUSED (1<<0) | ||
// no data about that node | ||
#define VOXELFLAG_NO_DATA (1<<1) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
padding around << operator?
👍 |
* Cpp11 initializers: last src root changeset Finish to migrate all src root folder files to C++11 constructor initializers
* Cpp11 initializers: last src root changeset Finish to migrate all src root folder files to C++11 constructor initializers
* Cpp11 initializers: last src root changeset Finish to migrate all src root folder files to C++11 constructor initializers
Finish to migrate all src root folder files to C++11 constructor initializers
also remove some unused parameters