Project

General

Profile

Actions

Task #4922

closed

Source code reorganization

Added by Davide Pesavento about 5 years ago. Updated almost 4 years ago.

Status:
Closed
Priority:
Low
Category:
Build
Target version:
Start date:
Due date:
% Done:

100%

Estimated time:

Description

The current NFD's source code layout is the product of historical design choices that were made at the beginning of the project. While many of those choices still apply to the codebase, a few others have been superseded and are no longer relevant today. Some notable examples of major design changes that rendered (or will render) the current source code layout partially obsolete are #2489, #4683, and #4528.

This task proposes the following reorganizations.

  • Move rib/ to daemon/rib/ (already done in commit 1b077f6028543f4304d1e5afe383823d79305ce1)
  • Move RibManager to daemon/mgmt/ (already done in commit 8a05c7fd2a00e7c8b53d5c57ccb3c53a756b4997)
  • Move core/ files that are only used by the nfd binary to daemon/common/ or any other appropriate subdir of daemon/
    • RttEstimator can be left alone since it will be superseded by ndn-cxx's estimator and removed soon in #4887

The split between unit-tests-daemon and unit-tests-rib also doesn't make much sense anymore. I see two possible approaches here: (a) merge unit-tests-rib into unit-tests-daemon, or (b) further split unit-tests-daemon into -face, -table, -mgmt, etc.

Actions #1

Updated by Davide Pesavento about 5 years ago

  • Description updated (diff)
Actions #2

Updated by Davide Pesavento about 5 years ago

  • Description updated (diff)
Actions #3

Updated by Davide Pesavento about 5 years ago

  • % Done changed from 0 to 50
Actions #4

Updated by Davide Pesavento almost 5 years ago

  • Tags set to CodeCleanup
Actions #5

Updated by Davide Pesavento over 4 years ago

  • Status changed from In Progress to Closed
  • Priority changed from Normal to Low
  • % Done changed from 50 to 100
Actions #6

Updated by Davide Pesavento almost 4 years ago

  • Tags changed from CodeCleanup to code-cleanup
Actions

Also available in: Atom PDF