Actions
Task #4877
closedThe ShouldAddToSyncDataCallback in FullProducer should check for sequence zero
Status:
Abandoned
Priority:
Normal
Assignee:
-
Target version:
-
Start date:
03/12/2019
Due date:
% Done:
0%
Estimated time:
Description
In the old code, onSyncInterest checked for sequence number zero
https://github.com/named-data/PSync/blob/e5fdcc3744d3787949aec0ef23ba9fc8a7db4572/PSync/full-producer.cpp#L198
if (m_prefixes[prefix] != 0 && !isFutureHash(prefix.toUri(), negative)) {
But in the new code, the ShouldAddToSyncDataCallback just checks the future hash.
https://github.com/8th-ndn-hackathon/PSync/blob/0fb388245a429e651ea15ac1f62696d4e5cfdbcb/PSync/full-producer.cpp#L46
Does it also need to check for sequence number zero? (Maybe this check was dropped accidentally during refactoring.)
Actions