Project

General

Profile

Actions

Feature #4288

closed

Move sequence number file setting to the constructor

Added by Nicholas Gordon about 7 years ago. Updated almost 6 years ago.

Status:
Closed
Priority:
Normal
Target version:
Start date:
Due date:
% Done:

100%

Estimated time:

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.


Related issues 1 (0 open1 closed)

Blocked by NLSR - Task #1952: Decouple classes from Nlsr object for easier unit testingClosedAshlesh Gawande

Actions
Actions #1

Updated by Nicholas Gordon almost 7 years ago

  • Target version changed from v0.4.0 to v0.5.0
Actions #2

Updated by Nicholas Gordon almost 7 years ago

  • Target version changed from v0.5.0 to Minor release 0.4.1
Actions #3

Updated by Ashlesh Gawande almost 7 years ago

  • Related to Task #1952: Decouple classes from Nlsr object for easier unit testing added
Actions #4

Updated by Nicholas Gordon almost 7 years ago

  • Related to deleted (Task #1952: Decouple classes from Nlsr object for easier unit testing)
Actions #5

Updated by Nicholas Gordon almost 7 years ago

  • Blocked by Task #1952: Decouple classes from Nlsr object for easier unit testing added
Actions #6

Updated by Ashlesh Gawande over 6 years ago

  • Target version changed from Minor release 0.4.1 to v0.5.0
Actions #7

Updated by Nicholas Gordon over 6 years ago

  • 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.

Actions #8

Updated by Ashlesh Gawande almost 6 years ago

  • Status changed from New to Code review
  • Assignee set to Ashlesh Gawande
Actions #9

Updated by Ashlesh Gawande almost 6 years ago

  • Tracker changed from Task to Feature
  • Status changed from Code review to Closed
  • % Done changed from 0 to 100
Actions

Also available in: Atom PDF