[Spice-devel] [spice 1/2] server: Don't reset the latency before showing it in the invalid net test error message.

[Uri Lublin]

On 07/15/2015 05:34 PM, Francois Gouget wrote:
> Signed-off-by: Francois Gouget <fgouget at codeweavers.com>
> ---
>   server/main_channel.c | 6 +++---
>   1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/server/main_channel.c b/server/main_channel.c
> index e7df3f9..f1b38af 100644
> --- a/server/main_channel.c
> +++ b/server/main_channel.c
> @@ -969,11 +969,11 @@ static int main_channel_handle_parsed(RedChannelClient *rcc, uint32_t size, uint
>                   mcc->net_test_id = 0;
>                   if (roundtrip <= mcc->latency) {
>                       // probably high load on client or server result with incorrect values
> +                    spice_printerr("net test: invalid values, latency %" PRIu64
> +                                   " roundtrip %" PRIu64 ". assuming high"
> +                                   " bandwidth", mcc->latency, roundtrip);
>                       mcc->latency = 0;
>                       mcc->net_test_stage = NET_TEST_STAGE_INVALID;
> -                    spice_printerr("net test: invalid values, latency %" PRIu64
> -                               " roundtrip %" PRIu64 ". assuming high"
> -                               "bandwidth", mcc->latency, roundtrip);
>                       red_channel_client_start_connectivity_monitoring(&mcc->base,
>                                                                        CLIENT_CONNECTIVITY_TIMEOUT);
>                       break;
>

Ack.