[Spice-devel] [PATCH 02/11] worker: Remove duplicate DISPLAY_CLIENT_TIMEOUT define

Frediano Ziglio fziglio at redhat.com
Wed Nov 11 07:50:24 PST 2015


> 
> On Wed, Nov 11, 2015 at 2:10 PM, Fabiano FidĂȘncio <fidencio at redhat.com>
> wrote:
> > On Wed, Nov 11, 2015 at 1:20 PM, Frediano Ziglio <fziglio at redhat.com>
> > wrote:
> >> From: Jonathon Jongsma <jjongsma at redhat.com>
> >>
> >> ---
> >>  server/red_worker.c | 1 -
> >>  1 file changed, 1 deletion(-)
> >>
> >> diff --git a/server/red_worker.c b/server/red_worker.c
> >> index 4d430ad..3a33596 100644
> >> --- a/server/red_worker.c
> >> +++ b/server/red_worker.c
> >> @@ -75,7 +75,6 @@
> >>  #define CMD_RING_POLL_RETRIES 200
> >>
> >>  #define DISPLAY_CLIENT_SHORT_TIMEOUT 15000000000ULL //nano
> >> -#define DISPLAY_CLIENT_TIMEOUT 30000000000ULL //nano
> >>  #define DISPLAY_CLIENT_MIGRATE_DATA_TIMEOUT 10000000000ULL //nano, 10 sec
> >>  #define DISPLAY_CLIENT_RETRY_INTERVAL 10000 //micro
> >>
> >> --
> >> 2.4.3
> >>
> >> _______________________________________________
> >> Spice-devel mailing list
> >> Spice-devel at lists.freedesktop.org
> >> http://lists.freedesktop.org/mailman/listinfo/spice-devel
> >
> > I've made some comments in the original patch:
> > http://lists.freedesktop.org/archives/spice-devel/2015-November/023448.html
> 
> As pointed by Pavel, it's used in another .c file. So removing it from
> the .h is a no way to go.
> ACK!
> 

Merged

Frediano


More information about the Spice-devel mailing list