q3map2/light_bounce: prevent infinite loop on obscure bias compute

New code produces same result without loop at all, so
it cannot fall in infinite loop, and it is faster in
use cases requiring more than one loop in previous code.

The Unvanquished vega map is known to trigger the bug:
https://github.com/UnvanquishedAssets/map-vega_src.dpkdir
I reproduced it multiple time on various hardware (8 core FX-9590,
12 core/24 thread Ryzen 9 3900X) with commit af40508 and using
final compilation profile edited to use -fastbounce instead
of -fast option.

The symptom is simple, q3map2 stucks there:

--- Radiosity (bounce 1 of 8) ---
--- RadCreateDiffuseLights ---
0...1...2...3..

Or somewhere else in that progression bar given your hardware
and the amount of core your CPU has.

When stuck, all the CPU cores are running 100% but the thread
never returns (a strace can reveals it, a gdb backtrace too).

Thanks to @slipher for the precious advices and improving my first
attempt to fix it.

For more information on the issue, I asked:

> which negative value never can become positive
> when incremented infinitely?

slipher said:

> for a double, any value less than -2^53 would have this property
> don't know for float off the top of my head

But then, it means that's theorically verified this loop was able
to run forever in some case.

I don't know what this code is doing anyway, but at least we can
keep the behaviour without requiring to understand it.

q3map2 * fix: do not affect styled lightmaps by floodlight
port of https://github.com/id-tech-3-tools/map-compiler/pull/12
https://github.com/id-tech-3-tools/map-compiler/commit/87b8589a3a5069c482243bb33c587fc18911746c

gamepacks: add Alien Arena (GPL)

Merge branch 'vfs' into 'master'

Some VFS fixes: attempt to load .pak and .wad, load png fallback even if game does not support png, do not load engine path twice

Closes #141 and #139

See merge request xonotic/netradiant!166

radiant/texwindow: use TextureBrowser_showWads when possible

instead of doing alternatively:
- !string_empty( g_pGameDescription->getKeyValue( "show_wads" ) )
- TextureBrowser_showWads()

the last one calling:
- !string_empty( g_pGameDescription->getKeyValue( "show_wads" ) )

radiant/texwindow: only show per-wad textures, update texture window title with wad name

radiant/texwindow: beautify wad list

radiant/texwindow: test .wad file extension case-insensitively on all systems, fix #139

radiant/image: load fallback images with png loader even if game does not support png, fix #141

Current implementation has a minor issue, when "Hide Fallback" option is
disabled, the fallback image does not appear in texture browser before radiant
makes use of it (for example creating a brush before selecting a texture).

Since "Hide Fallback" option is enabled by default and it never makes sense
to apply those fallback intentionally, I don't care for the moment.

I would like to see a better implementation that would work for all
editor images (like the one set with `qer_editorImage` shader keyword).

radiant/vfs: do not load enginepath twice

Do not load engine path twice when the home path and the engine path is the same,
This happens when a game does not have home path.

Also rework a bit that code, and prints useful information on console.

radiant/vfs: rework vfs loading, attempt to load wads and paks, ref #139

Also load .pak extracted as .pakdir (can be done using unpak).
NOTE: that feature would have to be supported by map compilers to
be useful. This is still handy for testing purpose so I'm keeping it.

Also prints on console the opened pak directories.

Fixing compilation with -DBUILD_SHARED_LIBS=ON

Libraries under libs should be forced STATIC since there circular
dependencies and implicit linking with external libraries. Otherwise
compilation will fails with underlinking errors.

daemonmap: update submodule for fixes for compilation with '-DBUILD_SHARED_LIBS=ON'

Merge branch 'illwieckz/gcc10' into 'master'

Fix compilation with GCC 10, <3 @SpiKe, fix #140

Closes #140

See merge request xonotic/netradiant!163

daemonmap: update submodule

h2data: use static for homonyms

q3data: already defined in cmdlib.h

q2map: declare once, define as extern

q2map: do not define globally when used locally

h2data: do not redefine if unused

h2data: declare once, define as extern

q3map2: declare or define with Q_EXTERN

q3map2: already defined in inout.h

readme: add download link, make sure people download source using git, and other rewording

q3map2: generate unvanquished minimap sidecar file

also enable the option for work-in-progress smokinguns port on dæmon

q3map2: remove useless zlib dependency

Comment ColorToBytesNonZero and fix zero detection in light.c.

Fix ioq3 compatibility bug with lightgrid and only one light.

Correct math requires the ambient component of the lightgrid to be zero
in that case. However, ioq3 ignores lightgrid cells with all zero ambient
value, EVEN if the directed value is nonzero.

This change sets the ambient value to #010101 if it'd be pitch black. This
should be a minimal change without affecting light hue that fixes
lightgrid rendering. In engines like DarkPlaces, almost no map should
look different from this.

Fixes #137.

easy-builder: make it compatible with Ubuntu 18.04/CMake 3.10

on CMake 3.10i, options -S and -B were not supported yet

readme: update

Merge branch 'clang' into 'master'

readme: gcc is not required anymore on FreeBSD and macOS

easy-builder: do not enforce gcc on FreeBSD and macOS

clang/error: fix: call to convertToOpaque is ambiguous

clang/error: fix: non-type template argument refers to function Export that does not have linkage

 clang/error: fix: non-type template argument refers to function ExportThunk that does not have linkage

radiant/build: add comment telling MaingFrame_getWindow() is suspicious

docs: set index.html to redirect

contrib: update urls and names

radiant/build: center buildmenu editor on screen

I haven't managed to center it on main windows
centering on screen is still better than top -left

bikeshedding: unifying some variable in contrib/bobtoolz

bikeshedding: unifying some variable in contrib/shaderplug

contrib/bobtoolz: center some dialog windows on main window

also rename the Path Plotter window title correctly

contrib/brushexport: center export window on main window

contrib/prtview: center config window on main window

also center color picker on config window

radiant/xywindow: center background select error message on main window

radiant/prefs: center preferences window on main window

radiant: center messagebox on main window

radiant/about: reword message

radiant/about: rework layout

radiant/about: add link to website

radiant/about: make issue tracker url clickable

contrib/bobtoolz: unify about dialog construction

contrib/sunplug: unify about dialog construction

contrib/prtview: unify about dialog construction

cmake: install global.xlink

global.xlink: fix q3map2 handbook url (was dead link)

global.xlink: update netradiant website url

daemonmap: sync submodule to print own git version

cmake: drop useless version.h

tools/cmake: do not compute git version again

bundle: make daemonmap buildable and bundlable like other tools as submodule

easy-builder: make possible to check for more than one submodule

easy-builder: strip release binaries, support older cmake

- use `install/strip` target for release build
- CMake used in Ubuntu bionic (currently used to build
  redistributable packages, old enough to be older than
  most of the commonly used distributions) does not
  support -S option yet

readme: use cmake -S instead of undocumented -H

undocumented -H for source directory was replaced
by documented -S and -H is now an alias to --help

easy-builder: improvements

- handle arguments with spaces properly
- provide fallback for nproc
- replace cmake undocumented -H with -S
- do not require to change directory

bikeshedding: minor readme fix

q3map2/smokinguns: fix .tex writing, <3 @vloup

Thanks ElBarto (@vloup) for reporting this

bikeshedding: reduce diff noise with Garux tree

see:
https://github.com/Garux/netradiant-custom/commit/4292115ef741ff066b382c094f8c7814afaccf59#diff-386c03feac6f15ccc15c4dab7bb97320R342

that's not fair

q3map2: pad with zero, not with null pointers, ref #160

see:

https://gitlab.com/xonotic/netradiant/-/merge_requests/160#note_287938815

radiant/buildmenu: rename ".[ExecutableType]" as "[ExecutableExt]"

- introduce GDEF_OS_EXE_EXT
- no need for RADIANT_EXECUTABLE anymore
- keep "[ExecutableType]" parsing but it looks useless

radiant/buildmenu: fixup, ref #138

radiant/buildmenu: better executable extension strip, ref #138

this fixes issue #138 on my end

bikeshedding: some touchup on library-bundler

Merge branch 'missingarg' into 'master'

q3map2: fix segfault when missing bspfile arg on light stage

See merge request xonotic/netradiant!161

q3map2: fix segfault when missing bspfile arg on light stage

Merge branch 'memoryfixes' into 'master'

q3map2: fix memory errors and misuses

See merge request xonotic/netradiant!160

q3map2: use safe_malloc0 when safe_malloc is followed by a memset to 0 with the same size

q3map2: zero the verts array on initialization, verts are tested to not be null in FindMetaVertex()

q3map: magic string, know about the asctime's trailing \n

q3map: magic string, no closing parenthesis

tools/quake2: use same strupr other tools use

tools/hd2data: strupr is windows only, define it elsewhere

some compiler may ask if strstr would have to be used instead, which is wrong

q3map2/vis: do not read outside of script before detecting end of script

autoexpand: add an alternative way to autoexpand while zeroing the allocated memory

use it to allocate shader lump, without that, garbage from uninitialized memory may be written
after the ending \0 of strings

bsp lump write: pad with zeros, not with random unitialized memory data

q3map2/vfs: free temporary allocated dirlist when not used

LoadPNGBuffer: use safe_malloc0 to allocate a zeored imate buffer

valgrind reports there may be issues if this is not initialized

introduce safe_malloc0, also 0 the memory before usage

note called safe_calloc because calloc does not have the same syntax
and this safe_malloc0 uses malloc syntax

unnecessary SAFE_MALLOC ifdef: always defined

valgrind: Conditional jump or move depends on uninitialised value

fix a buffer overflow due to a string not being large enough to store ending \0

Merge branch 'easy-builder' into 'master'

introduce easy-builder script

See merge request xonotic/netradiant!157

update readme

add simple easy-builder script

fail library-bundler script on error

enable crunch if available

crn_rgba: use “algorithm” header to avoid std::min/max being unknown

avoid this error on some systems (Clang on macOS or FreeBSD):

netradiant/libs/crnrgba/crn_rgba.cpp:93:38: error: no
      member named 'max' in namespace 'std'
    const crn_uint32 blocks_x = std::max(1U, (ti.m_width + 3) >> 2);

Merge branch 'illwieckz/gcclt7' into 'master'

Fix compiling using GCC older than 7, obsolete !143, fix #122, <3 @zturtleman, bananovenc

Closes #122

See merge request xonotic/netradiant!159

Fix compiling using GCC older than 7, obsolete !143, fix #122, <3 @zturtleman, bananovenc