Remove stray comma for XRandR detection

Fix a stray comma interpolating into the configure script as an action to
run, causing it to print harmless errors.

Noticed by Hendrik Jaeger.
Centralise check for global monitor

Let montor_should_ignore_global() check for the presence of the global monitor,
plus other configured monitors, and shift the check done in FScreenGetScrRect()
to use that as well.
Redefine how the "global" monitor is stored

Rather than making the "global" monitor part of the monitor queue in the case
where XRandR is in use, instead only add the global monitor to the queue if
XRandR isn't in use (and hence there's only ever one monitor).

This means the callers don't need to worry, when querying monitors, about the
global monitor since contextually, querying monitors is always going to be
correct.  If the global monitor is required, that can be queried in the same way
it is now, and hence those existing callers don't need to change.

In making this change, initialise the monitors when required, rather than based
on whether they're in the main monitor_q or not; meaning no special-casing of
the global monitor is needed to support initialisation.
monitor_by_name: handle global_monitor

Now that the global monitor is separate depending if XRandR is used, when
querying for the global monitor, make sure we handle this case.
Remove callers of monitor_should_ignore_global()

This is now handled internally, and hence there's no need for callers to
understand this.
Make monitor_should_ignore_global() static

This function doesn't need to be public anymore; mark is as static.
Deprecate FindScreenOfXY()

Move the functionality to monitor_by_xy() instead, and switch over the
callers of FindScreenOfXY to it.  Reduces the need for a transient function
serving no purpose.
FScreen.c:  Few style cleanups

* Reduce spacing between operators;
* Remove unnecessary else blocks by pre-initialising;
* Some reindentation, etc.
Remove FScreenConfigureModule / FScreenGetConfiguration

Modules relying on monitor-specific information will be configured locally, and
module packets sent to modules will instead receive the monitor such events
occurred on.
Officially deprecate Xinerama / XineramaPrimaryScreen

These no longer make sense with XRandR; may well be resurrected in some other
guise later on.

For now the commands exist as stubs only, with a deprecation warning present
until such time that we deal with configuration migration properly.
Introduce monitor_by_number()

Rather than internal APIs relying on the name of monitors when looking up
various things about them, instead assign each monitor an ID which can be used
internally.

This will also make things easier in the longer-term when trying to streamline
how RandR is going to be used, but for now this will make more sense in terms of
communicating monitor information with modules.
CONFIGARGS:  Include the monitor number

Augment the CONFIGARGS macro to also include the monitor number of the
configured window.  This will allow modules to easily ascertain monitor
information without the need to mangle a pointer to some struct montor *.
Ident:  Print monitor name selected window is on

When using Ident to show information about a window, also print out the name
of the monitor.
TODO:  Add item about monitor API