On Wed, Oct 8, 2014 at 11:06 AM, Marc-André Lureau <marcandre.lureau@xxxxxxxxx> wrote:
On Mon, Oct 6, 2014 at 4:23 PM, Christophe Fergeau <cfergeau@xxxxxxxxxx> wrote:ACK
On Mon, Oct 06, 2014 at 01:52:43PM +0200, Fabiano Fidêncio wrote:
> SpiceChannelError, for now, will be used to set more detailed errors
> with respect to the main channel event SPICE_CHANNEL_ERROR_AUTH.
> ---
> Changes since v1:
> - Change the SpiceChannelAuthError comments to match with the enum values
> - Add more details related to SPICE_CHANNEL_ERROR_AUTH
> ---
> doc/reference/spice-gtk-sections.txt | 3 +++
> gtk/map-file | 2 ++
> gtk/spice-channel.c | 11 +++++++++++
> gtk/spice-channel.h | 25 +++++++++++++++++++++++++
> gtk/spice-glib-sym-file | 2 ++
> 5 files changed, 43 insertions(+)
>
> diff --git a/doc/reference/spice-gtk-sections.txt b/doc/reference/spice-gtk-sections.txt
> index caaa92c..5c411f8 100644
> --- a/doc/reference/spice-gtk-sections.txt
> +++ b/doc/reference/spice-gtk-sections.txt
> @@ -73,6 +73,9 @@ spice_main_clipboard_release
> spice_main_clipboard_notify
> spice_main_clipboard_request
> <SUBSECTION Standard>
> +SPICE_CHANNEL_ERROR
> +spice_channel_error_get_type
> +spice_channel_error_quark
> SPICE_MAIN_CHANNEL
> SPICE_IS_MAIN_CHANNEL
> SPICE_TYPE_MAIN_CHANNEL
> diff --git a/gtk/map-file b/gtk/map-file
> index 90f14f1..c1d9e71 100644
> --- a/gtk/map-file
> +++ b/gtk/map-file
> @@ -6,6 +6,8 @@ spice_audio_new;
> spice_channel_connect;
> spice_channel_destroy;
> spice_channel_disconnect;
> +spice_channel_error_get_type;
> +spice_channel_error_quark;
> spice_channel_event_get_type;
> spice_channel_get_error;
> spice_channel_get_type;
> diff --git a/gtk/spice-channel.c b/gtk/spice-channel.c
> index 5d1a86e..a8b4e35 100644
> --- a/gtk/spice-channel.c
> +++ b/gtk/spice-channel.c
> @@ -76,6 +76,17 @@ static void spice_channel_send_migration_handshake(SpiceChannel *channel);
>
> G_DEFINE_TYPE(SpiceChannel, spice_channel, G_TYPE_OBJECT);
>
> +/**
> + * SpiceChannelError:
> + *
> + * The error domain for the spice channel.
> + **/
> +GQuark
> +spice_channel_error_quark(void)
> +{
> + return g_quark_from_static_string ("spice-channel-error");
> +}
> +
> /* Properties */
> enum {
> PROP_0,
> diff --git a/gtk/spice-channel.h b/gtk/spice-channel.h
> index 1c303b4..adbe8f5 100644
> --- a/gtk/spice-channel.h
> +++ b/gtk/spice-channel.h
> @@ -32,6 +32,17 @@ G_BEGIN_DECLS
> #define SPICE_IS_CHANNEL_CLASS(klass) (G_TYPE_CHECK_CLASS_TYPE ((klass), SPICE_TYPE_CHANNEL))
> #define SPICE_CHANNEL_GET_CLASS(obj) (G_TYPE_INSTANCE_GET_CLASS ((obj), SPICE_TYPE_CHANNEL, SpiceChannelClass))
>
> +/**
> + * SPICE_CHANNEL_ERROR:
> + *
> + * Error domain for #SpiceChannel operations. Error in this domain will be
> + * from the SpiceChannelError enumeration. See #GError for more
> + * information on error domains.
> + **/
> +#define SPICE_CHANNEL_ERROR spice_channel_error_quark ()
> +SPICE_CHANNEL_ERROR_* is unfortunately already used in SpiceChannelEvent enum.
Yeah, and I'd love to get rid of them at some point, moving them to SPICE_CHANNEL_ERROR.
Why not add error values to SpiceClientError instead? This is the generic error domain for all spice-gtk I am not sure we need other domains.
Does not look easier to read/understand than the way it's now, but if you prefer, we can do in this way.
Best Regards,
-- Fabiano Fidêncio
_______________________________________________ Spice-devel mailing list Spice-devel@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/spice-devel