Signed-off-by: Alon Levy <alevy@redhat.com>

Signed-off-by: Dan McGee <dpmcgee@gmail.com>

Not all files were including config.h

The tests include strings.h but don't need any of its functions

Remove any blank lines at the end of all source files

The free() function allows NULL to be passed in, so any
code which puts a if() before free() is wasting time

Several functions in server/ were not specifying an argument list,
ie they were declared as void foo(); When compiling with
-Wstrict-prototypes, this leads to:
test_playback.c:93:5: erreur: function declaration isn’t a prototype
[-Werror=strict-prototypes]

create_test_primary_surface::test_display_base.c creates a
QXLDevSurfaceCreate structure and initialize it, but doesn't set
the position field. Moreover, this structure has 4 bytes of padding
to the end (as shown by pahole from dwarves), so initialize the whole
structure to 0 before using it.