Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use a single source of truth for InFlight calculation #99

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Commits on Sep 23, 2020

  1. Use a single source of truth for InFlight calculation

    ndn-consumer-window and ndn-consumer-pcon use a simple count for
    updating the number of InFlight packets. This works fines as long as no
    packet arrives after its timeout has fired. In this case `m_inFlight` is
    decreased twice. Ultimately the counter (wrongly) reaches 0 and prevents
    proper working of the window algorithm.
    
    This fix does away with the manual calculation of the counter and replaces
    it with the size of the `m_seqTimeouts` set.
    
    Signed-off-by: Miguel Rodríguez Pérez <miguel@det.uvigo.gal>
    migrax committed Sep 23, 2020
    Configuration menu
    Copy the full SHA
    5b5e59e View commit details
    Browse the repository at this point in the history