removed unused function parameters

This issue was found by using GCC with option "-Wextra".

removed unused functions

This issue was found by Cppcheck (static C/C++ code analysis tool).

removed shadowing of outer variable by local variable

This issue was found by Cppcheck (static C/C++ code analysis tool).

reduced scope of some variables

This issue was found by Cppcheck (static C/C++ code analysis tool).

removed redundant and unused code

reduced scope of some variables

This issue was found by Cppcheck (static C/C++ code analysis tool).

fixed accessing array out of bounds

This issue was found by using Clang with option "-fsanitize=undefined".

reduced scope of some variables

This issue was found by Cppcheck (static C/C++ code analysis tool).

removed redundant initialization

This issue was found by Cppcheck (static C/C++ code analysis tool).

removed redundant conditions

This issue was found by Cppcheck (static C/C++ code analysis tool).

fixed undefined signed integer overflow for expression "1 << 31"

This issue was found by Cppcheck (static C/C++ code analysis tool)
and by using Clang with option "-fsanitize=undefined".

added optional sanitizer flags when using Clang

moved compiler flags for analyzing to CFLAGS / LDFLAGS

changed name of compilation flag to be more generic

fixed compiler warnings with GCC 11

added explicit operator precedence (stylistic only)

This issue was found by Cppcheck (static C/C++ code analysis tool).

removed shadowing of outer variable by local variable

This issue was found by Cppcheck (static C/C++ code analysis tool).

removed declaration of non-existing data object

This issue was found by Cppcheck (static C/C++ code analysis tool).

added dirty workaround to open program window on specified display

As SDL function "SDL_GetNumVideoDisplays()" seems to be broken on some
systems (tested with older Linux systems and SDL 2.0.0 and 2.0.6, but
also current macOS with SDL 2.0.20), do not use it for now to check if
the specified display number is valid.

changed using same (little endian) pixel format everywhere

This change also has the effect that the code compiles using versions
of the SDL library older than 2.0.5 (which introduced the pixel format
alias name used before).

added rule to Makefile to ensure reproducible builds

Declare build dependency to ensure consistent make execution order.
See https://reproducible-builds.org/ for why this is good.

Without this patch, parallelism would make a difference in
the produced config.o

This patch was done while working on reproducible builds for openSUSE.

Thanks to Bernhard M. Wiedemann for this patch!

changed resizing window instead of recreating it on window size changes

This has the advantage that the window stays at its screen position
instead of always being newly created in the middle of the screen.

added some missing window events to debug output

added command line option to select display to open program window

fixed program usage text

fixed using delay counter for Android

fixed compiler warning

added flag to tape files to explicitly mark solution tapes

merged functions for checking growing walls (normal and steel)

code cleanup

added ignoring remaining level chunks after chunks with wrong size

added ignoring remaining level chunks after chunks with errors

fixed crash bug when accessing invalid group data for broken level files

Thanks to Quipyowert for finding and fixing this bug!

fixed crash bug when accessing invalid change page for broken level files

Thanks to Quipyowert for finding and fixing this bug!

fixed compiling with SDL versions before 2.0.14 without SDL_OpenURL()

added alpha channel for anti-aliasing to element collecting image

changed function to write element collecting image to create PNG file

added asking for uploading remaining tapes when enabling score server

fixed bug with uploading tapes when score server is disabled

fixed bug with resetting level set handicap when using "last played" menu

added overwriting local with server score entries to add missing fields

fixed bug when using different normal and deadly shield time

fixed crash bug caused by custom loading screen support

added using globally predefined X/Y arrays

improved checking array (using modulo instead of duplicated values)

added delay counter data type and adjusted related functions

added level info config option to disable time limit for all levels

While it was already possible to disable the time limit in the setup
menu to play a level without any time limit, the resulting tapes only
replay successfully if the time limit is still disabled, of course.
Obviously, this is not the case on the central high score server.

However, the level sets in the "R'n'D jue 2022" games collection are
all designed to be played without time limit, and the time limit is
disabled in the default setup options of all games. This currently
has the following two disadvantages:

1. The resulting tapes fail on the high score server, which has time
   limit always enabled.
2. If the user of "R'n'D jur 2022" games enables the time limit in the
   setup menu, the levels cannot be solved anymore.

This patch adds a new time limit option to the level info config file,
which makes it possible to flag whole level sets as "to be played with
time limit disabled", regardless of what the individual user's setting
for "time limit" was set to.

This patch makes it possible to fix case (1) above by adding the new
level info config option to all affected levels on the high score
server, but would require a program update to also fix case (2) on the
client side.

added separate game variable to check for time limit setup option

renamed variable

fixed synchronization problem when configuring keyboard

This is a somewhat similar bug like that in the previous commit, but
in this case only one event was processed for each screen update,
causing many events (like those produced from analog sticks that are
moved while the keyboard is about to be configured) to be processed
one after the other, each with a full video frame delay in between.

The fix works just like that used when configuring game controllers.

fixed synchronization problem when configuring game controller

This fixes a bug that can happen due to bad initialization of a delay
counter (to limit the maximum time spent on handling events), causing
it to reach the maximum delay while the screen refresh is also waiting
for the video delay. Resetting the counter right after screen update
fixes this potential problem.

version number set to 4.3.2.3

added using setup option for displaying overlay touch buttons

added setup option for overlay touch buttons (not used yet)

added not using touch buttons if touch controls are disabled

fixed bug with short reactivation of overlay buttons while closing door

added printing envelope text when dumping level in debug mode

added support for "Home" and "End" keys to text gadgets

added support for fallback text file for element description files

fixed warning when not finding any valid level sets in level directory

fixed not creating user level set

changed creating directories with private permissions only

changed log level for using fallback artwork from "warning" to "debug"

fixed bug with broken text area content when editing envelope text

cleanup of CSS for Emscripten platform

added CSS to fill browser window for Emscripten platform

removed some setup options for Emscripten platform

removed larger default screen size for Emscripten platform

changed some platform-related preprocessor definition names

minor code cleanup

improved element descriptions for magic walls

added larger default screen size for Emscripten platform

added loading screen before starting the game for Emscripten platform

fixed another problem with leaving scores screen on Android platform

On Android, when touching the screen anywhere in the hall of fame
after playing a game, this should either start the next game or
return to the main menu. However, this is done when touching the
screen with "finger down", not with "finger up again", sending an
additional "release" event in the main menu, triggering the menu
entry under the finger.

This fix triggers leaving the hall of fame on the "release" event
only.

fixed bug with leaving scores screen on Android platform (again)

This is the same bugfix as in commit a42f3002 (which for whatever
reason was "unfixed" since that commit).

fixed continuing from hall of fame after playing on Android platform

added browser icon for Emscripten platform

fixed changing back link in hall of fame to "next" only when needed

added special mode to continue from hall of fame when pressing "space" key

added playing next level when leaving hall of fame via back link

minor whitespace change

added storing next level to play from hall of fame

moved code to start playing from hall of fame to separate function

added flag to continue from hall of fame when pressing "return" key

changed setting flag to continue playing next level from hall of fame

renamed variable

renamed variable

added showing score even if not added to local score file

This fixes a special case: If a score was not added to the local score
file (because all entries in the local score file have a better score,
and there are no free slots in the local score file anymore), the hall
of fame was completely skipped before. This was handled differently to
a score that was still not under the best 100 ranks on the high score
server for that level, but that was written to the local score file
(either because it was not full yet, or because it was better than the
last entry in the local score file).

This fix adds showing the score (with a rank of "???" on the last list
position, as usual) even if it was not written to the local score file.

changed name of preprocessor constant

fixed setting text size for clearing input on name selection screen

fixed using font draw offset when clearing input on name selection screen

minor whitespace change

fixed bug with leaving scores screen on Android platform

This change fixes a problem that occurs on the scores screen after
playing a game on Android, where the screen is left by any user input,
but then does not reset the flag that the last screen was the playing
screen, causing all other "choose tree" style screens to also exit on
any user input (which makes it impossible to select a different level
set on the level set selection screen afterwards, for example).

added support for global animations on name selection screen

fixed assumed position of screen keyboard for Android platform

fixed bug with referencing out-of-scope structures

fixed clicking "edit" text using font offset on name selection screen

changed unusual wording of "handicap" setup option

fixed handling gadgets when touch controls are disabled on Android

renamed activity name for Android platform