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

Fabiano FidĂȘncio fidencio at redhat.com
Wed Nov 11 05:21:21 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!


More information about the Spice-devel mailing list