Really really fix windows build

Really fix windows build

Remove -Wno-write-strings

Remove -Wno-parentheses

Remove -Wno-pedantic

Fix windows build

Remove -Wno-switch

Remove -Wno-extra

Add error flags (no changes yet)

Fix prtview about dialog regression

Wrap more GTK

Suppress warnings

Merge branch 'smooth' into 'master'

Smoothing stuff

See merge request !70

Merge branch 'vfsmodels' into 'master'

refresh VFS on model refresh, refresh models on VFS refresh

See merge request !72

Merge branch 'gamepackdeps' into 'master'

dpk vfs: now loads DEPS file from game pack

See merge request !71

Merge branch 'fixwarn' into 'master'

Fix some obvious warnings

See merge request !57

Add CPack targets package and package_source

Update CMakeLists.txt

refresh VFS on model refresh, refresh models on VFS refresh

Before:

- refreshing textures were refreshing VFS (to load enabled texture packs)
  but models were not refreshed
- refreshing textures were refreshing models but textures would be missing
  since the VFS was not refreshed and texture packs would be not loaded

After:

- refreshing textures or models both refresh VFS, textures and models,
  this way you don't anymore reload models without refreshing textures, or
  refresh VFS without reloading models, etc.

dpk vfs: now loads DEPS file from game pack

- this way, there is no need to hardcode the loading of
  tex-common or res-buildables to get common textures
  and games buildables models, the game developers just
  have to provide a DEPS file with their game pack, and
  there is no need to enforce the name for this kind of
  stuff

deduplicate empty string affectation

These the first line is useless, since being overwritten:

g_pStrWork = "";
g_pStrWork = new char[1];
g_pStrWork[0] = '\0';

Also, that first line uselessly raises this warning:

ISO C++ forbids converting a string constant to ‘char*’

p3dlib.c: include “cmdlib.h” on non Windows platform

- implicit conditional declaration of function ‘Q_filelength’ and ‘Q_stricmp’
  They come from “cmdlib.h”, so we must conditionally include it too.

use %ld instead of %d for “long int” arguments

- format ‘%d’ expects argument of type ‘int’, but argument 2 has type ‘long int’
  use ‘%ld’ instead

use NULL as sentinel instead of 0

- missing sentinel in function call
  Some functions like g_object_set() expect a variable list
  of arguments terminated with NULL sentinel, legacy code
  was using 0 instead.

some “char*” to “const char*” declaration

- ISO C++ forbids converting a string constant to ‘char*’
 These are obvious fixes when it's better to declare functions
 as expecting “const char*“.

one argument, one argument

Ingar was right (slashslash no more)

Merge branch 'TimePath/gtk++' into 'master'

GTK Upgrade

See merge request !59

Fix minizip regression

Closes #107

Useless memcpy wasn't useless...

Fix legacy makefile

Remove a few glib includes in headers

Remove a few gdk includes in headers

Remove the final gtk headers in headers

Rebase onto master

s/nullptr/ui::null/ where appropriate

Fix GCC7 issues

Fix MSYS2 issues

Fix GTK3 crash

Wrap GtkGLArea

Use non-deprecated GDK key constants

Wrap g_object_unref

Wrap g_signal_connect

Fix release build

Remove <gtk/gtk.h> from radiant/dialog.h

Remove <gtk/gtk.h> from radiant/textureentry.h

Remove <gtk/gtk.h> from gtkutil/xorrectangle.h

Remove <gtk/gtk.h> from gtkutil/window.h

Remove <gtk/gtk.h> from gtkutil/widget.h

Remove <gtk/gtk.h> from gtkutil/toolbar.h

Remove <gtk/gtk.h> from gtkutil/nonmodal.h

Remove <gtk/gtk.h> from gtkutil/entry.h

Remove <gtk/gtk.h> from gtkutil/dialog.h

Remove <gtk/gtk.h> from gtkutil/cursor.h

Remove <gtk/gtk.h> from gtkutil/container.h

Remove implicit Window constructor

Purge gtk_container_add

Purge GTK forward declarations

Purge gtk_widget_show from radiant/*

Wrap gtkutil/dialog

Wrap gtkutil/cursor

Wrap gtkutil/container

Remove <gtk/gtk.h> from most of radiant/*

Remove <gtk/gtk.h> from plugins/*

Wrap gtkutil/window

Wrap gtkutil/widget

Wrap gtkutil/toolbar

Wrap gtkutil/nonmodal

Wrap gtkutil/menu

Wrap gtkutil/image

Wrap gtkutil/frame

Wrap gtkutil/entry

Wrap gtkutil/dialog

Wrap gtkutil/button

Wrap GtkListStore

Align with GTK object hierarchy

Propagate ui::Window

Wrap GTK

Remove 156 files

CMake: really fix installing gamepacks

CMake: mark modules as a dependency of radiant

CMake: switch minizip casing

CMake: Make dll bundling optional

Fix the mapping wiki link

CMake: fix installing gamepacks

CMake: make default install prefix actually work

CMake: install all tools

CMake: add install target

Default symbol visibility to hidden

Merge branch 'subdiv' into 'master'

[q3map2] allow to set arbitrary default light subdivision value

See merge request !67

Merge branch 'illwieckz/files'

Remove leading . in CMAKE_EXECUTABLE_SUFFIX if it has one, but don't modify it

Remove leading . in CMAKE_EXECUTABLE_SUFFIX if it has one

remove version string from home config path

- GtkRadiant uses ~/.radiant/1.6.0/
- UFORadiant uses `~/.ufoai/radiant/
- DarkRadiant uses ~/.darkradiant/
- NetRadiant was using uses both ~/.netradiant/1.5.0/ and ~/.netradiant/1.1.5/, which is not defined by the version but the way you build it.
- AARadiant uses `~/.aaradiant/1.5.0/
- XrealRadiant used ~/.rxealradiant/

I guess no one Radiant out there needs a version number in config path to not mess with other Radiants.

supports XDG_CONFIG_HOME on Linux

- stores NetRadiant config in ~/.config/netradiant
  or in ${XDG_CONFIG_HOME}/netradiant if set
  instead of ~/.netradiant

fix homedir version

- was ~/.netradiant/1.1.5 (1.MAJOR.MINOR)
  instead of ~/.netradiant/1.5.0 (MAJOR.MINOR.0)

really filter notex (filter fallback)

- filter all /notex and /shadernotex
- filter both shadername and texture name
- only add group if shader isn't notex
- refresh the tree (hence groups) on toggling
- rename "filter notex" to "filter missing"
  to avoid confusion, because that's what that
  does

remember last opened folder

add command line support

- enable generic Gtk options
- allow to pass map filename as argument
  and open it

[dpkdir] introducing dpkdeps.h

It was a change included in a commit by Neumond I haven't picked-up
because I disagreed with some other changes this commit introduced.

But this part is legit, it moves some DPK's DEPS file read and
version computation code to its own file. This clean-up a bit the
vfspk3 code and by the way it will be needed by future code to write
DEPS file from NetRadiant itself.

This commit is only about reorganizing code a better way.

[dpkdir] restart VFS only for Map_Free() and Map_LoadFile(), refresh others

Since there is no map file observer anymore for that stuff,
we Restart or Refresh it explicitely when needed.

- Opening map or New map needs Restarting VFS.
- Saving map or Renaming map (Save as) needs Refreshing VFS.

The Refresh also triggers the Texture browser.

A previous fix for #105 was done Refreshing VFS instead of Restarting it
but it was not perfect, for example map opening behavior is better with
VFS restart, and map saving needs to refresh the VFS and the texture
browser but not restart (that was the earlier fault). So, it's clearly
not a good idea to trigger either Refresh or Restart on any map event
since the behavior differs according to the event.

If one day one plugin wants to act on VFS on map event, it must refresh or
restart VFS according to what the plugin does. It does not make sense to
trigger the same task for different events.