[Spice-commits] Changes to 'wip'

Wed Mar 14 18:01:36 PDT 2012

New branch 'wip' available with the following commits:
commit 0da9c0d6ac4d709e00dd92b744cdd9f1bd1ca7de
Author: Marc-AndrÃ© Lureau <marcandre.lureau at gmail.com>
Date:   Wed Mar 14 23:22:46 2012 +0100

    Remove need for SPICE_CANVAS_INTERNAL
    
    Why is this useful?

commit e9e4f1587a057c127ca81c33f4db525feacc906a
Author: Marc-AndrÃ© Lureau <marcandre.lureau at gmail.com>
Date:   Wed Mar 14 23:07:15 2012 +0100

    fix void* arithmetic
    
    marshaller.c:528:50: error: pointer of type 'void *' used in arithmetic [-Werror=pointer-arith]

commit 98d7bacd7badc75f3baf38471c9aec46241813ad
Author: Marc-AndrÃ© Lureau <marcandre.lureau at gmail.com>
Date:   Wed Mar 14 22:40:53 2012 +0100

    build: replace INCLUDES with AM_CPPFLAGS
    
    Fix warning `INCLUDES' is the old name for `AM_CPPFLAGS' (or `*_CPPFLAGS')

commit 2adc40973c291bcdfde8f7ab0b6e6b70e49088eb
Author: Daniel P. Berrange <berrange at redhat.com>
Date:   Fri Feb 10 15:03:31 2012 +0000

    Add missing includes & make some functions static
    
    A number of functions were used without prior declaration. In
    some cases this was due to missing include files. In other cases
    the functions should have just been static.
    
    Ideally this would allow -Wmissing-declarations to be enabled, but
    the files generated by spice_codegen.py will still trip up on this.

commit bd12a6bd2fa32cffd98a2925c83f527f023128a3
Author: Daniel P. Berrange <berrange at redhat.com>
Date:   Fri Feb 10 14:30:56 2012 +0000

    Add printf format annotations to all '...' functions
    
    To allow the compile to detect incorrect printf formats, any
    var-args function should have a format annotation
    
    * common/macros.h: Helper to define ATTR_PRINTF for code
      which can't depend on glib
    * common/canvas_base.c, common/lz.h, common/macros.h: Annotate
      some var-args methods

commit 161e37ea9f20c1ffc26720d050cf4b833e63b1ba
Author: Daniel P. Berrange <berrange at redhat.com>
Date:   Fri Feb 10 14:05:27 2012 +0000

    Fix some integer range checks which always evaluate false
    
    There are some integer range checks which always evaluate false
    due to use of unsigned integer types. One of these would prevent
    detection of encoding errors from celt. The others are simply
    no-ops.
    
    * common/pixman_utils.c: SpiceROP is an enum & thus unsigned

commit d8dda8213ba0be75719a7edccdd0fdfa70f95ccb
Author: Daniel P. Berrange <berrange at redhat.com>
Date:   Fri Feb 10 13:56:04 2012 +0000

    Avoid warnings about empty conditional statement bodies
    
    Add extra {} braces around if/else statements which only
    call SPICE_DEBUG to avoid:
    
    ../common/ssl_verify.c: In function 'verify_pubkey':
    ../common/ssl_verify.c:87:50: warning: suggest braces around empty body in an 'else' statement [-Wempty-body]
    ../common/ssl_verify.c: In function 'verify_hostname':
    ../common/ssl_verify.c:254:53: warning: suggest braces around empty body in an 'if' statement [-Wempty-body]
    ../common/ssl_verify.c: In function 'verify_subject':
    ../common/ssl_verify.c:381:41: warning: suggest braces around empty body in an 'else' statement [-Wempty-body]

commit 9ed80dd6cfb6de8674b8f6475979385a5322d8e9
Author: Marc-AndrÃ© Lureau <marcandre.lureau at redhat.com>
Date:   Thu Dec 22 01:33:48 2011 +0100

    build: remove 'win' directory
    
    Let's use the only included file directly instead.

commit 3f962fd861a0c49d9da7a1af55705206f5c57a34
Author: Christophe Fergeau <cfergeau at redhat.com>
Date:   Thu May 5 16:39:23 2011 +0200

    messages.h: add smartcard bits

commit 349f70808b201c92b526655fe91be000f19c365d
Author: Marc-AndrÃ© Lureau <marcandre.lureau at gmail.com>
Date:   Wed Mar 14 21:09:31 2012 +0100

    Add client_marshallers.h client_demarshallers.h from spice-gtk
    
    spice-gtk uses a more up to date version of client/marshallers.h and client/demarshallers.h

commit 004a011b20b2993690c9b4fa31a8690284674120
Author: Marc-AndrÃ© Lureau <marcandre.lureau at gmail.com>
Date:   Wed Mar 14 21:01:48 2012 +0100

    Use SPICE_{BEGIN,END}_DECLS

commit 1074c36af2944ebc2f48c055401c2762d83d8859
Author: Christophe Fergeau <cfergeau at redhat.com>
Date:   Fri May 6 16:40:43 2011 +0200

    add const to arrays in marshalling functions

commit e22a9e9ebea2e79b8af9f72118ea924167c98cb4
Author: Christophe Fergeau <cfergeau at redhat.com>
Date:   Sat Apr 30 18:20:23 2011 +0200

    ssl_verify: include <string.h>
    
    ssl_verify.c is using memcmp which comes from string.h, this was
    breaking compilation with -Werror -Wall on Mac OS X

commit 4a076d7fbdf88140c4a3a822cd2e4e445788e0c7
Author: Marc-AndrÃ© Lureau <marcandre.lureau at gmail.com>
Date:   Wed Mar 14 20:50:52 2012 +0100

    ssl_verify: comment X509_NAME undef

commit 4a8383728a2b79156c5d2bfc9d74b6012219074e
Author: Marc-AndrÃ© Lureau <marcandre.lureau at redhat.com>
Date:   Sun Apr 3 15:49:36 2011 +0200

    Use a log handler to modify abort() behaviour
    
    Be more library friendly, by not aborting in library errors.
    
    spice_common now includes a proper log handler that will abort by
    default when reaching a warning.
    
    SPICE_ABORT_LEVEL can be changed to modify run-time abort level.
    
    SPICE_DEBUG_LEVEL can be changed to be more verbose. By default, only
    log level more importants than WARNING.
    
    Only memory-related functions are allowed to abort(), since they are
    not recoverable errors in the library.

commit 93a7da6789e193ad2902e3b54c6e79591a6dc5b7
Author: Marc-AndrÃ© Lureau <marcandre.lureau at gmail.com>
Date:   Wed Mar 14 18:01:38 2012 +0100

    update gitignore

commit bfc8a43b48aafd65a5ac7f22c05dd629f58a2cd6
Author: Marc-AndrÃ© Lureau <marcandre.lureau at redhat.com>
Date:   Sun Apr 3 02:29:14 2011 +0200

    common: remove unnecessary outdated c++ debug

commit bb4bf7b8e3ec14b1311d11eb3c4a40157c1e4856
Author: Marc-AndrÃ© Lureau <marcandre.lureau at redhat.com>
Date:   Mon Jan 24 21:09:52 2011 +0100

    common: ring.h should include stddef for NULL usage