Project

General

Profile

Actions

Bug #3383

closed

Deprecated makeDummyClientFace usage

Added by Junxiao Shi over 8 years ago. Updated almost 8 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
RIB
Target version:
Start date:
12/31/2015
Due date:
% Done:

100%

Estimated time:
1.50 h

Description

AutoPrefixPropagator RibManager RibStatusManager and some other test suites and fixtures are using deprecated ndn::util::makeDummyClientFace and DummyClientFace::sentDatas.
They should be replaced with the recommended non-deprecated counterparts.


Related issues 1 (0 open1 closed)

Blocked by ndn-cxx - Feature #3146: Face::getTransport methodClosedAlex Afanasyev

Actions
Actions #1

Updated by Junxiao Shi over 8 years ago

This could be a beginner task.

To find deprecated usages, execute grep -r -E 'makeDummyClientFace|sentDatas' tests/ on NFD repository.

To ensure no deprecated usage still exists after your patch, comment out NDN_UTIL_DUMMY_FACE_KEEP_DEPRECATED in ndn-cxx/util/dummy-client-face.hpp, and try to compile NFD.

Actions #2

Updated by Junxiao Shi over 8 years ago

Actions #3

Updated by Junxiao Shi almost 8 years ago

  • Status changed from New to In Progress
  • Assignee set to Junxiao Shi
Actions #4

Updated by Junxiao Shi almost 8 years ago

  • Status changed from In Progress to Code review
  • % Done changed from 0 to 100

NFD https://gerrit.named-data.net/2927

ndns https://gerrit.named-data.net/2928

ndn-tools https://gerrit.named-data.net/2929

ndn-group-encrypt https://gerrit.named-data.net/2930

ChronoSync https://gerrit.named-data.net/2932

NLSR https://gerrit.named-data.net/2933

nTorrent https://gerrit.named-data.net/2934

Among other projects currently on Jenkins:

No deprecated usage is found in ChronoChat ndn-traffic-generator repo-ng.

NFD-Android and ndnSIM use ndn-cxx forks so they are not affected.

ndn-cxx https://gerrit.named-data.net/2931

Actions #5

Updated by Junxiao Shi almost 8 years ago

Notice is sent to ndn-lib nfd-dev. Deadline is Jul 18 12:00 UTC.

Actions #6

Updated by Junxiao Shi almost 8 years ago

  • Status changed from Code review to Closed
Actions

Also available in: Atom PDF