Fix session resize after mirroring on Linux vncviewer

If monitor mirroring was enabled while in a session with vncviewer
running on Linux, the session would not be properly resized on the
server. This was a consequence of only looking at the size and
coordinates of each screen when matching against existing screens after
the screen layout was changed, when in fact we have two (or more)
monitors with the same coordinates and size (but differing ids). This
led to the same monitor being added twice to the layout which would
later fail layout validation, resulting in no resize request being sent
to the server.

When matching, we now also check if the existing screen is not already
present in the layout before considering it a match.

Fix handling of VMware cursors

This is a regression from ad0f061. If a VMware cursor rect was split up
over multiple read()s then the stream would become corrupted as we set
the restore point at the wrong place.

Merge branch 'selinux-restorecon' of https://github.com/grulja/tigervnc

SELinux: restore SELinux context in case of different policies

Improve full-screen workaround on connect

If we've explicitly requested a position (using -geometry) then we were
still triggering legacy full-screen request logic. This is very counter
intuitive as having a position that doesn't align with a monitor should
disqualify us for that.

It turns out that at least marco (and probably everything
metacity-derived) ends up ignoring the requested position and we go back
to triggering the legacy logic again.

Work around this by assuming that the window manager will adjust our
initial position and don't consider that in the workaround logic.

Merge pull request #1405 from TigerVNC/monitorArrangementSync

Update MonitorArrangement widget to match parameter settings after screen configuration change

Update MonitorArrangement widget after change

If the options dialog was open when a screen configuration happened the
widget could get out of sync from the settings. A scenario when this
happened was:

 1) 3 monitors, fullscreen selected on the two right-most screens
 2) disconnect the left-most screen (the one not selected)

In this case, using GNOME, vncviewer would appear in fullscreen on the
right of the two remaining monitors, but the widget would show both
monitors selected. The reason was that the MonitorArragement index
doesn't work the same way as FLTK's screen index.

It's debatable how vncviewer should behave here, but the GUI should at
least match the actual setting.

Properly set XDG_CURRENT_DESKTOP

There is no such thing as XDG_DESKTOP_NAMES.

Update copyright year to 2022

Don't add . to error message string

The norm is to have error strings without punctiation, so avoid adding
one as it just confuses the translators.

Avoid creating windows larger than current monitor

Many window managers don't have proper logic to restrict the window size
to something sane, so we have to take care of that ourselves.

Don't move window in full-screen workaround

If this is done before the window is shown then it can accidentally
override the window manager's placement of the window. Reducing the size
should be sufficient to avoid trigger a full-screen request.

Be consistent in including config.h

The generally recommended way is to include it from source files, not
headers. We had a mix of both. Let's try to be consistent and follow the
recommended way.

Only restore level if it is wrong

Tweak the workaround from 9023a2e so that it only triggers if there
actually is a problem to fix. This is important since changing levels
also has the side effect of raising the window to the top of the level.

Relicense MonitorArrangement widget

This was designed as a general FLTK widget and we want to encourage
others to use it and improve it. So relicense these particular files in
a way that allows it to be used by anyone using FLTK.

As part of this we need to stop using other parts of TigerVNC as that
code is still covered by the GPL.

Avoid leaking overlay memory

Fl_RGB_Image doesn't take ownership of the pixel data, so we have to
explicitly free this as well once we are done.

Handle empty lines in server history

Fix typo in mirror monitor detection

Bug introduced in fb561eb but still somehow passed manual testing.
Resulted in some stray reads off the end of the stack, which were
hopefully harmless.

Be more paranoid about monitor info

These function are not guaranteed to succeed, and we've had reports that
they fail on ARM. So make sure we more properly check return values.

Update Serbian translation

Merge branch 'display' of https://github.com/CendioOssman/tigervnc

Add headings/groups for options

Try to be more consistent that related options should have a group box
around them.

Rename "Screen" tab to "Display"

This is what pretty much everyone else calls these kind of settings, so
let's change so users feel familiar.

Simplify screen settings

Try to focus on the commonly used settings here in order to keep things
understandable. The uncommon settings will need to be specified as
command line arguments instead.

Add comments to stored parameters list

Clarify what this list is used for, and organize it so it is easier to
match to the options dialog.

Document raised CMake requirement

We raised the requirement in cb3c78f but forgot to update the
documentation.

xserver21.1.1.patch: Add Xorg 21 patch

xvnc: adapt for 1.21

VENDOR_STRING is gone as is DEFAULT_LOG_VERBOSITY
site.h has been removed and is not needed.

xorg-version.h: Increase supported Xorg version to 1.21

Merge pull request #1382 from williamsjoblom/master

Be consistent in "full screen" and "full-screen" use

Be consistent in "full screen" and "full-screen" use

Previously, there were a number of inconsistencies in the use of
"full-screen" and "full screen" in logs, documentation, and user
interfaces. The consensus seems to be that "full screen" is correct when
used as a noun while "full-screen" is correct when used as an
adjective. These inconsistencies have now been adjusted to follow
consensus.

Merge pull request #1380 from williamsjoblom/master

Deduplicate vncviewer error messages

Deduplicate vncviewer error messages

vncviewer previously had a the same localized error message duplicated
on a bunch of places. Pull these duplicates out into a single function.

Fix formatting of -rfbport in man pages

Update French translation

Merge branch 'cmake' of https://github.com/CendioOssman/tigervnc

Raise CMake requirement to 3.10

It's difficult to support both old and new versions, so raise the
requirement to the oldest that is commonly used, which is CMake 3.10.2
on Ubuntu 18.04.

Don't specify SOURCE in custom targets

They don't really contribute to anything as the command line is fixed,
and it makes CMake 3.20+ upset since it cannot find that file and
instead thinks we meant maketarball.in (CMP0115).

Drop Ubuntu 16.04 from builds

This version of Ubuntu is no longer fully supported so let's stop caring
about builds there.

Log decoding stats on disconnect

Can be helpful to see what encodings were actually used during a
connection, and how they performed.

Increase maximum input buffer

macOS' built in VNC server unfortunately sends the entire monitor in a
single rect, so we need to be prepared to buffer a lot of data in case
the monitor has a large resolution.

Throw more descriptive error on rect read error

Gives a bit more context where this error happened.

Remove single cpu decoder shortcut

Single CPU machines are extremely rare now, so let's avoid the hassle of
multiple code paths.

Update Swedish translation

Fix closing reconnect dialog logic

If you close the reconnect dialog then we should assume that you want to
close things, not that you want to reconnect. Unfortunately FLTK returns
the first choice when closing the dialog, so that needs to be the
"Cancel" alternative.

Merge branch 'master' of https://github.com/twhitehead/tigervnc

Don't fail on user CA/CRL problems

The certificates might still be fine using the system trust store, or
the user can make an exception. So let's just log and continue on.

This got very common after 960c7d2 where we now always have a default
value for these settings.

System certificate load returns number loaded not GNUTLS_E_SUCCESS

Merge pull request #1349 from samhed/fullscreenLevel

Workaround FLTK bug that causes OptionsDialog to appear behind DesktopWindow

Workaround options window appearing behind viewer

FLTK's fullscreen_x() function will always put the window on a high
level (NSStatusWindowLevel = 25), even if the window doesn't have focus.
This causes the OptionsDialog to end up behind the DesktopWindow when
the fullscreen function is called. Until we can rest assured that most
people building TigerVNC will use a fixed version of FLTK, we will need
this workaround.

Merge pull request #1350 from samhed/serverdialogmodal

Remove modal property from ServerDialog

Merge branch 'master' of https://github.com/johahauf/tigervnc

Remove modal property from ServerDialog

Modal windows are meant to be child windows such as menus or popups that
are intended to always appear in front of its parent.

Modal FLTK windows without a parent seem to behave strangely on macOS.
Such windows can appear in front of their modal children windows.
When opening OptionsDialog from ServerDialog, it was easy to get to a
broken state by then switching focus back to the ServerDialog. This
would cause the ServerDialog to appear in front, but not be useable as
the options window would still have control of mouse and keyboard.

Additionally, modal windows without parents appear in front of other
normal applications on macOS. This means that the ServerDialog couldn't
be put behind Safari for example.

The fact that ServerDialog was a "modal" window was a bit strange,
seeing as it doesn't have any parent windows. Removing the modal
property from this window causes all the strange-ness to go away.

Merge pull request #1344 from samhed/grabOnFocus

Only grab keyboard when we have focus

Only grab keyboard when we have focus

There was an issue with the new grabs that were introduced in commit
2289342de7222e4ff2483eae95bb9898830e6097. On macOS when running
vncviewer in fullscreen on one monitor, we saw that we sometimes got
resize and fullscreen events when switching the focus away from
vncviewer. With the changes in the commit mentioned above this meant
that we immediately grabbed the keyboard again, thus never releasing it.

Fixes #1332.

Create helper function for grabKeyboard conditions

Adjust default directory for configuration file save and load

Update Korean translation

Update Russian translation

Update Swedish translation

Update Chinese (traditional) translation

Update Finnish translation

Update Czech translation

Update Bulgarian translation

Test Java builds in github actions

Update Hebrew translation

Update German translation

Update Ukrainian translation

Update Indonesian translation

Give all monitor names when mirrored

If a monitor is part of a set of mirrored monitors, then give the name
of all used monitors as we don't know which one the user will most
strongly associate with what they see.

More user friendly monitor name on Windows

The device name is very internal and likely of little use to a user.
Instead give the more user friendly string we can get out of the system.

Use std::string for monitor names

Makes it a lot easier to deal with the allocation of memory for the
string, avoiding any limits.

Inline the Windows monitor name code

It's just used in this one place, and isn't very large. Also move things
around a bit to avoid having to define new complex types.

Fix corner case in monitor index calculation

Fix a bug in the comparison function that could result in the wrong
index being assigned to a monitor. Change the logic to more directly map
to the description.

Handle mirrored monitors on X11

macOS and Windows present mirrored monitors as a single virtual monitor,
but X11 exposes this scenario as two distinct monitors with identical
coordinates. This messes up our logic, and is likely confusing for the
user. So instead we'll ignore any monitors that have identical
coordinates to any already seen monitors.

Change development version to 1.12.80

Update translation template file

Clean up some error strings

Try to reuse and split things so that it makes things easier for the
translators.

Consistently use exceptions for parameter errors

Clean up the structure around storing and loading parameters and server
history so that failures will always be reported by using exceptions.

Use classic FILE I/O for storing server history

Error handling is more straight forward in the C file interface.

Merge pull request #1253 from binary-manu/accented-chars

Handle keys reported only by KeyEvent.getExtendedKeyCode

Clean up global event handler structure

Let's separate the standard global event handling from the hacky
workarounds, for clarity.

Restore FLTK event dispatcher on exit

Merge branch 'fullscreens-added-removed' of https://github.com/x11clone/x11clone

Tolerate carriage return in config files

If the file has been transferred from a Windows system then it might
have \r\n line endings instead of \n. Make sure this doesn't leak in to
the parameter values.

Fix inconsistent line endings in vncviewer configuration file

Include translations in macOS and Windows builds

Disable dangerous macOS SDK macros

They define macros such as "check()" which causes our code to break as
we have methods with that name.

Merge branch 'fb-session-nfs' of https://github.com/zpytela/tigervnc

Expose reconnect setting in options dialog

Retry connection in case of an error

Merge branch 'x0-interface-option' of https://github.com/sotho/tigervnc

Add -interface option to x0vncserver

Merge branches 'host-name-max', 'fix-build-viewer-off' and 'fix-case-insensitive-fs' of https://github.com/arichardson/tigervnc

Keep ownership of second selection when first is lost

This fixes regression introduced by the extended clipboard extension.
Previously it was possible for the server to hold on to the CLIPBOARD
selection even if another application took ownership of PRIMARY. This is
important to handle the common use case of selecting something in order
to paste over it.

The new request based model doesn't readily support this as we assume
the client has lost its data once we push the new PRIMARY selection to
it. So to handle this we have the maintain a cache of the client's data,
and make sure to fill that cache before we do anything that might cause
the client to lose the data.

Simply screen refresh handler

We only need to check the event type once, and preferably early, and
avoid doing more work than necessary.

Only log error when setting FullScreenSelectedMonitors

We don't want the log to spam every time this parameter is used so only
complain from the parses when the value is changed.

Don't reference specific variable in class

This class is generic and should not reference a specific instance of
it.

Avoiding final period in log messages

Try to be consistent with existing log messages.

Clean up style in recent monitor code

Fix some indentation issues and make sure everything* uses CamelCase.

* MonitorArrangement is left as snake_case in order to pretend it is an
  independent FLTK widget class.