[Spice-devel] [PATCH v2 07/19] Return invalid value from very old obsolete functions
Frediano Ziglio
fziglio at redhat.com
Wed Nov 30 12:43:56 UTC 2016
>
> Hi,
>
> Why not return -1 in spice_server_get_sock_info() and
> spice_server_get_peer_info() and remove these main channel functions.
>
Next commit remove this function.
> Also the public function should be marked as deprecated if their usage
> should be avoided
>
They already are.
> Pavel
>
Frediano
> On Fri, 2016-11-25 at 14:52 +0000, Frediano Ziglio wrote:
> > These functions are not used since years and are not supporting
> > multiple clients.
> >
> > Signed-off-by: Frediano Ziglio <fziglio at redhat.com>
> > ---
> > server/main-channel.c | 8 ++++----
> > 1 file changed, 4 insertions(+), 4 deletions(-)
> >
> > diff --git a/server/main-channel.c b/server/main-channel.c
> > index 37b2f25..3d107c4 100644
> > --- a/server/main-channel.c
> > +++ b/server/main-channel.c
> > @@ -306,16 +306,16 @@ MainChannelClient
> > *main_channel_link(MainChannel *channel, RedClient *client,
> > return mcc;
> > }
> >
> > +/* very obsolete and old function, retain only for ABI */
> > int main_channel_getsockname(MainChannel *main_chan, struct
> > sockaddr *sa, socklen_t *salen)
> > {
> > - return main_chan ?
> > - getsockname(red_channel_get_first_socket(RED_CHANNEL(main_c
> > han)), sa, salen) : -1;
> > + return -1;
> > }
> >
> > +/* very obsolete and old function, retain only for ABI */
> > int main_channel_getpeername(MainChannel *main_chan, struct
> > sockaddr *sa, socklen_t *salen)
> > {
> > - return main_chan ?
> > - getpeername(red_channel_get_first_socket(RED_CHANNEL(main_c
> > han)), sa, salen) : -1;
> > + return -1;
> > }
> >
> > MainChannel* main_channel_new(RedsState *reds)
>
More information about the Spice-devel
mailing list