[tor-bugs] #32211 [Core Tor/Tor]: write description of subsystem initialization/shutdown architecture
Tor Bug Tracker & Wiki
blackhole at torproject.org
Thu Nov 14 13:58:35 UTC 2019
#32211: write description of subsystem initialization/shutdown architecture
-------------------------------------------------+-------------------------
Reporter: catalyst | Owner: nickm
Type: task | Status:
| needs_review
Priority: Medium | Milestone: Tor:
| 0.4.3.x-final
Component: Core Tor/Tor | Version:
Severity: Normal | Resolution:
Keywords: network-team-roadmap-september, | Actual Points: .2
s31-docs |
Parent ID: #29215 | Points:
Reviewer: catalyst | Sponsor:
| Sponsor31-must
-------------------------------------------------+-------------------------
Comment (by catalyst):
Replying to [comment:3 nickm]:
> See branch `ticket32211` for a small first draft here. I imagine that
more documentation is probably desirable; any guidance would be welcome.
PR at https://github.com/torproject/tor/pull/1520
Thanks! Looks good so far. What areas do you think need more work?
We might want to explain why we would want to add new fields in the middle
of `subsys_fns_t`.
In `initialization.dox`, maybe move the reference to `subsys_fns_t` closer
to the beginning of that paragraph? Maybe like
> `subsys_fns_t` describes a subsystem and a set of functions [...] To
define a subsystem...
As a minor style nit, do we need to use backslashes for Doxygen commands
in that places you use them here? I would like us to move towards more
consistently using at-signs for Doxygen commands.
--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/32211#comment:6>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online
More information about the tor-bugs
mailing list