Task #1283
closed
Task #1205: NDNLP
NDNLP-TLV encode FragIndex and FragCount using nonNegativeInteger
Added by Junxiao Shi over 10 years ago.
Updated over 10 years ago.
Description
Change NDNLP-TLV spec and implementation to encode FragIndex and FragCount using nonNegativeInteger.
Sequence field still uses 8-octet fixed length number, because sequence number can wrap around, but nonNegativeNumber can increment forever.
This is a decision from 20140221 conference call.
The original reason for using fixed length fields for FragIndex and FragCount is the concern on encoding performance.
ndn-cpp-dev has an efficient EncodingBuffer
, which makes the above reason invalid.
There is no disadvantage in moving to variable length nonNegativeNumber encoding.
- Description updated (diff)
- % Done changed from 0 to 30
- Status changed from In Progress to Code review
- % Done changed from 30 to 80
Let's be more consistent and update all counters to nonNegativeInteger... Less chances of having portability problems.
Also about wrapping around. Is it even feasible with 8-byte number? You gave me an argument before that it will take many years to do so...
NdnlpSequence
is not a counter. It should not be defined as nonNegativeInteger.
There is no known portability problem of using a fixed length field.
what is sequence number then? Is it fragment ID (which is unique for all fragments) or real sequence (which is increased by one for all segments)?
20140224 conference call decides not to change NdnlpSequence
field.
- % Done changed from 80 to 100
- Status changed from Code review to Closed
Also available in: Atom
PDF