cleanup unnecessary code per comments

keeping the old callback and adding the new one at the end of the struct, so that it can be potentially binary compatible.

Fix server side code to be aware of rereadEvent

Merge pull request #3560 from akallabeth/gfx_10_2

Gfx 10.2 caps and deactivate GDI unsupported functions

Merge pull request #3364 from hardening/limit_frames

Limit the time passed in transport check loop

Added reference to capability specifications

Merge pull request #3571 from akallabeth/clang_fixes

Clang fixes

Merge pull request #3554 from akallabeth/color_refactor

Replaced _VF formats with copy flags.

directly using FREERDP_PIXEL_FORMAT.

Fixed NULL dereferences.

Fixed uninitialized return.

Fixed missing NULL checks.

Fixed NULL pointer dereferences.

Merge pull request #3570 from rjcorrig/tz201610

winpr: Updates time zone data to October 2016

Updates time zone data to October 2016

Merge pull request #3568 from hardening/wayland_fixes

Implement callbacks for version 5 of wl_pointer

Merge pull request #3569 from akallabeth/gdi_resize_fix

Fixed invalid arguments for gdi_resize_ex

Fixed invalid arguments for gdi_resize_ex

Added a command line parameter to set the time passed in packet loop

Implement callbacks for version 5 of wl_pointer

This should prevent crash with compositor that implement this version.

Merge pull request #3567 from mfleisz/file_mode_fix

crypto: Remove unneeded update (+) flag from file open mode

crypto: Remove unneeded update (+) flag from file open mode

Updated symbol visibility of GFX channel.

Deactivated unsupported operations.

GFX: support 10.2 stack.

Applied astyle formatting

Updated the astyle formatting

Release the reread event on transport_free

Took in account comment remarks

test-connect: use a random port instead of always 3389

Doing so we have less chances to have a failure when we run 2 tests in parallel
on the same host (as sfrerdp-server listen on a socket).

sample-server: add a "local only" option to just listen on an UNIX socket

This makes things nicer for tests that don't need internet access.

Limit the time passed in transport check loop

This patch make it possible to limit the time that is passed when we call
XXX_check_fds functions. This should smooth the treatment between handling inputs
and handling incoming bitmap updates.
The default maximum time is set to 100 ms.

Adjusted non X11 implementations.

Replaced _VF formats with copy flags.

Merge pull request #3497 from realjiangms/fix_gfx_win10

server/shadow: Fix black screen with win10 mstsc

Merge pull request #3544 from akallabeth/multi_warn_fixes

Fix multiple warnings

Adapt makefile specific file paths

Try to use relative file paths only when building a release (build types
Release and RelWithDebInfo) with the Unix Makefile generator.

Before it was tried to do this with all generators causing multiple
problems.

Removed unused variable.

Fixed uninitialized variable.

Fixed uninitialized variable.

Fixed uninitialized variables.

Fixed uninitialized arguments, WLog tag.

Properly escaping __FILE__ replacement.

Merge pull request #3545 from akallabeth/legacy_rfx_fixes

Fixed Windows 7 RFX issues.

Merge pull request #3537 from akallabeth/gdi_speedup

Inlined heavily used functions.

Reverted NSC format, decoded data is always BGRA!

Merge pull request #3547 from realjiangms/fix_after_color_conversion_nsc_server

Fix nsc server side after color conversion.

Color format for surface bits unified.

Fixed offset and stride for X11 surface bits.

Respecting tile offsets.

Fixed format flip and offsets for GDI.

Fix nsc server side after color conversion.
We use new color format system and PIXEL_FORMAT_BGRX32 is the unified buffer color format. Add support for PIXEL_FORMAT_BGRX32 in nsc

Merge pull request #3538 from akallabeth/mac_server_fix

Mac shadow server regression fix

Optimized ROP for GDI_SRCCOPY and GDI_DSTCOPY.

Optimized gdi_FillRect

Inlined heavily used function.

Inlined heavily used functions.

Fixed Windows 7 RFX issues.

Merge pull request #3470 from untoldone/master

mac BUILD_SHARED_LIBS=ON fix (on master)

Merge pull request #3531 from rjcorrig/macdialogfix

client/Mac: Runs Mac password dialog on main UI thread

Merge pull request #3536 from akallabeth/memory_leak_fixes

Memory leak fixes

Merge pull request #3540 from realjiangms/fix_after_color_conversion_gfx_h264

Regression fix: Fix gfx h264 server

Merge pull request #3541 from realjiangms/fix_after_color_conversion_nsc

Fix nsc client after after color conversion huge fix

server/shadow: Fix black screen with win10
Request full screen update on RDPGFX_CAPS_ADVERTISE_PDU. Win10 client seems to clean the screen after gfx channel opened. If there happens to be no screen update from server, we will get black screen in mstsc client.

Fix nsc client after after color conversion huge fix
nsc codec client doesn't work:
1. We should mark invalid region for software gdi
2. Checked the code before color conversion fix, the correct color format should be PIXEL_FORMAT_BGRX32_VF (corresponds to old PIXEL_FORMAT_XRGB32_VF)
3. For gdi:hw: xfc->bitmap_buffer is never used/initialized. However gdi->primary_buffer is always maintained. So use primary_buffer to hold the decoded bitmap data

Regression fix: The usage of RDPGFX_SURFACE_COMMAND::format is changed after color conversion huge fix.
Fix server side rdpgfx code to match the change.

Merge pull request #3533 from akallabeth/glyph_cache_disable

Glyph cache disable

Fixed memory leak in certificate check.

Merge pull request #3529 from akallabeth/warning_fixes

Variadic macro warning fixes

Reformatted file.

Fixed regression from rebase.

Removed unused function.

Merge pull request #3534 from akallabeth/x11_pointer_color

Fixed pointer color mapping.

Merge pull request #3535 from akallabeth/leak_check

Added leak detector option, made conflicting options exclusive.

Fixed leak in bitmap handling.

Fixed memory leak in pointer free.

Added leak detector option, made conflicting options exclusive.

Fixed pointer color mapping.

Deactivated glyph cache.

Reformatted file.

Runs Mac password dialog on main UI thread

Using C99 compatible variadic macros.

Fixed comma at end of enum

Fixed function argument and variable types.

Fixed invalid const type.

Thread using proper function pointer type.

Addin loader using proper function pointer defines.

WLog using C99 compatible variadic macros.

Merge pull request #3524 from bmiklautz/cleanup_3381

winpr/wait: add missing parameters

winpr/wait: add missing parameters

Add the missing parameters introduced with PR #3381

Merge pull request #3500 from akallabeth/color_conversion_fix_v6

Color conversion fix v6

Fixed GDI_RGN->null initialization.

Refactored ClipboardSetData.

Fixed memory leak.

Fixed use after free.

Added documentation, removed unused function.

Added color format documentation.

Fixed uninitialized value.

Unified RFX server color format.

Fix compiler warnings

clang version 3.9.1-svn281634-1~exp1 (branches/release_39)

freerdp-firerds/libfreerdp/core/channels.c:270:9: warning: returning
      'const WtsApiFunctionTable *' (aka 'const struct _WtsApiFunctionTable *') from a function with
      result type 'PWtsApiFunctionTable' (aka 'struct _WtsApiFunctionTable *') discards qualifiers
      [-Wincompatible-pointer-types-discards-qualifiers]
        return &FreeRDP_WtsApiFunctionTable;
               ^~~~~~~~~~~~~~~~~~~~~~~~~~~~

uwac/libuwac/uwac-display.c:502:12: warning: comparison of
      unsigned enum expression < 0 is always false [-Wtautological-compare]
        if (error < 0 || error >= UWAC_ERROR_LAST)
            ~~~~~ ^ ~