On 1/4/07, Gerrit Renker <[EMAIL PROTECTED]> wrote:
Hi Ian,

I have tested this at home and found a small error:

|  +    if (hctx->ccid3hctx_x != old_x) {
|  +            ccid3_pr_debug("X_prev=%llu, X_now=%llu, X_calc=%u, "
|  +                           "X_recv=%llu\n", old_x, hctx->ccid3hctx_x,
|  +                           hctx->ccid3hctx_x_calc, hctx->ccid3hctx_x_recv);
|               ccid3_update_send_time(hctx);
==> old_x, x, and x_recv are all scaled by 2^6 and need to be divided (would 
give wrong results).

I have fixed this in the online version 
(3c_CCID3_add_more_debugging_McDonald.diff);
the inter-diff between previous and now is (having also updated `x' -> `X' for 
consistency):

Yes I did realise this but I should have been consistent and altered
the label. I thought it might be useful to look at the full number
sometimes....

Anyway I'm happy with your changes.

Ian
--
Web: http://wand.net.nz/~iam4
Blog: http://imcdnzl.blogspot.com
WAND Network Research Group
-
To unsubscribe from this list: send the line "unsubscribe dccp" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to