More fine-grained modular decomposition
We should chop up our huger C files into smaller ones, based on actual module boundaries.
This will make it harder for us to merge pending branches that touch that code, but those are at a low ebb right now, so it's a good time.
The top 10 offenders in our current codebase are:
4614 src/or/rendservice.c
4839 src/or/channel.c
5200 src/or/connection.c
5386 src/or/or.h
5648 src/common/util.c
5666 src/or/directory.c
5688 src/or/routerparse.c
5771 src/or/routerlist.c
7223 src/or/control.c
8006 src/or/config.c
(updated May 2017)
- Show closed items
Activity
-
Newest first Oldest first
-
Show all activity Show comments only Show history only
- Nick Mathewson changed milestone to %Tor: unspecified
changed milestone to %Tor: unspecified
- Author
Trac:
Keywords: N/A deleted, tor-relay added - Author
Trac:
Component: Tor Relay to Tor - Author
Trac:
Milestone: Tor: 0.2.4.x-final to Tor: 0.2.5.x-final - Author
Trac:
Milestone: Tor: 0.2.5.x-final to Tor: 0.2.??? - Author
These may be worth looking at for 0.2.7.
Trac:
Milestone: Tor: 0.2.??? to Tor: 0.2.7.x-final - Author
Trac:
Status: new to assigned - Author
Marking some tickets as triaged-in for 0.2.7 based on early triage
Trac:
Keywords: N/A deleted, 027-triaged-1-in added Trac:
Priority: normal to major
Points: N/A to large
Version: N/A to Tor: 0.2.7
Keywords: N/A deleted, SponsorS added- Author
Trac:
Milestone: Tor: 0.2.7.x-final to Tor: 0.2.8.x-final - Author
Trac:
Keywords: N/A deleted, 028-triaged added - Author
Bulk-replace SponsorS keyword with SponsorS sponsor field in Tor component.
Trac:
Sponsor: N/A to SponsorS
Keywords: SponsorS deleted, N/A added - Author
It is impossible that we will fix all 234 currently open 028 tickets before 028 releases. Time to move some out. This is my first pass through the accepted and assigned tickets in my name, looking for things to move to 0.2.9.
Trac:
Owner: N/A to nickm
Milestone: Tor: 0.2.8.x-final to Tor: 0.2.9.x-final - Author
Trac:
Owner: nickm to N/A - Author
Trac:
Sponsor: SponsorS to SponsorS-can - Author
Trac:
Owner: N/A to nickm - Author
Trac:
Keywords: N/A deleted, tor-modularity added Trac:
Points: large to 6- Author
Trac:
Points: 6 to parent
Reviewer: N/A to N/A
Severity: N/A to Normal - Author
Dropping this out of 0.2.9, though I hope my module-mangling tools will make this infinitely easier in the future.
Trac:
Milestone: Tor: 0.2.9.x-final to Tor: 0.2.???
Owner: nickm to N/A - Author
Trac:
Status: assigned to new - Author
Finally admitting that 0.3.??? was a euphemism for Tor: unspecified all along.
Trac:
Keywords: N/A deleted, tor-03-unspecified-201612 added
Milestone: Tor: 0.3.??? to Tor: unspecified - Author
Remove an old triaging keyword.
Trac:
Keywords: tor-03-unspecified-201612 deleted, N/A added - Author
Trac:
Keywords: N/A deleted, 027-triaged-in added - Author
Trac:
Keywords: 027-triaged-in deleted, N/A added - Author
Trac:
Keywords: 027-triaged-1-in deleted, N/A added - Author
Trac:
Keywords: 028-triaged deleted, N/A added - Author
Trac:
Description: We should chop up our huger C files into smaller ones, based on actual module boundaries.This will make it harder for us to merge pending branches that touch that code, but those are at a low ebb right now, so it's a good time.
The top 10 offenders in our current codebase are:
3890 src/or/dirserv.c 3968 src/or/directory.c 4286 src/or/connection.c 4452 src/or/or.h 4735 src/or/control.c 4899 src/common/util.c 5263 src/or/routerparse.c 5468 src/or/config.c 5836 src/or/circuitbuild.c 5945 src/or/routerlist.c
to
We should chop up our huger C files into smaller ones, based on actual module boundaries.
This will make it harder for us to merge pending branches that touch that code, but those are at a low ebb right now, so it's a good time.
The top 10 offenders in our current codebase are:
4614 src/or/rendservice.c 4839 src/or/channel.c 5200 src/or/connection.c 5386 src/or/or.h 5648 src/common/util.c 5666 src/or/directory.c 5688 src/or/routerparse.c 5771 src/or/routerlist.c 7223 src/or/control.c 8006 src/or/config.c
(updated May 2017)
Keywords: tor-modularity deleted, tor-modularity refactoring code-movement technical-debt added - Nick Mathewson mentioned in issue #6838 (moved)
mentioned in issue #6838 (moved)
- Trac moved to tpo/core/tor#6837 (closed)
moved to tpo/core/tor#6837 (closed)