Subject: [quicwg/base-drafts] 290a51: Split PTO calculation into a dedicated method

From: Jana Iyengar <[email-masked]>
Date: Sat, 30 May 2020 19:11:54 -0700

Branch: refs/heads/master
  Home:   https://github.com/quicwg/base-drafts
  Commit: 290a5113d74c36b112d3ada80edc2f3ede23325b
      https://github.com/quicwg/base-drafts/commit/290a5113d74c36b112d3ada80edc2f3ede23325b
  Author: ianswett <[email-masked]>
  Date:   2020-05-20 (Wed, 20 May 2020)

  Changed paths:
    M draft-ietf-quic-recovery.md

  Log Message:
  -----------
  Split PTO calculation into a dedicated method

As @martinthomson pointed out in #3666, it was far too complex and error-prone as-is.

Als fixes #3564 and #3674 by calculating the PTO timeout for each PN space and picking the earliest, as well as checking that there are inflight packets before calculating PTO for a PN space.

  Commit: d0050a08a17463fbf9f3bf25b61b50a982b4a07f
      https://github.com/quicwg/base-drafts/commit/d0050a08a17463fbf9f3bf25b61b50a982b4a07f
  Author: ianswett <[email-masked]>
  Date:   2020-05-20 (Wed, 20 May 2020)

  Changed paths:
    M draft-ietf-quic-recovery.md

  Log Message:
  -----------
  Update draft-ietf-quic-recovery.md

  Commit: 65b036d2c0437e81e0533a76e863bebfaa3207a3
      https://github.com/quicwg/base-drafts/commit/65b036d2c0437e81e0533a76e863bebfaa3207a3
  Author: ianswett <[email-masked]>
  Date:   2020-05-20 (Wed, 20 May 2020)

  Changed paths:
    M draft-ietf-quic-recovery.md

  Log Message:
  -----------
  Update draft-ietf-quic-recovery.md

  Commit: 2630bfdfe743909fb7bd3a92a41bd5f794cc503d
      https://github.com/quicwg/base-drafts/commit/2630bfdfe743909fb7bd3a92a41bd5f794cc503d
  Author: ianswett <[email-masked]>
  Date:   2020-05-20 (Wed, 20 May 2020)

  Changed paths:
    M draft-ietf-quic-recovery.md

  Log Message:
  -----------
  Update draft-ietf-quic-recovery.md

  Commit: 83727ca4f10a4d3866ea026efff3cb833111d1f8
      https://github.com/quicwg/base-drafts/commit/83727ca4f10a4d3866ea026efff3cb833111d1f8
  Author: ianswett <[email-masked]>
  Date:   2020-05-20 (Wed, 20 May 2020)

  Changed paths:
    M draft-ietf-quic-recovery.md

  Log Message:
  -----------
  Update draft-ietf-quic-recovery.md

  Commit: f10875ed05f2edd3fe121631bf89c4a877204fb5
      https://github.com/quicwg/base-drafts/commit/f10875ed05f2edd3fe121631bf89c4a877204fb5
  Author: ianswett <[email-masked]>
  Date:   2020-05-20 (Wed, 20 May 2020)

  Changed paths:
    M draft-ietf-quic-recovery.md

  Log Message:
  -----------
  Fix errors

  Commit: 242e38d3b7e6170b59b347de6d8b3686af356c11
      https://github.com/quicwg/base-drafts/commit/242e38d3b7e6170b59b347de6d8b3686af356c11
  Author: ianswett <[email-masked]>
  Date:   2020-05-21 (Thu, 21 May 2020)

  Changed paths:
    M draft-ietf-quic-recovery.md

  Log Message:
  -----------
  Apply suggestions from code review

Co-authored-by: Martin Thomson <[email-masked]>

  Commit: 7a330219f1b116f3bc1ddbcf8cb8fd8fdd7355b8
      https://github.com/quicwg/base-drafts/commit/7a330219f1b116f3bc1ddbcf8cb8fd8fdd7355b8
  Author: ianswett <[email-masked]>
  Date:   2020-05-21 (Thu, 21 May 2020)

  Changed paths:
    M draft-ietf-quic-recovery.md

  Log Message:
  -----------
  Update draft-ietf-quic-recovery.md

  Commit: 6ce4f9b1841f8f2296e7420b08a928a41f7f2a5a
      https://github.com/quicwg/base-drafts/commit/6ce4f9b1841f8f2296e7420b08a928a41f7f2a5a
  Author: ianswett <[email-masked]>
  Date:   2020-05-27 (Wed, 27 May 2020)

  Changed paths:
    M draft-ietf-quic-recovery.md

  Log Message:
  -----------
  Marten's suggestions

  Commit: 820497e882f930dcdc5d3ae7401add631ec0ea38
      https://github.com/quicwg/base-drafts/commit/820497e882f930dcdc5d3ae7401add631ec0ea38
  Author: ianswett <[email-masked]>
  Date:   2020-05-28 (Thu, 28 May 2020)

  Changed paths:
    M draft-ietf-quic-http.md
    M draft-ietf-quic-invariants.md
    M draft-ietf-quic-recovery.md
    M draft-ietf-quic-tls.md
    M draft-ietf-quic-transport.md

  Log Message:
  -----------
  Merge branch 'master' into ianswett-split-pto-pseudocode

  Commit: 829501299ef329a19ab52f9f0702a85e144c18a7
      https://github.com/quicwg/base-drafts/commit/829501299ef329a19ab52f9f0702a85e144c18a7
  Author: ianswett <[email-masked]>
  Date:   2020-05-30 (Sat, 30 May 2020)

  Changed paths:
    M draft-ietf-quic-recovery.md

  Log Message:
  -----------
  Add an assert

  Commit: 92c2e22eaa7bce92d055dba4bf72ecfbc505e5dc
      https://github.com/quicwg/base-drafts/commit/92c2e22eaa7bce92d055dba4bf72ecfbc505e5dc
  Author: ianswett <[email-masked]>
  Date:   2020-05-30 (Sat, 30 May 2020)

  Changed paths:
    M draft-ietf-quic-recovery.md

  Log Message:
  -----------
  Apply suggestions from code review

Co-authored-by: Jana Iyengar <[email-masked]>

  Commit: 8644f6db1937825ce8162558ea33b135967c3eeb
      https://github.com/quicwg/base-drafts/commit/8644f6db1937825ce8162558ea33b135967c3eeb
  Author: Jana Iyengar <[email-masked]>
  Date:   2020-05-30 (Sat, 30 May 2020)

  Changed paths:
    M draft-ietf-quic-recovery.md

  Log Message:
  -----------
  Merge pull request #3681 from quicwg/ianswett-split-pto-pseudocode

Split PTO calculation into a dedicated method

Compare: https://github.com/quicwg/base-drafts/compare/72e6455c1499...8644f6db1937