[metrics-bugs] #22141 [Metrics/metrics-lib]: Deprecate `DescriptorFile` and add relevant information to `Descriptor`

Tor Bug Tracker & Wiki blackhole at torproject.org
Wed Jun 7 20:11:05 UTC 2017


#22141: Deprecate `DescriptorFile` and add relevant information to `Descriptor`
---------------------------------+-----------------------------------
 Reporter:  karsten              |          Owner:  metrics-team
     Type:  enhancement          |         Status:  needs_revision
 Priority:  Medium               |      Milestone:  metrics-lib 1.9.0
Component:  Metrics/metrics-lib  |        Version:
 Severity:  Normal               |     Resolution:
 Keywords:                       |  Actual Points:
Parent ID:                       |         Points:
 Reviewer:                       |        Sponsor:
---------------------------------+-----------------------------------
Changes (by iwakeh):

 * status:  needs_review => needs_revision


Comment:

 It would be great, if the functionality in `readDescriptorFiles` could be
 broken down into
 smaller and testable units.  I know that there are other places like this
 in the code, but
 we should rather work toward nicer code with new additions like this. (and
 then apply this
 to improve the legacy code)

 There are some checkstyle warnings about unused imports and javadoc.

 Should the parts concerning #16225 be mentioned there for reference?

--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/22141#comment:5>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online


More information about the metrics-bugs mailing list