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).

disabled auto-incrementing level number when playing in network mode

disabled auto-playing next level when playing in network mode

swapped condition logic for better readability

added sending level file (and level template) for network games

As the level to play may not exist on the receiving side (but only on
the sending side -- the client who initiates the network game), this
change adds sending the level file (and the level template, if needed)
to all receiving network clients, so all clients always play the exact
same level (and do not have to worry about having the same level sets).

prepared level number in game panel for network games

removed storing two strings in function for level filenames

Before, the function "getLevelFilenameFromBasename()" stored two
filenames in static string pointers, one for the level filename, and one
for the corresponding level template filename, because the results were
only used as references before. As they are now stored as string copies
outside if this function, it is sufficient to only store the last result.

This change is directly related to commit 759e0bea.

changed filename string handling for level file info structure

Before, level filenames in the level file info structure (members for
"basename" and "filename") were only handled as references to the
basename/filename strings allocated elsewhere, so if those strings
changed, the references in the level file info structure also pointed
to those changed strings. When determining filenames for both level
files and corresponding level templates, references to level filenames
were replaced by references to template filenames, which resulted in
unexpected/wrong behaviour.

Now, these level filenames are handled as copies of the strings, so if
the strings change, the string copies still stay the same.

added missing check for changed level in editor ("use template for CEs")

fixed extremely nasty bug when using level template in level editor

When checking the checkbox "use template for custom elements" on the
"editor" tab on the level properties page, the playfield area was
replaced with a "broken" level, caused by horrible bugs in the level
initialization function "setLevelInfoToDefaults_MM()". Fixed now.

This bug was introduced with the MM game engine in version 4.1.0.0.

added functions to network code for sending files

added functions to handle network level directories

added function to dump network message buffer (for debugging)

fixed bug which caused wrong level number in "hall of fame" after playing

This bug is related to commit e2260726, which forced displaying the
high score values for the correct level number, but refused to show
the correct level number in the headline of the "hall of fame" (but
displayed the number of the next level to play). This is fixed now.

completely refactored client and server side network protocol code

improved handling network protocol version mismatch

In case of different network protocol versions at network client and
server, show an error message and disable networking instead of
aborting the program with a "fatal error" style message.

changed using network protocol by starting with protocol version check

added support for protocol version check as first protocol command

changed network protocol version from 1.2.0 to 2.0.0

removed unused variables

fixed bug in network mode with pausing game on errors when not playing

fixed warning for missing level template when loading level

added disabling networking when executing command line functionality

Auto-testing solution tapes could cause crashes if network was enabled.

added using counter of players that still have to enter an exit

added counter of players that still have to enter an exit (not used yet)

renamed function

fixed pausing network games if the game already ended

fixed exiting players with custom elements when playing multi-player games

Before, exiting a player using custom element actions only worked in
single-player mode. (In multi-player mode, the level was either
immediately solved if the exited player was the "local player"
(ignoring other players still existing on the playfield), or the
exited player was not removed from the playfield, if it was not the
"local player".)

Now it works correctly also in multi-player games.

fixed solving level when playing multi-player games

Before, solving a level was broken in both local and network
multi-player games:

In local multi-player games, the level was only solved if the player
marked as "local player" (which is only one of all players in a level)
entered the exit (but not if any other player entered the exit). As an
additional bug, the level was *always* solved as soon as the "local
player" entered the exit, regardless if any other player was still on
the playfield.

In network multi-player games, the level was always solved for the
first player entering the exit (as this player is always the "local
player" in one of the participating network clients), while the game
was frozen on all other network clients (and had to be manually ended
by using the Escape key or stop button).

With this fix, multi-player games will always be solved (locally or on
all network clients) if the last player on the playfield has entered
the exit.

added function to remove player and cleanup last player field

fixed ignoring mouse actions with MM engine using virtual buttons

As virtual buttons do not make much sense when playing games using the
Mirror Magic game engine, mouse clicks should be handled as usual even
when touch input is set to "virtual buttons" (causing mouse clicks to
be ignored before this change).

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

fixed using private level set if network game level set not found

Before, if a network client started playing a level from a level set
that is not installed for some other network client, that client did
a fallback to playing the default private level set. This caused the
network clients effectively playing completely different levels with
the input actions of the other clients, which does not make any sense.

Now, the network game is stopped for all clients in such a case.

added configurable border size also for drawing network players

added drawing players with level preview when using network or team mode

fixed setting player network flag without being connected to network server