Feature #4267
openRemove explicit TLV classes
0%
Description
Currently, NLSR maintains a separate set of classes and data structures for the purposes of data serialization. This is overly complicated and not viable in the long-term.
- NLSR will not duplicate existing data structures for the purposes of serialization.
Updated by Nicholas Gordon over 7 years ago
- Tracker changed from Task to Feature
- Description updated (diff)
- Start date deleted (
09/05/2017)
Currently NLSR maintains a set of separate classes for encoding data structures to TLV format. In my view, this is not an optimal approach. This is a simple problem to solve, since the process is similar each time, and is a basic transformation to a string. This can be accomplished by a free function, for example, or by a method that employs a common helper function. Further, continually duplicating TLV classes is not a tenable long-term design paradigm -- as the number of published things increases, complexity of these structures will increase, leading to double the maintenance work. We should aim to minimize the code surface, and only add complexity when necessary.
Updated by Ashlesh Gawande over 4 years ago
- Has duplicate Task #4787: Consolidate src/tlv/**lsa** into src/**lsa** added