minor comments and whitespace changes

removed unneeded screen redraw

fixed bug with not redrawing level number in main menu changed by cursor

removed code and image for using virtual buttons from image file

fixed potential crash bug for special versions without "classic" artwork

This bug can happen when using the program binary within a special,
customized version of R'n'D like "R'n'D jue" that does not contain
the usual "classic" artwork (like the graphics set "gfx_classic").

The problem is solved by doing a fallback to the default artwork set
(as it can be defined using "default_graphics_set") if the "classic"
artwork set cannot be found.

improved some code with more compact variant of same functionality

added options for alignment of selection lists on setup sub-screens

This adds two new "graphicsinfo.conf" options that are a bit special:

- menu.list.SETUP[CHOOSE_OTHER].align
- menu.list.SETUP[CHOOSE_OTHER].valign

These new options can be used for horizontal and vertical alignment
of selection lists on setup sub-screens (which cannot be adjusted
individually like the other setup screens using "menu.draw_xoffset"
and "menu.draw_yoffset").

Values are as usual: "left", "center" and "right" for ".align" and
"top", "middle", "bottom" for ".valign" (with "left" and "top" being
the default values).

renamed variable

moved some code outside loop that does not change in loop

fixed bug with cutting text on setup screens without scrollbar

When drawing selection lists on setup sub-screens, text was always cut
on the right side to prevent drawing text over a potential scrollbar,
even if the selection list is short enough that there is no scrollbar.

Now it is checked if the setup sub-screens has a scrollbar or not, and
the right edge where text has to be cut is calculated accordingly.

removed file that was committed into the repository by error

fixed bug when using Shift+click (IntelliDraw) in editor using empty space

Using "empty space" as the drawing element in the level editor caused
"high resolution" (half tile size) drawing even if the element under
the cursor is not a "high resolution" (Mirror Magic style) element,
causing buggy drawing when using "IntelliDraw" (Shift+click) drawing.

fixed output of debug key actions

removed drag-and-drop support if compiled with SDL version older than 2.0.5

fixed incorrect drawing of players which are moving next to each others

Before, players which were moving next to each others without a free
tile between them were drawn incorrectly (with (usually black)
background tile graphics partially drawn over one of the players).

This problem is now fixed by drawing all player tiles layer by layer
(first drawing the background tiles for all players, then drawing the
players, then drawing elements in front of the players etc.).

removed unused variable

fixed bug with player moving when releasing one of several pressed snap keys

When using TAS snap keys, pressing more than one snap key at the same
time and then releasing one of them caused the player to move.

improved handling and displaying dropped level set zip files into window

This change added the possibility to drop and extract zip files into
the currently displayed level set sub-directory on the "choose level
set" screen.

After dropping and extracting zip files on the "choose level set" or
"choose custom artwork" screens, the newly added level or artwork set
is displayed and pre-selected in the list of level or artwork sets,
so it can easily be loaded. (When dropping and extracting zip files
with level sets on the main menu screen, the last added level set will
be automatically selected and loaded.)

improved handling and confirming dropped level set zip files into window

added selecting new level set if dropped as zip file into window

fixed bug with incorrectly initializing snapping in single-step mode

When in single-step mode, snapping a tile (by using either TAS snap
keys or by using snap+direction keys) is followed by immediately
returning to pause mode. While this is fine, it was also followed by
initializing snapping (just as if the keys for snapping have been
released, which is usually indeed the case when in single-step mode,
but may be followed by pressing keys for snapping again, so the game
engine gets pressed snap keys in two subsequent frames without any
frame with "no snapping" action in between). When replaying a tape
with such snap actions (as recorded while playing), there is only one
single snap action (lasting two frames) when replaying, while there
were two snap actions (both lasting one frame) when recording, which
may result in different behaviour when replaying a recorded game.
(This can easily be tested by snapping several falling gems while in
single-step mode.)

This bug was fixed by removing the incorrect initialization of
snapping when returning to pause mode in single-step mode.

fixed memory leak

re-added some removed (deactivated) code (to be evaluated again)

removed warning from "ar" on newer systems

removed obsolete special functions to add new level or artwork set to tree

added generic function to add new level or artwork set to tree

added adding extracted level and artwork info from dropped zip files

added extracting level and artwork sets from zip files dropped into window

added experimental (cross-platform) drag and drop test code

removed experimental (Windows specific) drag and drop test code

added extracting level and artwork sets from zip files at program start

added generic support for listing and extracting zip files (using minizip)

added some more automatically generated files to be ignored by Git

fixed potential crash bug (accessing invalid memory)

This change fixes a bug that occured when invoking the changed
function with a "graphic" parameter indicating undefined graphic,
which is defined as "-1" and caused an "array out of bounds" access.

To fix this bug, the function parameter is checked before used.

fixed potential crash bug (accessing already destroyed SDL renderer)

This bug was caused by destroying textures after reinitializing the
window, which causes the SDL renderer to be destroyed and created
for the new window. Unfortunately, to destroy the old textures (which
were created using the previous renderer), the old renderer is used,
causing access to already free()'d memory.

To fix this bug, all textures are destroyed before the renderer is
destroyed (and re-created).

This bug caused regular crashes of the Windows version of the game
when switching between level sets that redefine the window size
(while no such crashes were observed with the Linux or Mac version).

fixed bug with not fading menu music on title screen when changing level set

fixed editor option to automatically count gems needed

fixed network games playing empty level files that use level template

added quickly toggling team mode by clicking on main menu "name/team" text

added using preferred network player also in single-player mode

added stopping text input when pressing joystick button

fixed bug with not updating game panel on maximum overload in MM engine

fixed not closing doors simultaneously after solving a level

fixed playfield flashing black when game started from level editor

fixed graphical bug with drawing tape controls over closed door

This bug happened in two cases where the game was started with the
tape door closed: When starting the game from the "hall of fame"
screen, and when starting the game from the level editor. In these
cases, tape controls were drawn over the closed tape door right
before the door was opened.

fixed main menu buttons for first level, last level and level number

These three buttons were handled as "main menu screen buttons"
(like the buttons and text labels for the main menu functions),
but should instead be handled as "real gadgets" (just like the
buttons to select previous or next level, but instead triggering
when mouse button is released).

This change was needed to show the button graphics for the active
state of these three buttons when mouse button was held clicked on
them.

removed lists of setup token names that were not really needed at all

fixed bug with not saving editor cascade settings for two element groups

This bug was caused by adding two new element groups to the level
editor palette area (for Mirror Magic and Deflektor elements), but not
adding them to the list of cascade settings to be saved.

The real problem was that two lists were kept in "src/files.c" for the
settings, and only one was updated when adding the new cascade groups.

To prevent similar bugs in the future, one of the two lists was
removed, as those definitions were not used anyway in this case.

fixed graphical bug when digging (crumbled) sand next to other player

fixed bug in handling debug keys when compiled without symbol DEBUG defined

fixed bug with CE actions "show envelope" and "exit player" when moving

When using the CE actions "show envelope" or "exit player" with the
condition "after explosion of player", those actions are not performed
if the player was moving at the time of explosion, as they are both
waiting for the player to reach the next tile (which will never happen
in this case).

The best solution is to only check for a "non-moving" tile position of
the player if the player is still active, which will fix the problem.

However, as the check for a "non-moving" tile position is used very
frequently, setting the tile position to "non-moving" for inactive
players was added.

added using gadgets that do not overlap with virtual buttons on Android

added toggling virtual buttons using 'menu' button on Android

changed request dialog to disable virtual buttons instead of hiding them

fixed short reactivation of overlay buttons while closing door after game

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)