Fix indentation in XDesktop.cxx

It wasn't properly updated in 3112f50 when it was split off from
x0vncserver.cxx.

Unset pixel buffer when x0vncserver client disconnects.

In XDesktop::start() we allocate pixel buffer and set it as the backend to the given VNCServer.
In XDesktop::stop() we deallocate the buffer, so we must unset it from the VNCServer as well.
Otherwise the VNCServer could try to access it and crash, for example in deferred update.

VNCServerST: Allow unsetting pixel buffer.

Originally calling VNCServertST::setPixelBuffer(PixelBuffer* pb_) with pb_=0 would do nothing.
With this change pb will be set to 0 and deferred update timer will be stopped.

Fix rounding error in pixel down conversion

Simple shifting can give noticable rounding errors if there is a large
difference in the number of bits between the formats. Do the proper
thing via a lookup table, the same way things are done for up conversion.

Fix calculation of conversion test target values

Bit shifting gives an approximate, but not properly accurate value
when upconverting. Do a proper calculation.

Check for TrueColor support in x0vncserver

Colour map support was removed in b6b4dc6, but x0vncserver didn't
properly check if the X server required it.

Merge branch 'escape-opts' of https://github.com/olivergondza/tigervnc

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

Fix incorrect numbers in performance debug graph

Copy paste errors/brain fart:

 - Pixels do not need a factor 8 for any conversion

 - Multiplying bytes by 8 gives bits, so lower case 'b', and bits
   use SI prefixes, not IEC

Make sure auth argument is escaped

Update visible copyright year to 2018

Fix for RENDER hooks on Xorg < 1.12

TriFan and TriStrip were added in 1.12, so we need to have that code
conditional to support older versions of Xorg.

Add overlooked parameters to Xvnc and x0vncserver man pages

There were a bunch of parameters that weren't properly added to the
man pages when the features were added. Make sure everything is
properly in sync.

Sync x0vncserver and Xvnc man pages

They share a lot of settings, so make sure the documentation is
identical in those cases.

Remove DeferUpdate from man pages

The funcationality was removed in 6e49e95, so let's also clean up
the documentation referring to it.

Merge branch 'readme-rst' of https://github.com/x11clone/x11clone

Rename README.txt to README.rst and convert to reStructuredText

Update Spanish translation

Don't parse FLTK arguments using Fl::arg; handle -display as other options

Fl::arg accepts several arguments which we are not documenting. Also,
it accepts a -geometry argument, which clashes with our own -geometry
option.

Hook remaining XRender operations

We had overlooked a few so we didn't track all screen changes properly.

Update UI look on macOS

macOS no longer uses the old Aqua look, so tweak our things to fit
in better with the current state of things.

Restore menukey function to java viewer

Menukey was broken by recent refactoring of OptionsDialog

Fixes for erros in java hextile/zrle decoders

Various errors exposed when connecting to RealVNC servers
on alternative platforms (ARM, SPARC).  SSLEngineManager
was also cleaned up but most of the changes are cosmetic.

Use XShmGetImage for large reads

XShmGetImage is much faster. On my machine, this reduces the CPU usage
of a fullscreen Youtube video playback from 40% to 25%.

Use current server time for XUngrabPointer

Similar to 841e9f32d5952d6f16f67294c2da8bf395066a9b but for the
pointer. Fixes a problem where the desktop environment is unresponsive
(to the mouse) after fullscreen exit; Fixes #579.

My copyright was listed twice, fixed

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

Adjust dimensions for rotated monitors

We need to flip the dimensions in the case the monitor is rotated
90° or they won't map correctly to the framebuffer.

Make VNC extension startup mandatory

No longer tolerate loading the VNC extension but not being able to
initialize it. This avoids a lot of error prone checking to see
if the extension has started fully or not.

Call FatalError on fatal exceptions

Display security state when asking for password (java)

Move UserPasswordGetter out of CConn

Fix secure/insecure images

They were converted at the wrong resolution, and FLTK chokes on color
names so make sure all colors are specified in hex.

Update package dependencies for Ubuntu Trusty

Stop using CopyRect in WinVNC

It cannot keep itself in sync with the actual screen contents well
enough for CopyRect to work accurately. Graphical glitches could
be seen in some cases.

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

Add crude congestion window debug trace

Allows us to compare our computed congestion window with the
underlying one used by the TCP layer.

Add simple slow start to congestion control

Improved congestion control handling

Refine the previous method by interpolating the values we need.
This reduces the effect of the problem that we cannot send enough
ping packets.

Move congestion control to its own class

It's a general function and it's better to have that particular
complexity in its own place.

Reduce header dependencies in server classes

Use a queue for congestion pings

This reduces the data sent, and avoids any problems with the
client corrupting it.

Handle timers which should be executed right away

Remove double cursor workaround in client

We've fixed the issue more properly in the server now, so remove the
workaround we had in the client.

Fix bug where server cursor might disappear

We shouldn't unconditionally remove the server side cursor just
because the client supports client side cursors. We need to also
check that the client fulfills the other criteria for taking over
the cursor.

Avoid extra framebuffer update messages

In most cases we will send an update at the end of processing client
messages so these extra calls only slow things down. Server initiated
events still need an explicit call though.

Fix for initial client side cursor

Improve the tracking of what cursor we've sent to the client to make
sure the initial cursor is sent properly. We previously tried to infer
this information based on if we've rendered a server side cursor or
not. This logic broke down if things triggered before we've sent the
first update to the client.

Fix "int-in-bool-context" compile errors

Fix compile error on windows when not building with GnuTLS

Override default window manager key bindings better

Merge pull request #500 from bphinz/issue106-rebase

Hook up the "Load X.509 Cert/Key" dialogs in winvnc.  Fixes #106

Update Chinese (simplified) translation

avoid putting more includes in headers than necessary

Move gnutls functions into SSecurityTLS

Handle FLTK arguments at any position

E.g. vncviewer server.example.com:2 -display :2

Don't open X11 display too early

We have to wait until we've passed the command line arguments or
we might overlook a -display argument.

Fix build Java errors caused by UTF-8 characters in source comments

See issue #534

Raise minimum supported Java version to 7

Changes to keyboard handling require methods from the KeyEvent
class that were not introduced until Java 7.

Override implicit focus transfer when "F10" key is pressed

Java L&F has pre-defined hook that transfers the focus to the
menu of the RootPane.  Since the viewer doesn't have a menu
this just causes a loss of focus and subsequent unexpected
behavior of keyboard and mouse.

Refactoring and implementation changes to keyboard and mouse handling

See issue #505

Add formatted string function to LogWriter.

Need to be careful with escapes though.

Rename CMsgWriter functions to match C++ code

Handle Ctrl+Alt+Delete in raw keyboard mode

Ctrl+Alt+Delete requires special handling to trigger on Windows.
Make sure this works in raw keyboard mode as well.

Send proper keysym when Break is pressed

Ctrl+Pause sends break which results in a different VKEY on Windows.
Make sure we map that properly to the equivalent keysym.

Handle right shift better on Windows

The shift buttons are a bit buggy on Windows unfortunately, but we
can do slightly better than we're doing today.

Handle Windows scan code exceptions

Windows mostly follows the AT set 1 scan codes that we want, but
there are a few exceptions.

Move XDesktop specific parameters in to same file

Fully standardise on send()/recv()

We already assume sockets here since we use select().

Merge branch 'x0-xdesktop-initial-cursor' of https://github.com/x11clone/x11clone

x0vncserver does not support SetDesktopSize; remove -AcceptSetDesktopSize

Add a Configuration::removeParam to support such cases.

Configuration::listParams wants width/nameWidth; not ConfigurationObject

Removed IRIX/Solaris overlay support

Legacy systems.

Remove support for Lynx/OS

Added my/Cendio copyright where relevant

Set the correct cursor when starting

XFixes only reports cursor changes. We must manually call
XFixesGetCursorImage() when starting.

Use send() with MSG_DONTWAIT when possible

Define cork() as pure virtual in Socket class

This makes it possible to create a derived class from Socket which is
not TCP based, without having VNCSConnectionST.cxx trying to call
setsockopt() on a non-socket.

Move XDesktop to separate file

Define ledNames in XDesktop.cxx, rename N_LEDS to XDESKTOP_N_LEDS

Using namespace rfb in header files is a bad idea

Build with support for MIT-SHM

No need for ifdefs; we can assume MITSHM. Already done in vncviewer.

Avoid corner case of applications rendering at frame rate

There is a problematic corner case where an application is updating
at exactly the same rate that we're sending update. In that case we
may miss updates regularly, getting a very uneven final frame rate.
Avoid this by introducing a slight offset when we start updating.

Fix broken logic for sending mouse cursor

The logic got inverted resulting in the cursor being sent needlessly
whenever the cursor moved.

Update distro package requirements

Patch the el7 xorg source appropriately

Add debug trace when the clipboard changes

Don't validate timestamp on clipboard request

The normal Xorg server doesn't check this field, so neither can we
as there are applications relying on this leniancy.

Make exception classes have clearer messages

Include the type of exception in the string generated by each
subclass. Also simplify the constructs to what is needed.

Add Travis CI configuration

Improve test if library is a CMake target

Newer CMake complains when you try to get the property of something
that isn't a target so use the specific if operator instead.

Avoid getting LOCATION property

It's not supported in newer CMake, so try to deduce the library
filename from the target name.

Push query connect timeout back in to Xvnc

It was moved to the common code in f8e3b34c6, but it was unreliable
because the state could sometimes get out of sync. Push it back in
to Xvnc since it isn't necessarily something all servers will have.

Don't queue a query if it is directly rejected

Otherwise we'll wedge the desktop object which expects an approval
or rejection to eventually show up.

Keep alpha in cursor alpha conversion

Don't touch the existing value or we might mess it up when we avoid
division by zero.

Use current server time for XUngrabKeyboard

We might be calling this outside a normal FLTK event, so we cannot
trust fl_event_time to have a sane value.

Remove accidental debug output

Make sure to start the frame timer after a buffer change

The buffer is considered changed right away, so we might not get any
add_changed() calls to trigger a start of the timer.

Get rid of getFbSize()/getDesktopSize() in common

It was only used by WinVNC, so push it there instead.

Switch back to low resolution mode on macOS

This is a revert of 6c0181c. Testing showed that we got a big
performance hit by enabling this, and we also saw some graphical
artifacts. More work is needed before this can be enabled by default.

Update keyboard test notes for raw keyboard mode

Add keyboard test notes for LED state handling