Re: [PATCH spice-gtk] session: update spice_session_connect() doc

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

 



Hey,

On Fri, Aug 21, 2015 at 02:02:42PM +0200, Marc-André Lureau wrote:
> Update the documentation about the return value, and how to watch for
> connection success.
>
> Releated to:
> https://bugzilla.redhat.com/show_bug.cgi?id=1253848
> ---
>  gtk/spice-session.c | 6 +++++-
>  1 file changed, 5 insertions(+), 1 deletion(-)
>
> diff --git a/gtk/spice-session.c b/gtk/spice-session.c
> index 1a68d7d..234ff54 100644
> --- a/gtk/spice-session.c
> +++ b/gtk/spice-session.c
> @@ -1489,7 +1489,11 @@ SpiceSession *spice_session_new_from_session(SpiceSession *session)
>   * Open the session using the #SpiceSession:host and
>   * #SpiceSession:port.
>   *
> - * Returns: %FALSE if the connection failed.
> + * Returns: %FALSE if the session state is invalid for connection
> + * request. %TRUE if the connection is initiated. To know whether the
> + * connection is established, you must watch for channels creation
> + * (#SpiceSession::channel-new) and the channels state
> + * (#SpiceChannel::channel-event).
>   **/
>  gboolean spice_session_connect(SpiceSession *session)
>  {

Looks good for me.
  toso

> --
> 2.4.3
> 
> _______________________________________________
> Spice-devel mailing list
> Spice-devel@xxxxxxxxxxxxxxxxxxxxx
> http://lists.freedesktop.org/mailman/listinfo/spice-devel
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
http://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]