moved (and renamed) global variables to game structure (ExitX/ExitY)

moved (and renamed) global variable to game structure (AllPlayersGone)

improved internal handling of "game over" state

moved game values for score and health from player to game structure

moved flag for "game over" from player to game structure

This also removes asymmetric behaviour depending on players being the
"local player" or any other player in team mode (which happened if one
player dies and the other enters the exit, for example -- in one case,
the game will be treated as "game over", in the other case the game
will continue running without asking to play again).

moved game values for things still needed from player to game structure

renamed functions (after removing parameter in previous commit)

removed parameter from functions (which always had the same value)

renamed function (after removing parameter in previous commit)

removed parameter from function (which always had the same value)

moved flags and values for "level solved" from player to game structure

added flags for "level solved" and "game over" to EM engine

changed variable name

added custom artwork mappings to fix warnings in level set 'rnd_alan_bond'

small code cleanup

added compatibility check if Sokoban objects need to be solved

added editor settings if Sokoban fields and objects need to be solved

fixed unexpected behaviour with solving levels with Sokoban elements

This patch fixes problems with levels using a different number of
Sokoban objects and Sokoban fields, causing unexpected behaviour.

Before, levels with Sokoban objects, but without Sokoban fields, could
be solved by just pushing any of the Sokoban objects (which triggers a
check if there are any empty Sokoban fields left in the level, which
is not the case in this scenario). In addition, pushing a Sokoban
object to a Sokoban field would also solve a level with two Sokoban
objects, but only one Sokoban field, but not a level with two Sokoban
fields, but only one Sokoban object, which seems confusing or at least
unexpected. (Things get even more complicated and confusing if Sokoban
fields or objects are created during the game, for example as content
of yam-yams.)

Now, Sokoban fields and objects in a level are tracked independently
and the level is only solved if there are no Sokoban fields or objects
left on the playfield.

Also see the following forum post for a description of the problem:
https://www.artsoft.org/forum/viewtopic.php?f=7&t=2654

fixed bug with Sokoban auto-exit option only working for local player

added special global animation mode 'NO_TITLE' for all but title screens

changed comments from old to new style (auto-generated files)

changed comments from old to new style (multiple-line comments)

changed comments from old to new style (one-line comments only)

removed unneeded 'extern' keyword for function declarations in header files

removed 'inline' keyword (compilers are better at this today)

replaced preprocessor constants with enum values

added setup option to ask to play again after game is over but not solved

fixed asking all network clients to "play again" after unsolved game end

Before, all network clients were asked to play again after a game was
lost, which could cause problems (or at least confusion) when starting
a new network game from different network clients at the same time.

Now, only the network client which started the network game is asked
to play again, while all other clients only confirm that the game is
over and return to the main menu.

fixed handling game actions during "play again" request in network mode

fixed bug with synchronization problem with "play again" in network mode

This bug caused different frame counters at the network server and
network client due to executing "GameActions()" directly after
restarting the game in "RequestRestartGame()", which increments the
frame counter on the client, but not on the server in the first game
frame.

added request dialog to play again after game is over without solving level

added handling game actions also after unsolved game end (not used yet)

added functions to check if game is over (and solved or not) (not used yet)

fixed bug with handling networking packets if request dialog is active

Before, it was possible for a network client to start a new game while
another client was still about answering a request dialog (like asking
if a tape for a solved level should be saved), causing synchronization
problems with network packet handling.

With this bugfix, handling network packets is paused during a request.

moved flag for active/inactive game panel from player to game structure

moved duplicated code to separate function

fixed bug with repeating last level in levelset when auto-playing next level

fixed bug with high scores for wrong level when auto-playing next level

added checkbox to level editor for using "game of life" bugs

added flag to level files for using "game of life" bugs

fixed bugs in game logic of "game of life" and "biomaze" elements

added playfield array "Last[][]" to store tile from last game frame

changed variable name

fixed bug with redrawing state changes for "game of life" style elements

The cause for this bug was checking the "Stop[][]" field for deciding
if a "game of life" style element has changed and needs to be redrawn,
but with the current implementation (which is in fact not correct), it
may happen that the same element changes twice in the same game frame,
causing the corresponding graphics not be updated again (and therefore
showing the wrong state).

Directly comparing the new with the old state fixes this problem.

minor code formatting change

improved checking game speed and vsync settings when selecting either one

version number set to 4.1.1.1

fixed compilation error for SDL 1.2 target

fixed compiler warning when building without DEBUG symbol

fixed target name typo in Makefile

version number set to 4.1.1.0

improved screen output text when trying to connect to network server

removed unneeded spaces in request texts

fixed problem when determining maximum word length in request text

added setup option to use extended list for game speed selection

code cleanup (using new and already existing structures)

added check if game speed is high enough for 60 Hz vsync to work

fixed buggy behaviour when only one player needed to solve a level

Before, when setting "[x] only one player must enter exit", a level
with two players was solved when one player was killed, and then the
other player entered an exit. But it was not yet solved if one player
entered an exit and the other player was still on the playfield.

Now, the first player really solves the level when entering an exit,
regardless of any other players still around on the playfield.

fixed calculating maximum length of string values in setup menu

small code cleanup

fixed drawing preferred network player in setup menu

fixed handling mouse button events when waiting for event to continue

added waiting for key or button after starting or stopping network mode

added title and footer text when starting or stopping network mode

moved starting or stopping network mode to separate function

added drawing program info on startup when initializing network

moved drawing program info on startup to separate function

added text input setup option to configure network server hostname

added ignoring screen navigation events if text input gadgets are active

improved handling network errors (show message instead of stopping program)

added gadget tag to always execute the callback action when leaving gadget

changed setting cursor when automatically activating text input gadgets

When activating text input gadgets using function "ClickOnGadget()",
do not set the cursor position at the start, but at the end of the
text in the gadget.

added support for (normal and adaptive) vertical sync (vsync)

An option was added to the setup menu to enable vsync for redrawing
the screen in synchronization with the vertical retrace, resulting
in ultra-smooth scrolling on systems where it is supported. To make
use of it in R'n'D together with common LCD displays with 60 Hz screen
refresh frequency, a faster game speed than "normal" has to be chosen,
because "normal speed" in R'n'D means 50 frames per second (which is
the original speed of the game on classic CRT displays, which used a
screen refresh frequency of 50 Hz in Europe by default, and which was
also the original game speed of classic Emerald Mine, for example).

If adaptive vsync is chosen, but not supported, a fallback to normal
vsync is done. (See the SDL wiki for SDL_GL_SetSwapInterval() for a
few more details.)

fixed compiler warnings (after adding "-Wmissing-prototypes")

Adding this warning option revealed lots of unused functions (which
are deactivated in the code using preprocessor directives for now and
which will be checked and probably removed in a later step).

fixed compiler warnings (after adding "-Wstrict-prototypes")

Adding this warning option revealed a bug with an invalid function
parameter (fixed by previous commit 085eaa4c).

fixed bug with calling function with invalid (although ignored) parameter

fixed bug with dragon fire destroying indestructible steel letter walls

Before this change, steel letter walls could be destroyed by dragon
fire, although they are set to be indestructible. This change makes
sure that indestructible elements cannot be destroyed by dragon fire
(with the exception of the "acid" element, which is to be discussed,
but would break level 014 of "rnd_paul_e_collins_iii" if changed).

As far as this was tested agains existing solution tapes, this change
does not seem to break any existing levels or tapes.

fixed list of "indestructible" elements as part of "solid" elements

The list of "solid" elements contains a section with "indestructible"
elements that claims to be identical (with one exception) with the
list that only contains the "indestructible" elements, but quite some
elements were missing (that were probably added to the other list at
some time without also being added to the list of "solid" elements).

This change adds these missing elements. (This should be solved in a
more cleaner way, without listing (nearly) the same elements in two
different lists.)

As far as this was tested agains existing solution tapes, this change
does not seem to break any existing levels or tapes.

added checkbox to player settings in editor if first player solves level

A recently made change (commits 2bf392f1 and 6999c82d) added checking
if all players in a level entered an exit before the level is treated
as being successfully solved, changing the previous behaviour where a
level was solved by the first player entering an exit, which differs
from the behaviour in the EM/EMC game engine, where team-mode levels
require all players to enter an exit before the level is solved.

However, this change broke at least one existing level that was based
on the previous behaviour (level 032 of level set "rnd_falk_sobe"), so
a checkbox in the player settings of the level editor was added to be
able to select the desired behaviour, using the "new" behaviour (all
players must enter an exit) as the default setting.

fixed nasty bug with uninitialized game engine value

This change fixed a nasty (although rarely occurring) bug caused by an
uninitialized game engine value for explosion handling (which can only
occur in the very first game frame of the very first game played after
starting the program).

This bug happens, for example, when doing "autotest rnd_helge_hafting"
or "autotest rnd_paul_e_collins_iii 013", while (as described above)
doing "autotest rnd_paul_e_collins_iii 012 013" works just fine (as
the game engine is correctly initialized by playing level 012 when it
comes to level 013, which contains that case of explosion in the first
frame that would else be played differently, causing a failed tape).

removed unused fields from structure definition

small change to prevent compiler warning

This change makes Clang happy (which issued a warning before).
(Interestingly, GCC did not care about the previous code.)

re-added file "RocksBusy.ilbm" (that was accidentally deleted)

The file "graphics/gfx_classic/RocksBusy.ilbm" was deleted by commit
5ef9148d, apparently by accident. This commit recovers the file.

fixed bug with losing image filename when scaling bitmaps

fixed bug with expiring loop sounds for global animations when playing

When playing the game, loop sounds are automatically expired if they
are not "refreshed" at regular intervals (to prevent playing sounds
for game elements that do not exist on the playfield anymore).

However, this also affects loop sounds for global animations played
during the game, so special care has to be taken to prevent them from
accidentally being expired, too.

improved clearing event queue after fading and after door/envelope requests

This is an improvement of commit ba20bcc8, which totally cleared the
event queue after fading and after door/envelope requests, and which
prevented entering multiple key presses by intention (like starting
the game and immediately going to pause mode with the very first
game frame, by quickly pressing "space" twice in the main menu).

This change only removes key presses entered by keyboard auto-repeat
during fading or door/envelope animations, effectively preventing
accidentally entered multiple key events by pressing a key like
"space" or "return" a bit too long (which then triggers auto-repeat),
while it keeps multiple key events that were manually entered.

fixed bug with not recognizing ".mode_loop: false" for music

Before, music was always played in loop mode, even if it was defined
with option "<music>.mode_loop: false".

Now, disabling loop mode for music in config files will indeed cause
music to be played only once.

Note: This commit changed the behaviour of the existing function
"PlayMusic()" (which always played music in loop mode before), which
will now play music only once, while a new function "PlayMusicLoop()"
was added, which always plays music in loop mode.

cleanup of network level function names and parameters

fixed bug with displaying the wrong level number in the game panel

When introducing a global levelset identifier and level number (see
commit b7fe72dc), the level number to be displayed on the game panel
was updated in the main menu and for network games, but not when
selecting another level from the main menu or when automatically
playing the next level after solving a level (and skipping the main
menu before starting the next level), which results in displaying the
wrong level number in the game panel in many cases.

This was fixed now by updating the level number when loading the level
from the level file (moving the function to update the level number to
be displayed from the main menu to the function to load the level).

fixed bug with different global border when playing from level editor

When using different screen border images for the in-game screen
(while playing the game, as defined by "global.border.PLAYING") and
for the level editor (defined by "global.border.EDITOR"), test-playing
a level from the level editor resulted in a graphically broken global
screen border (while everything looks correct when playing from the
main menu).

improved preprocessor macro to determine array size

fixed bug with overwriting level file info from level template

This bug (which was causing crashes due to double free()ing strings)
was a nasty side effect of changing string pointers in the file info
structure from using string references (allocated and freed somewhere
else) to copies of strings allocated and freed for each change, while
still copying the file info from the level template over to the file
info of the level file in function "ActivateLevelTemplate()" (causing
duplicates of string pointers which were therefore freed twice).

This bug was most probably the real cause for the problems fixed by
the changes in commit 759e0bea.

added checking for invalid/malicious packet size in network protocol

added checking for invalid/malicious filenames in network protocol

added using global levelset identifier and level number for network games

This change/addition was required for correctly handling identifiers
and level numbers when playing network games (for which the related
levelset possibly does not exist locally, but was sent by the network
server). This includes showing the correct level number for the level
that is played over the network, and saving the resulting high scores
for the correct level in the correct levelset score directory (again,
even if this levelset or level number does not exist locally).

added (and set) identifier and level number to global levelset structure

fixed bug with accessing string buffer that was already free()'ed

The function "getNetworkLevelDir()" returns a reference to a string
buffer that will be invalidated by each following invocation. As
"InitNetworkLevelDirectory()" also calls this function, it must be
called before "getNetworkLevelDir()".

moved level set info structure to game library layer

added loading custom artwork for network games

added always playing network games in team mode with the EM engine

When playing levels using the Emerald Mine (EM) game engine in network
mode with setup option "team mode" disabled, assume that multi-player
mode should be enabled anyway (just as it is handled with levels using
the R'n'D game engine).