Re: [PATCH 4/5] 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:09 PM, Pavel Grunt <pgrunt@xxxxxxxxxx> wrote:
> On Wed, 2015-11-11 at 00:37 +0100, Fabiano Fidêncio wrote:
>> On Tue, Nov 10, 2015 at 9:41 PM, Jonathon Jongsma <jjongsma@xxxxxxxxxx> wrote:
>> > ---
>> >  server/red_worker.c | 1 -
>> >  1 file changed, 1 deletion(-)
>> >
>> > diff --git a/server/red_worker.c b/server/red_worker.c
>> > index dd4633c..8a2ff0e 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
>>
>> As DISPLAY_CLIENT_TIMEOUT is only used in red_worker.c I would keep
>> the define there and remove the one from server/red_worker.h
> It is also used in server/cursor-channel.c

Oh, I missed that one. ACK then!

>
> Pavel
>
>> _______________________________________________
>> 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]