Without this the menu won't emit the show signal the first time it's shown.

There is still 3 signed values implicitly converted to unsigned not
fixed, but they aren't really legitimate and would require a behavior
change in some corner cases.

Most noteworthy change is that all build commands IDs and groups are
now unsigned everywhere negative values aren't explicitly handled with
a special meaning.  This should not change anything in behavior, only
makes clear the index won't underflow.

These actions were marked as document sensitive but their sensitivity
is already managed by the navqueue, so making them document sensitive
only leads to them being wrongly updated when the document count
changes from or to 0.

So, make them initially insensitive and let the navqueue correctly
handle their later sensitiveness.

This also fixes a wrong style key name for filetypes.d.
(Nested comments aren't necessarily doc-comments).

Properly handle the case the saved geometry isn't valid, avoiding
a crash with corrupted preferences.

Instead of using the garbage values in memory for default size.
This fixes what was attempted to be fixed in the last two reverted commits.

This reverts commit 00473dff.

This reverts commit d090e7dd.

This seems required for GtkBuilder to be able to connect signals. However, this change may break Windows
builds. Needs to be tested and worked around if it is the case.

The only necessary bits were to remove interface.c and to install the Glade file.

Having C language snippets as global is wrong, moved them to several
language positions.  This files low rate of change and lack of ML
interest in putting language specific snippets in it suggests that
it isn't worth fixing the repitition or moving snippets to filetype
files.

Also remove old comment that doesn't apply anymore.

Fixes #3421282.
Occurred with Windows e.g. Vista, 7.

Fixes #3396195.

Note: GTK+ 2.24 is the current maintained version of the developer
files bundle:
http://www.gtk.org/download/win32.php

Conflicts:
	geany.glade
	src/Makefile.am
	src/callbacks.c

split "always wrap search and hide find dialog" pref into "always wrap search" and "hide find dialog"

Upstream Scintilla applied it as 3924:e2385dff420a.

Closes #3432877.

"Insert multiline comment" did insert single-line style comment when
available even if multi-line style was also supported for the filetype.

Add filetype_get_comment_open_close() to get appropriate start and end
comment markers from a filetype, and use it everywhere these markers
are fetched.

This fixes a crash in editor_insert_multiline_comment() if only single
comments are available (closes #3449635).

Although this doesn't change anything in practice, it started to be a
nightmare trying to deal with compilers warnings about "missing"
initializers, which are generally good but not in that file.

When checking whether a keybinding is already in use before updating
it in the preferences, check against the keybindings the user may have
updated, not the ones that are currently applied.

This fixes changing action A from key K to key L and action B from key
L to key K, which led to conflicts back and forth since when setting B
to K the check found A as a duplicate even after having changed it.

Use the global variable only when needed and add an initializer for it
in the default options.

Use a GCC pragma to disable -Wmissing-field-initializers in this file.
This works with both GCC and CLang, and should be harmless on other
compilers since unknown pragmas are supposed to be ignored.