[PVS-Studio] Use bool for Gui{Radio,Toggle}Button::m_pressed

V601 The 'false' value is implicitly casted to the integer type.
guiradiobutton.cpp 9
V601 The 'true' value is implicitly casted to the integer type.
guiradiobutton.cpp 30
V601 The 'false' value is implicitly casted to the integer type.
guitogglebutton.cpp 9
[PVS-Studio] Merge identical if-then branches in TerrainHeightRuggedLava

V523 The 'then' statement is equivalent to the 'else' statement.
terrainheightruggedlava.cpp 64
V523 The 'then' statement is equivalent to the 'else' statement.
terrainheightruggedlava.cpp 68
[PVS-Studio] Sound::PlaySfx and Sound::Event::Play take an Op, not a bool

V601 The 'true' value is implicitly casted to the integer type.
Inspect the fourth argument. ambientsounds.cpp 61

V601 The 'false' value is implicitly casted to the integer type.
Inspect the fourth argument. sound.cpp 158
[PVS-Studio] Don't try to use Pi::player before checking for NULL

V595 The 'Pi::player' pointer was utilized before it was verified
against nullptr. Check lines: 377, 384. worldview.cpp 377
[PVS-Studio] Avoid undefined behaviour (shifting ints of unknown size)

V610 Undefined behavior. Check the shift operator '<<. The right
operand ('i' = [0..63]) is greater than or equal to the length in
bits of the promoted left operand. polit.cpp 112
[PVS-Studio] Use format specifier %u for unsigned values.

V576 Incorrect format. Consider checking the seventh actual
argument of the '_snprintf' function. The SIGNED integer type
argument is expected. luaserializer.cpp 191

V576 Incorrect format. Consider checking the eighth actual
argument of the '_snprintf' function. The SIGNED integer type
argument is expected. luaserializer.cpp 191

V576 Incorrect format. Consider checking the fourth actual
argument of the '_snprintf' function. The SIGNED integer type
argument is expected. luaserializer.cpp 199

V576 Incorrect format. Consider checking the second actual
argument of the 'printf' function. The SIGNED integer type
argument is expected. luamodelviewer.cpp 497
[PVS-Studio] Don't check a SystemBody::Type against a SystemBody::SuperType

V556 The values of different enum types are compared:
switch(ENUM_TYPE_A) { case ENUM_TYPE_B: ... }.
starsystem.cpp 1094
[PVS-Studio] If branches with multiple statements need to be in braces

V612 An unconditional 'break' within a loop. starsystem.cpp 1882
Initial commit.
Track GL_LIGHT_MODEL_AMBIENT.
Update planet rings draw code to use new GetLights graphics interface. Move lighting to it's own function.
Fade-in ambient light as camera gets closer to buildings during dark part of day.
Move uniform calculation out to it's own function.
Add atmosphere params to the struct LmrObjParams as well as a constructor.
Render surface starport/city with new combined model/atmosphere shader. Is activated through options EnableModelsInAtmosphereShader.
Enable activation through options EnableModelsInAtmosphereShader.
Remove atmospheric model shader but retain lighting tweaks/documentation.
Send down const camera into render functions and fix misc stuff.
Merge remote-tracking branch 'ae/Fade-building-illumination' into x

Conflicts:
	src/Planet.cpp
	src/Planet.h
split Light into a basic Light for the renderer, and a version with a Body for use with Camera