[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