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
C++11 cleanup on constructors #6000
Conversation
a8cd47d
to
50cec19
Compare
src/script/cpp_api/s_base.cpp
Outdated
@@ -72,8 +72,7 @@ class ModNameStorer | |||
*/ | |||
|
|||
ScriptApiBase::ScriptApiBase() : | |||
m_luastackmutex(), | |||
m_gamedef(NULL) | |||
m_luastackmutex() |
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.
this constructor is useless you can remove it
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.
Please rename m_vm to vm like original case,
else LGTM
@Zeno- @SmallJoker can you take a look ? |
src/script/lua_api/l_areastore.h
Outdated
@@ -49,7 +49,7 @@ class LuaAreaStore : public ModApiBase | |||
static int l_from_file(lua_State *L); | |||
|
|||
public: | |||
AreaStore *as; | |||
AreaStore *m_as = nullptr; |
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.
m_
prefix is used for private
members. Revert to as
or areastore
.
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.
+1 once TravisCI reports success
src/threading/event.cpp
Outdated
@@ -32,7 +32,6 @@ void Event::wait() | |||
while (!notified) { | |||
cv.wait(lock); | |||
} | |||
notified = false; |
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 this disappears ?
0f7f305
to
ad080c3
Compare
* C++11 cleanup on constructors dir script
* C++11 cleanup on constructors dir script
No description provided.