radiant/texwindow: fix crash when assigning a tag, fix #146

Fix crash when assigning a tag, fix #146

Seems to be a bug introduced by @TimePath
in 0203ac024e9396ebc180d9721da40b90b7c33dc6 (Wrap more GTK)

It was probably a typo, this is the change for which a revert fixes the bug:

-gboolean TextureBrowser_tagMoveHelper( ui::TreeModel model, ui::TreePath path, GtkTreeIter* iter, GSList** selected ){
+gboolean TextureBrowser_tagMoveHelper( ui::TreeModel model, ui::TreePath path, GtkTreeIter iter, GSList** selected ){

The commit just re-add the pointer mark.

q3map2: implement symlink resolution

Ported from Dæmon engine implementation by slipher
which was a complete rewrite of one I did on Dæmon
by taking inspiration from Darkplaces engine.

- https://github.com/DaemonEngine/Daemon/blob/master/src/common/FileSystem.cpp
- https://gitlab.com/xonotic/darkplaces/-/blob/div0-stable/fs.c

netradiant: implement symlink resolution

Ported from Dæmon engine implementation by slipher
which was a complete rewrite of one I did on Dæmon
by taking inspiration from Darkplaces engine.

- https://github.com/DaemonEngine/Daemon/blob/master/src/common/FileSystem.cpp
- https://gitlab.com/xonotic/darkplaces/-/blob/div0-stable/fs.c

q3map2: better use strncpy instead of strcpy

q3map2: fix a leak (an allocated string not being freed before returning)

netradiant: some bikeshedding (whitespace, wording)

q3map2: dds/ prefix, print log at the right time, when image is found

netradiant: dds/ prefix, use makeQuoted() in log printing

preferences: onlrework the path page

- only ask for the basic options (engine path) on first startup
- mark other options as advanced and add some notes to discourage
  people not knowing what pakpath is to use it by mistake.
- just name Pak Path as Pak Path, no need to display the number,
  also makes code simpler.

radiant/build menu: reword Commandline as Command line

q3map2: accept -bsp stage option name without complaining

- using it is far more consistent with other stages,
- makes reading options less prone to errors:
  reading “q3map3 -meta” may induce to human
  reader that meta is a stage while in fact
  it's “q3map2 -bsp -meta” and meta is an
  option for the bsp stage,
- it is not bad to use the explicit -bsp stage option,
- there is no reason to recommend to not use it.

macos: delete now useless gitignore file in setup/

macos: set NSHighResolutionCapable to true

daemonmap: sync submodule

cmake: fix whitespace

macos: ignore warnings about OpenGL being deprecated

hide those warnings on maCOS:

warning: 'glBegin' is deprecated: first deprecated in macOS 10.14 - OpenGL API deprecated. (Define GL_SILENCE_DEPRECATION to silence these warnings)

macos: rework the plist and rename apple to macos in repository

library-bundler: don't use -a, it may fail on network shares

also there is no need to preserve owner

quake3: delete dead strupr

quake2,heretic2: attempt to fix undeclared strupr, <3 Kai

Attempt to fix this:

netradiant/tools/heretic2/h2data/tmix.c:292:5: error: implicit declaration of function 'strupr' is invalid in C99
      [-Werror,-Wimplicit-function-declaration]
                                strupr( buffer );
                                ^

library-bundler: rely on gcp instead of cp on macos

library-bundler: fix path

path starting with // does not work on msys2
this was introduced by mistake anyway

windows: reword a bit the gtkglext workaround

macos: set NSHighResolutionCapable to file in plist to attempt to avoid to render only the square of the GL viewport

macos: also read arguments

macos: don't run in background

macos: remove undefined value in plist

macos: introduce ugly packing hack to attempt to reduce the size of the residual GL viewport when another tab is expected to be above

HACK: The OpenGL viewport is reduced to one line on the bottom of the window when other tabs are displayed,
so the rectangle is not drawn over the window but a non-bostructing bottom line instead.

macos: support for macOS-like GTK theme with light and dark variant selection

easy-builder: get number of cpus on macOS a better way

q3map2: also look for .dds file in dds/ prefix like Doom3 or Darkplaces

also tell user which image file is found for the given texture path

netradiant: also look for .dds file in dds/ prefix like Doom3 or DarkPlaces

also tell user which image file is found for the given texture path

about: minor spacing fixes

Make About window message more explicit about maintainer and latest build

Merge branch 'max' into 'master'

bump MAX_IMAGES, MAX_MODELS

See merge request xonotic/netradiant!170

bump MAX_IMAGES, MAX_MODELS

Merge branch 'dds' into 'master'

fix dds loading in 64 bit

See merge request xonotic/netradiant!169

fix dds loading in 64 bit

readme: update with macOS and bundling instructions

plugin/vfspk3: prevent buffer overflow/stack smashing

This isssue:

```
netradiant/plugins/vfspk3/vfs.cpp:595:7: warning: array index 1024 is past the end of the array (which contains 1024 elements)
      [-Warray-bounds]
                                                tmppath[PATH_MAX] = '\0';
                                                ^       ~~~~~~~~
netradiant/plugins/vfspk3/vfs.cpp:587:5: note: array 'tmppath' declared here
                                char tmppath[PATH_MAX];
                                ^
netradiant/plugins/vfspk3/vfs.cpp:607:7: warning: array index 1024 is past the end of the array (which contains 1024 elements)
      [-Warray-bounds]
                                                tmppath[PATH_MAX] = '\0';
                                                ^       ~~~~~~~~
netradiant/plugins/vfspk3/vfs.cpp:587:5: note: array 'tmppath' declared here
                                char tmppath[PATH_MAX];
```

is producing this (on macOS):

```
* thread #1, queue = 'com.apple.main-thread', stop reason = signal SIGABRT
    frame #0: 0x00007fff58f4f2c6 libsystem_kernel.dylib`__pthread_kill + 10
libsystem_kernel.dylib`__pthread_kill:
->  0x7fff58f4f2c6 <+10>: jae    0x7fff58f4f2d0            ; <+20>
    0x7fff58f4f2c8 <+12>: movq   %rax, %rdi
    0x7fff58f4f2cb <+15>: jmp    0x7fff58f49457            ; cerror_nocancel
    0x7fff58f4f2d0 <+20>: retq
Target 0: (netradiant) stopped.
(lldb) thread backtrace all
* thread #1, queue = 'com.apple.main-thread', stop reason = signal SIGABRT
  * frame #0: 0x00007fff58f4f2c6 libsystem_kernel.dylib`__pthread_kill + 10
    frame #1: 0x00007fff5900abf1 libsystem_pthread.dylib`pthread_kill + 284
    frame #2: 0x00007fff58eb9745 libsystem_c.dylib`__abort + 144
    frame #3: 0x00007fff58eb9ff3 libsystem_c.dylib`__stack_chk_fail + 205
    frame #4: 0x000000010a8a7e59 libvfspk3.so`InitDirectory(char const*, Modules<_QERArchiveTable>&) + 7689
    frame #5: 0x00000001000e0a57 netradiant`QE_InitVFS() + 1351
    frame #6: 0x0000000100094673 netradiant`VFSModuleObserver::realise() + 19
    frame #7: 0x000000010009478a netradiant`HomePathsModuleObserver::realise() + 58
    frame #8: 0x000000010008906a netradiant`EnginePath_Realise() + 58
    frame #9: 0x00000001000d3f5f netradiant`Radiant::Radiant() + 351
    frame #10: 0x00000001000d15d1 netradiant`Radiant_Construct(ModuleServer&) + 353
    frame #11: 0x000000010008a84a netradiant`Radiant_Initialise() + 42
    frame #12: 0x0000000100088087 netradiant`main + 535
    frame #13: 0x00007fff58e143d5 libdyld.dylib`start + 1
  thread #2
    frame #0: 0x00007fff58f4abfe libsystem_kernel.dylib`__workq_kernreturn + 10
    frame #1: 0x00007fff59007636 libsystem_pthread.dylib`_pthread_wqthread + 458
    frame #2: 0x00007fff590073fd libsystem_pthread.dylib`start_wqthread + 13
  thread #3
    frame #0: 0x00007fff58f4abfe libsystem_kernel.dylib`__workq_kernreturn + 10
    frame #1: 0x00007fff59007636 libsystem_pthread.dylib`_pthread_wqthread + 458
    frame #2: 0x00007fff590073fd libsystem_pthread.dylib`start_wqthread + 13
  thread #4
    frame #0: 0x00007fff58f4abfe libsystem_kernel.dylib`__workq_kernreturn + 10
    frame #&1: 0x00007fff590076e6 libsystem_pthread.dylib`_pthread_wqthread + 634
    frame #2: 0x00007fff590073fd libsystem_pthread.dylib`start_wqthread + 13
```

Merge branch 'illwieckz/gtkglext' into 'master'

Make possible to build custom GtkGLExt and build macOS bundle

See merge request xonotic/netradiant!168

cmake: minor tweaks

macos: produce working NetRadiant.app

macos: rename radiant to netradiant

bundle: bundle macos binaries (no .app file done yet)

tools: update daemonmap submodule

tools: no need for minizip/ directory, pkg-config tells it

easy-builder: also build builtins, make macos use builtin gtkglext

macos: disable drawing xorrectangle (rectangle selection feedback)

disable drawing xorrectangle (rectangle selection feedback)
to workaround crash, selection still works

macos: workaround faulty destructor

gtkglext: build macos non-x11 gtkglext

gtkglext: build and bundle built-in gtkglext

There is a chicken-and-egg problem, so the build must be done in two steps,

First build built-in libraries:

cmake -G "Unix Makefiles" -S. -Bbuild -DBUILTIN_GTKGLEXT=ON -DBUNDLE_LIBRARIES=ON
cmake --build build --parallel $(nproc) -- builtins

Then detect the built-in libraries and build netradiant and install the bundle as usual:

cmake -S. -Bbuild
cmake --build build --parallel $(nproc) install

There is no need to set cmake options on second step but cmake still has to be called again to detect the new files.

bundler: detect missing libraries

gtkglext: build builtin GtkGLExt as cmake external project

Do:

cmake -G "Unix Makefiles" -S. -Bbuild -DBUILTIN_GTKGLEXT=ON
cmake --build build --parallel $(nproc)

A GtkGLExt tree knonn to build will be downloaded into:

build/gtkglext-prefix/src/gtkglext

and installed into:

build/external

NetRadiant does not build against this builtin GtkGLExt yet.

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