Actions
Bug #3279
closedSchedule doesn't encode RepetitiveInterval in ascending order
Start date:
10/20/2015
Due date:
% Done:
100%
Estimated time:
Description
In Schedule::wireEncode, the fields are encoded in reverse order (as normal), so the RepetitiveInterval iterator should go backwards to put the encoded elements in ascending order. But the iterator goes forward: https://github.com/named-data/ndn-group-encrypt/blob/7cc09fc4e97adcab02731b99efb96eecdf81364b/src/schedule.cpp#L50
Is it your intention that the RepetitiveInterval elements should be encoded in ascending order? Or does the encoding specify and order? (If there is not a defined order, then it's hard to make the unit tests pass because the results will vary from test cases.)
Updated by Yingdi Yu about 8 years ago
- Blocked by Bug #3533: RepetitiveInterval "less" operator is not implemented correctly. added
Updated by Yingdi Yu about 8 years ago
- Status changed from New to Code review
- Assignee set to Yingdi Yu
- % Done changed from 0 to 80
Updated by Yingdi Yu about 8 years ago
- Status changed from Code review to Closed
- % Done changed from 80 to 100
Actions