[tor-bugs] #30914 [Core Tor/Tor]: Move struct manipulation code out of confparse.c
Tor Bug Tracker & Wiki
blackhole at torproject.org
Thu Aug 22 14:27:22 UTC 2019
#30914: Move struct manipulation code out of confparse.c
--------------------------+------------------------------------
Reporter: nickm | Owner: nickm
Type: defect | Status: merge_ready
Priority: Medium | Milestone: Tor: 0.4.2.x-final
Component: Core Tor/Tor | Version:
Severity: Normal | Resolution:
Keywords: | Actual Points: 1
Parent ID: #29211 | Points: 1
Reviewer: teor | Sponsor: Sponsor31-can
--------------------------+------------------------------------
Changes (by teor):
* status: needs_review => merge_ready
Comment:
This branch and the general design seem ok to me.
Thanks for cleaning it up!
It would be helpful to have a diagram showing the ideal state, and where
we are in the refactor.
There is a bit of boilerplate, but there seems to be less boilerplate as
we go along. It's starting to feel a bit like trunnel. Not sure if code
generation would help here, or later in the process.
I left a few minor questions on the PR, feel free to fix them and then get
someone else to merge. Or leave them and fix them later.
--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/30914#comment:9>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online
More information about the tor-bugs
mailing list