Feature #4288
closed
Move sequence number file setting to the constructor
Added by Nicholas Gordon about 7 years ago.
Updated almost 6 years ago.
Description
Currently the SequencingManager relies on Nlsr::initialize to be in a valid state regarding the path for its sequence number file. Instead, it should be done at constructor time, and not rely on an external function.
- Target version changed from v0.4.0 to v0.5.0
- Target version changed from v0.5.0 to Minor release 0.4.1
- Related to Task #1952: Decouple classes from Nlsr object for easier unit testing added
- Related to deleted (Task #1952: Decouple classes from Nlsr object for easier unit testing)
- Blocked by Task #1952: Decouple classes from Nlsr object for easier unit testing added
- Target version changed from Minor release 0.4.1 to v0.5.0
- Assignee deleted (
Nicholas Gordon)
This work should still be done. Unfortunately, the path of the file containing the sequence numbers is not set during ctor time, and that path is not available until Nlsr::initialize
. This means that this change would require a change in that class, too.
- Status changed from New to Code review
- Assignee set to Ashlesh Gawande
- Tracker changed from Task to Feature
- Status changed from Code review to Closed
- % Done changed from 0 to 100
Also available in: Atom
PDF