fixed compiler warning due to missing function prototype

changed fading complete screen only if all viewports have changed

(... or if the global border has changed, of course)

This change partially reverses the behaviour caused by commit 5c6c42f1
("fixed fading complete screen if playfield viewport has changed"),
which prevented partially fading screens that only partially changed.

The main intention of this change is to make it possible that the tape
recorder viewport, if unchanged, keeps being visible while fading the
remaining parts of the screen (especially with artwork sets without
global border graphics).

However, if *all* viewports changed, the whole window will be faded.

renamed function

fixed bug with comparing wrong viewport variables

fixed bug with not clearing editor doors if no background image defined

added support for dynamic window and playfield size

This change added a few new and extended directives for the graphics
configuration done in "graphicsinfo.conf" files in artwork and level
sets.

For the window and playfield definitions, there are new properties
".min_width" / ".min_height" and also ".max_width" / ".max_height"
to define minimum and maximum window and playfield sizes. This is
useful to dynamically support different screen sizes which have a
different screen aspect ratio (like tablets with 4:3 or 16:9 sized
screens), always using the full screen size without black borders
that cannot be used for the game.

To be able to place the "doors" (game panel and tape recorder) also
dynamically aligned with the right or bottom screen border, margins
can be defined for the playfield to not use these areas, which can
then be used for the doors being aligned to the corresponding window
edges. That's what the new ".margin_left", ".margin_right",
".margin_top" and ".margin_bottom" directives are for, together with
".align" and ".valign" properties to align the playfield or the doors
horizontally ("left", "center" or "right") or vertically ("top",
"middle", or "bottom").

To make sure that the game playfield always uses complete tiles (or
half tiles, or any other size), there's also a new directive called
".align_size" to specify the alignment tile size for the playfield
(which is set to half a tile by default).

The last enhancement extends playfield border sizes to be defined
for each border individually, so instead of a unique ".border_size"
for all playfield borders, different border sizes can be specified
by using ".border_left", ".border_right", ".border_top" and
".border_bottom". If they are not set, they inherit their value from
".border_size" just like before. Using the value "-1" causes borders
to be calculated dynamically -- this only makes sense when using a
variable window size (by using ".min_width" and/or ".min_height") with
a fixed playfield size, so borders can also have variable size. This
is mainly useful for using a fixed (or limited, by ".max_width/height)
playfield size while still being able to display global animations on
the whole surrounding playfield border area.

added support for dynamically sized (and placed) editor palette

When using the value "-1" for columns or rows of the editor palette
(that is, values "editor.palette.cols" and "editor.palette.rows"),
these numbers are dynamically calculated according to the size of the
editor playfield area.

When setting "editor.palette.x" or "editor.palette.y" to "-1", also
the position of the editor palette is dynamically calculated (to be
centered within the playfield area).

(As these values for columns and rows are used at a very early stage
in the level editor to allocate memory for palette buttons, make sure
that no negative numbers are used here for memory calculations.)

fixed potential crash bug with empty (padding) element buttons in editor

This fixes a bug that can cause a crash when accessing memory that was
already free()'ed again, because the variable that contains the amount
of allocated memory is not updated when memory allocation changes.

The bug is fixed by updating that variable when memory is allocated.

fixed nasty x/y typo bug

improved loading menu graphics config settings (more generic solution)

replaced preprocessor constants with enum values

removed some unused stuff from MM engine code

fixed handling debug keys to prevent handling keys twice

This change has two effects:

- handle debug keys before handling all other defined keys
- if debug key was processed, do not handle this key again

added deactivating text (and selectbox) gadgets with "Escape" key

removed leaving screens with "Escape" key if gadgets are active

added showing virtual keyboard for playfield text input in editor (Android)

fixed bug with stopping editor text input before reaching the last line

fixed bug with asking to play again after game over when playing tape

fixed some bugs with (not) displaying virtual buttons

This corrects the following situations regarding virtual buttons:

* Fixed displaying virtual buttons while tapes are played.
* Fixed displaying virtual buttons after all players were gone.
* Fixed *not* displaying virtual buttons when tape stopped playing and
  the user can continue playing the game.

removed support for SDL 1.2

added support for level sketch copy/paste using Ctrl-c/v in level editor

This enhances the existing support for dumping brushes from the level
editor as level sketches (to STDOUT and/or "stdout.txt", which can
then be added to posts in the R'n'D web forum), as follows:

- Ctrl-c copies the current level (or brush) into the clipboard
- Ctrl-x copies the current level, but as small sized level sketch
- Ctrl-v pastes a level sketch from the clipboard into the current
  level (or brush, if active)

This makes it easy to copy level sketches from the level editor to the
R'n'D web forum, and also from the forum to the level editor (just
temporarily go to the "reply" form for a post containing a level
sketch, then copy the "source code" text of the level sketch to the
clipboard using Ctrl-c, then paste it into the level editor using
Ctrl-v).

This will also make it easy to copy a whole playfield from one level
file to another in the level editor (playfield size will automatically
be adjusted to the size of the pasted playfield).

moved (and renamed) global variables to game structure (ZX/ZY)

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.