Opened 23 months ago

Closed 23 months ago

Last modified 19 months ago

#27892 closed defect (implemented)

Split the non-stats part of the stats module into different modules

Reported by: nickm Owned by: nickm
Priority: Medium Milestone: Tor: 0.3.5.x-final
Component: Core Tor/Tor Version:
Severity: Normal Keywords: refactor
Cc: Actual Points: .3
Parent ID: Points:
Reviewer: dgoulet Sponsor:


Part of rephist.c is about predicted ports, which aren't really stats.

Part of geoip.c is about stats, but part is about mapping IPs to countries, which isn't stats.

Let's fix this while we can.

Child Tickets

Change History (6)

comment:1 Changed 23 months ago by nickm

Status: assignedneeds_review

Branch is split_stats, PR at

comment:2 Changed 23 months ago by dgoulet

Keywords: refactor added
Reviewer: dgoulet

comment:3 Changed 23 months ago by dgoulet

Status: needs_reviewmerge_ready

Small detail pointed out, minor. The rest lgtm!

comment:4 Changed 23 months ago by nickm

thanks! Fixed and merged!

comment:5 Changed 23 months ago by nickm

Resolution: implemented
Status: merge_readyclosed

comment:6 Changed 19 months ago by nickm

Actual Points: .3
Note: See TracTickets for help on using tickets.