Re: [Spice-devel] [PATCH spice-server 10/10] tests: Avoid to disable all deprecation warnings just for expect functions

2017-09-12 Thread Christophe Fergeau
On Tue, Sep 12, 2017 at 03:48:06AM -0400, Frediano Ziglio wrote: > > > +static inline void > > > +g_test_assert_expected_messages_internal_no_warnings(const char *domain, > > > + const char *file, > > > int > > > line, const char *func) > > > +{

Re: [Spice-devel] [PATCH spice-server 10/10] tests: Avoid to disable all deprecation warnings just for expect functions

2017-09-12 Thread Frediano Ziglio
> > On Mon, Sep 11, 2017 at 11:15:47AM +0100, Frediano Ziglio wrote: > > In case GLib don't provide these functions we use replacements so > > there's no need to have a warning if these functions are called. > > This potentially capture other compatibility issues in the tests > > that would be

Re: [Spice-devel] [PATCH spice-server 10/10] tests: Avoid to disable all deprecation warnings just for expect functions

2017-09-11 Thread Christophe Fergeau
On Mon, Sep 11, 2017 at 11:15:47AM +0100, Frediano Ziglio wrote: > In case GLib don't provide these functions we use replacements so > there's no need to have a warning if these functions are called. > This potentially capture other compatibility issues in the tests > that would be ignored having

[Spice-devel] [PATCH spice-server 10/10] tests: Avoid to disable all deprecation warnings just for expect functions

2017-09-11 Thread Frediano Ziglio
In case GLib don't provide these functions we use replacements so there's no need to have a warning if these functions are called. This potentially capture other compatibility issues in the tests that would be ignored having all deprecation warnings disabled. Tested with GLib 2.28 and 2.52.