Opened 17 months ago

Closed 17 months ago

Last modified 14 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 17 months ago by nickm

Status: assignedneeds_review

Branch is split_stats, PR at

comment:2 Changed 17 months ago by dgoulet

Keywords: refactor added
Reviewer: dgoulet

comment:3 Changed 17 months ago by dgoulet

Status: needs_reviewmerge_ready

Small detail pointed out, minor. The rest lgtm!

comment:4 Changed 17 months ago by nickm

thanks! Fixed and merged!

comment:5 Changed 17 months ago by nickm

Resolution: implemented
Status: merge_readyclosed

comment:6 Changed 14 months ago by nickm

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