-
Notifications
You must be signed in to change notification settings - Fork 2k
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 2 #5999
Cpp11 initializers 2 #5999
Conversation
* Drop obsolete bool MainMenuData::enable_public (setting is called with cURL in server loop)
f054cec
to
01118f8
Compare
src/guiChatConsole.cpp
Outdated
} | ||
else | ||
{ | ||
else { |
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.
if (!m_font) {
} else {
}
as soon one part isn't an one-liner.
src/httpfetch.h
Outdated
|
||
HTTPFetchResult(const HTTPFetchRequest &fetch_request) | ||
: succeeded(false), timeout(false), response_code(0), data(""), | ||
caller(fetch_request.caller), request_id(fetch_request.request_id) | ||
: caller(fetch_request.caller), request_id(fetch_request.request_id) |
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.
LINT doesn't like this line
f32 m_pitch = 0.0f; | ||
bool camera_barely_in_ceiling = false; | ||
aabb3f m_collisionbox = aabb3f(-BS * 0.30f, 0.0f, -BS * 0.30f, BS * 0.30f, | ||
BS * 1.75f, BS * 0.30f); |
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.
Could you please add the new line an argument earlier, so the X,Y,Z values are grouped nicely
Apart from the comments above, LGTM. |
ty for review i will fix them soon |
01118f8
to
fc79504
Compare
@SmallJoker all points fixed |
* C++11 patchset 10: continue cleanup on constructors * Drop obsolete bool MainMenuData::enable_public (setting is called with cURL in server loop) * More classes cleanup * More classes cleanup + change NULL tests to boolean tests
* C++11 patchset 10: continue cleanup on constructors * Drop obsolete bool MainMenuData::enable_public (setting is called with cURL in server loop) * More classes cleanup * More classes cleanup + change NULL tests to boolean tests
continue previous work: cleaning up constructors. Also fix some NULL checks on nullptr replaced values to be coherent & portable between NULL & nullptr