Re: [PATCH 02/11] worker: Remove duplicate DISPLAY_CLIENT_TIMEOUT define

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

 



> 
> On Wed, Nov 11, 2015 at 2:10 PM, Fabiano Fidêncio <fidencio@xxxxxxxxxx>
> wrote:
> > On Wed, Nov 11, 2015 at 1:20 PM, Frediano Ziglio <fziglio@xxxxxxxxxx>
> > wrote:
> >> From: Jonathon Jongsma <jjongsma@xxxxxxxxxx>
> >>
> >> ---
> >>  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@xxxxxxxxxxxxxxxxxxxxx
> >> 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
_______________________________________________
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]