Update FLTK patches based on feeback from upstream

Protect the optional IM disabling calls with #ifdef:s

Use different keysyms for left and right alt on OS X

The server will get them confused otherwise if they are pressed
at the same time.

Disable input methods when the viewport is focused

Input methods are way too complex for us to map them to the VNC
protocol in any sane manner. Best just to disable them and rely
on simple keyboard behaviour when the viewport is active.

Ignore FLTK events when we have low level hooks

Some events can sneak through, e.g. from some virtual keyboards.
We need to ignore these to avoid confusion.

Add OS X keyboard handler

Add Windows keyboard handler

Add X11 keyboard handler

Add xhandler hook

Boiler plate code to intercept system events from FLTK so that
we can generate proper keyboard messages.

Add helper to get combining codepoint from a spacing character

Better codepoint for combining ypogegrammeni (dead iota)

Attempt to handle Ctrl+key

Hacky attempt to try to figure out the actual key pressed when
FLTK is giving us control codes.

Remove simple keyboard model

Remove the FLTK simple keyboard system and reorganise things in
preparation for a more direct approach.

Ship FLTK patches instead of referring a URI

It makes things more self contained (although you still need to
download FLTK itself), avoids the confusion of STR:s in varying
states, and allows us to refer to patches that aren't available
on an STR.

Update FLTK requirement

We actually require FLTK 1.3.2 as we reference FL_SOCKET.

Windows debug builds of vncviewer should have a console

Makes it easier to debug it by having convenient access to the
log output.

Use NoSymbol as the error code

This is how the X11 API does things, so it's clearer if we use
the same principle.

Make keysymtab static and const

Static to avoid namespace polution, and const so that it can be
shared among instances.

Use printf constructor safely

The input string might contain formatting marks, so make sure those
are ignored.

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

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

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

String literals are constant

Remove PixelTransformer and TransImageGetter

There is nothing using these classes anymore, so remove the clutter.

Move image encoding logic into a central EncodeManager class

This allows us to apply a lot more server logic
independently of which encoder is in use.

Most of this class are things moved over from the
Tight encoder.

Use PixelBuffer objects as the interface for encoders and decoders

This avoid a lot of unnecessary middle men. This also pushes the
responsibility for pixel format conversion into the encoders and
decoders. The new bufferFromBuffer() is used for direct conversion,
rather than PixelTransformer/TransImageGetter.

Store the mouse cursor in the ConnParams object

Like we do for everything else. This also gets rid
of the callback, which is a bit out of place
compared to everything else.

Allow PixelBuffers to be const

Add helper class for a rendered cursor

Add a magical cursor framebuffer class for handling when you
want to render the cursor on the server side. Keeps the cursor
specific magic in one contained place.

Add convenience variant of getImage()

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

Handle closing of input devices

The core can decide to close an input device (e.g. during a
server reset). Make sure we respect this by forgetting about
the device and recreating it again later.

Remove private storage from InputDevice

We don't need this anymore as we now only have a single
InputDevice object and can easily refer to it directly.

Make InputDevice into a singleton

Xorg input devices are global so it doesn't make sense that we
have one InputDevice object per XserverDesktop.

Move all functions that InputDevice uses into the class

Let XserverDesktop handle syncing cursor position

This makes InputDevice cleaner and just a glue layer for Xorg
input devices.

Re-init VNC extension on server reset

fc23895 blocked out a bit too much code when trying to make
libvnc handle server resets. Even though configuration should
only be handled once, the underlying VNC extension needs to
be reinitilised as the X core has removed all extensions on
reset.

We need to set a label on all pointer buttons

Add convenience functions to ModifiablePixelBuffer

Allows you to modify the buffer with data in a
different pixel format.

Test results comparing PixelTransformer and bufferFromBuffer()

Add optimised buffer conversion

Handles the common cases when the target or source are in
the preferred 888 format.

If one of the buffers is not 888, then it must also be properly
aligned (which is commonly the case). Performance is now in many
cases on par with PixelTransformer.

Add ability to directly convert between two pixel formats

This is a lot easier and cheaper than having to set up a complete
PixelTransformer object.

Use lookup tables for pixel upconversion

You have to loop many times when converting e.g. 2 bits to 8,
so have a couple of lookup tables to speed things up. We still
use bit replication to get from 8 bits to 16 bits though.

Put some space between the groups of tests

Add more 8 bpp conversion tests

Only grab the regions we actually need

Allows pixel buffers that do damage tracking to get
proper information.

Provide a better R/W base PixelBuffer class

Clearly separates the read API from the write API
and also from actual implementation.

Get rid of getStride()

It was confusing and not properly used everywhere.
Callers should use the stride they get when they get
the buffer pointer.

Get rid of the direct access abuse of FullFramePixelBuffer's data

We don't need a fillRect() that is this optimised

Keep things simple instead and allows us to remove the extra setPF()
methods.

Avoid having virtual methods where not needed

getImage() doesn't need to be overloadable, keep it simple

Create a proper interface base class for the viewport's framebuffer

This allows us to gracefully fall back to the FLTK code in case the
platform specific code cannot be used.

Encoders/decoders should track the connection object

The connection object is a much more appropriate object for the
decoders and encoders to keep track of. Besides the streams, it also
contains state like connection parameters.

Push encoder and decoder handling down into the connection objects

This keeps the reader and writer objects clean and simple protocol
decoders/encoders.

Remove full support for colour maps

Gets rid of a loooot of code and complexity.

Colour map clients are still supported through an
automatically generated map, but we lose the ability to
develop a client or server that uses colour maps
internally.

Get rid of unnecessary macros as they are only ever set to a single thing

Push encoding specific formats into the encoders and decoders

Keep the generic stream classes clean and general.

Consolidate the different palette handler implementations

Handle CopyRect like any other encoding

Avoids having to special case things. Keeps the code simpler.

Avoid reader/writer duplication in every encoder/decoder

Remove partial update functionality

Get rid of the ability of encoders to only send part of the
rectangle they are given as we don't want partial updates
anyway.

Move serialisation of pseudorects into separate functions

Keeps the code clearer.

Merge the "V3" message classes into the normal ones

We have no need for this abstraction so let's keep things simple.

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

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

Remove a lot of platform compatibilty stuff

It's either not used, or no longer relevant.

Remove remnants of Windows only client

Fix some offenders that poke around in the PixelFormat internals

Avoid code duplication for the linear pixel format conversion methods

Consistent use of stride vs pitch

Consistently use the term stride rather than pitch. Also
consistently represent the stride in number of pixels rather
than number of bytes. There is so much code that assumes
proper alignment already that we do not need the extra resolution.

Fix possible information leakage

Zero the padding byte used when converting 24-bit RGB to 32-bit.

Use the same algorithm for conversions

Make sure that the routines that convert between buffer
and rgb representations follow the same algorithm so it's
easier to compare them.

Optimise the RGB to pixel conversion to simple shifts

Might as well make it inline at this point as well as it is used
heavily in other routines that convert entire blocks of pixels.

Remove magic JPEG variables from ConnParams

Custom compression level and JPEG on/off are not inherent parts of
the protocol negotiation so they do not belong in ConnParams. Let the
UI frontend handle such things instead.

Clean up the encoding parser in ConnParams

Make the subsampling setting follow the common style

Remove some premature optimisation

Reduces header dependencies.

Add some missing whitespace

Stop storing a copy of the encodings as we don't use it for anything

Get rid of the register mechanism for encoders and decoders

We have no use for it, so let's keep things simple.

Be more consistent in referring to pixel byte streams as buffers

Document the different pixel representations that we deal with

Get rid of unused things in the ZRLE encoder

Get rid of unused shared MemOutStream and variable max length
functionality in the ZRLE encoder.

Remove unused method prototype

Remove all unused TX widgets

Remove the scaled pixel buffer classes as they are not used.

There were also reports of them being broken last time we still
made use of them.

Remove unused (and commented out) PixelBuffer::getPixel()

Remove unused alternative code path from the ZRLE and Hextile decoders

Remove unused pixel conversion method

Remove unsused, commented out code

Clean up header dependencies in the vncviewer directory

Reduce dependencies on TransImageGetter.h

Support keeping libjpeg's default compression setting

Check that we have a cursor to restore

Forgot to check that we actually have a cursor from the server
before trying to switch back to it after dismissing the context menu.

Converting to RGB might involve a precision increase

Need to do more than a simple shift to get the appropriate
new value. This isn't quite as exact as a proper multiplication
and division, but the error is so small it's not worth the
extra cycles.

Add pixel conversion speed test

Add a tests directory where we can put tests programs that can
be useful during testing. Start with a speed test for the pixel
conversion routines.

librfb needs librdr so make sure it always gets pulled in

Make sure we check that PixelFormats we get are actually valid.

Embedded applet should not show server dialog