Opened 7 months ago

Closed 7 months ago

Last modified 3 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:

Description

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 7 months ago by nickm

Status: assignedneeds_review

Branch is split_stats, PR at https://github.com/torproject/tor/pull/376

comment:2 Changed 7 months ago by dgoulet

Keywords: refactor added
Reviewer: dgoulet

comment:3 Changed 7 months ago by dgoulet

Status: needs_reviewmerge_ready

Small detail pointed out, minor. The rest lgtm!

comment:4 Changed 7 months ago by nickm

thanks! Fixed and merged!

comment:5 Changed 7 months ago by nickm

Resolution: implemented
Status: merge_readyclosed

comment:6 Changed 3 months ago by nickm

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