[tor-bugs] #32764 [Core Tor/Tor]: Solve code issues that block running clang-format on our code.
Tor Bug Tracker & Wiki
blackhole at torproject.org
Thu Jan 9 08:46:07 UTC 2020
#32764: Solve code issues that block running clang-format on our code.
--------------------------+------------------------------------
Reporter: nickm | Owner: nickm
Type: defect | Status: merge_ready
Priority: Medium | Milestone: Tor: 0.4.3.x-final
Component: Core Tor/Tor | Version:
Severity: Normal | Resolution:
Keywords: | Actual Points: 1
Parent ID: #29226 | Points: 1
Reviewer: teor | Sponsor:
--------------------------+------------------------------------
Changes (by teor):
* status: needs_review => merge_ready
Comment:
These changes seem fine to me.
GitHub is showing some conflicts, I'm happy for the merger to fix them up
when they merge,
The following inline header functions depend on some members of
or_options_t, which is a dependency we don't need:
* options_validate_dirauth_mode()
* options_validate_server_transport()
* options_validate_relay_mode()
And the dependency only exists when the relay or dirauth modules are
disabled.
So I'd like to open another ticket to remove that dependency.
(We could add stub C files that are only compiled when relay/dirauth mode
is disabled.)
--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/32764#comment:13>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online
More information about the tor-bugs
mailing list