Re: [PATCH 4/9] gtk: move SpiceGrabSequence

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi

On Tue, Feb 16, 2016 at 5:26 PM, Fabiano Fidêncio <fidencio@xxxxxxxxxx> wrote:
> On Tue, Feb 16, 2016 at 3:49 PM, Marc-André Lureau
> <marcandre.lureau@xxxxxxxxx> wrote:
>> Do not leak internals of SpiceGrabSequence in our headers. This makes
>> also the class final, which let us extend more easily without fear of
>> breaking ABI.
>>
>> Signed-off-by: Marc-André Lureau <marcandre.lureau@xxxxxxxxx>
>> ---
>>  src/Makefile.am               |  1 +
>>  src/spice-grabsequence-priv.h | 28 ++++++++++++++++++++++++++++
>>  src/spice-grabsequence.c      |  1 +
>>  src/spice-grabsequence.h      | 13 -------------
>>  src/spice-widget.c            |  1 +
>>  5 files changed, 31 insertions(+), 13 deletions(-)
>>  create mode 100644 src/spice-grabsequence-priv.h
>>
>> diff --git a/src/Makefile.am b/src/Makefile.am
>> index e35c046..d6935cf 100644
>> --- a/src/Makefile.am
>> +++ b/src/Makefile.am
>> @@ -140,6 +140,7 @@ SPICE_GTK_SOURCES_COMMON =          \
>>         vncdisplaykeymap.h              \
>>         spice-grabsequence.c            \
>>         spice-grabsequence.h            \
>> +       spice-grabsequence-priv.h       \
>>         desktop-integration.c           \
>>         desktop-integration.h           \
>>         usb-device-widget.c             \
>> diff --git a/src/spice-grabsequence-priv.h b/src/spice-grabsequence-priv.h
>> new file mode 100644
>> index 0000000..81b4bf2
>> --- /dev/null
>> +++ b/src/spice-grabsequence-priv.h
>> @@ -0,0 +1,28 @@
>> +/* -*- Mode: C; c-basic-offset: 4; indent-tabs-mode: nil -*- */
>> +/*
>> +  Copyright (C) 2015 Red Hat, Inc.
>
> 2016 ?
>

ok

>> +
>> +  This library is free software; you can redistribute it and/or
>> +  modify it under the terms of the GNU Lesser General Public
>> +  License as published by the Free Software Foundation; either
>> +  version 2.1 of the License, or (at your option) any later version.
>> +
>> +  This library is distributed in the hope that it will be useful,
>> +  but WITHOUT ANY WARRANTY; without even the implied warranty of
>> +  MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
>> +  Lesser General Public License for more details.
>> +
>> +  You should have received a copy of the GNU Lesser General Public
>> +  License along with this library; if not, see <http://www.gnu.org/licenses/>.
>> +*/
>> +#ifndef __SPICE_GRABSEQUENCE_PRIV_H__
>> +#define __SPICE_GRABSEQUENCE_PRIV_H__
>> +
>> +#include <glib.h>
>> +
>> +struct _SpiceGrabSequence {
>> +    guint nkeysyms;
>> +    guint *keysyms;
>> +};
>> +
>> +#endif
>> diff --git a/src/spice-grabsequence.c b/src/spice-grabsequence.c
>> index a51d9e6..50e774d 100644
>> --- a/src/spice-grabsequence.c
>> +++ b/src/spice-grabsequence.c
>> @@ -24,6 +24,7 @@
>>  #include <gdk/gdk.h>
>>
>>  #include "spice-grabsequence.h"
>> +#include "spice-grabsequence-priv.h"
>>
>>  GType spice_grab_sequence_get_type(void)
>>  {
>> diff --git a/src/spice-grabsequence.h b/src/spice-grabsequence.h
>> index 28979ec..6e68e29 100644
>> --- a/src/spice-grabsequence.h
>> +++ b/src/spice-grabsequence.h
>> @@ -35,19 +35,6 @@ G_BEGIN_DECLS
>>
>>  typedef struct _SpiceGrabSequence SpiceGrabSequence;
>>
>> -/**
>> - * SpiceGrabSequence:
>> - *
>> - * The #SpiceGrabSequence struct is opaque and should not be accessed directly.
>> - */
>> -struct _SpiceGrabSequence {
>> -        /*< private >*/
>> -       guint nkeysyms;
>> -       guint *keysyms;
>> -
>> -       /* Do not add fields to this struct */
>> -};
>> -
>>  GType spice_grab_sequence_get_type(void);
>>
>>  SpiceGrabSequence *spice_grab_sequence_new(guint nkeysyms, guint *keysyms);
>> diff --git a/src/spice-widget.c b/src/spice-widget.c
>> index c3577a1..ef166cb 100644
>> --- a/src/spice-widget.c
>> +++ b/src/spice-widget.c
>> @@ -40,6 +40,7 @@
>>  #include "spice-widget-priv.h"
>>  #include "spice-gtk-session-priv.h"
>>  #include "vncdisplaykeymap.h"
>> +#include "spice-grabsequence-priv.h"
>>
>>  #include "glib-compat.h"
>>  #include "gtk-compat.h"
>> --
>> 2.5.0
>>
>> _______________________________________________
>> Spice-devel mailing list
>> Spice-devel@xxxxxxxxxxxxxxxxxxxxx
>> https://lists.freedesktop.org/mailman/listinfo/spice-devel



-- 
Marc-André Lureau
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel




[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]