[ndnSIM] Redundant code in Forwarder::onIncomingData pipeline?

[Sonia]

Hi,

Please guide me if I am missing something.

In onIncomingData pipeline when there is at least one PIT entry match (  if
(pitMatches.size() == 1) { ....... )  it calls Strategy.afterReceiveData()
method and this method sends data to all pending down streams via its
this->sendDataToAll(pitEntry,
inFace, data); method.

My question is then why the else statement (for multiple downstream faces)
is there in Forwarder::onIncomingData pipeline. Isn't this code redundant
as the  Strategy.afterReceiveData() method already have implemented the
same logic?

My second concern is that I have run many scenarios (both p2p and adhoc)
and multiple strategies (including multicast and broadcast), but I always
get only one downstream face never got more than 1 pending downstream face.
I checked it in Forwarder log.

Any quidance will be highly appreciated.

Thanks
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.lists.cs.ucla.edu/pipermail/ndnsim/attachments/20210423/51288319/attachment-0001.html>