Remove need for options_validate to use "defaults".

The options_validate() function takes a "defaults" argument; it doesn't need to.

Now that #32175 (moved) is in, this should be much simpler than the last time I tried to do this.

changed milestone to %Tor: 0.4.3.x-final

Trac:
Parent Ticket: #31241 (moved)

added actualpoints::.1 component::core tor/tor milestone::Tor: 0.4.3.x-final network-team-roadmap-august owner::nickm parent::31241 points::.1 priority::medium resolution::fixed severity::normal sponsor::31-can status::closed type::defect labels

https://github.com/torproject/tor/pull/1446 is the PR here. Once this is in I can start simplifying and refactoring validation at last.

I'll put this into needs_review after CI has passed.

Travis has passed; waiting for appveyor.

Trac:
Status: assigned to needs_review

Trac:
Actualpoints: N/A to .1

(Note: I'm going to actually remove these arguments in #32187 (moved))

The patch looks good, but appveyor isn't ready yet. I'm putting this to merge_ready under the condition that appveyor goes green before it gets merged :-)

Trac:
Status: needs_review to merge_ready

Appveyor passed; merged this to master.

Trac:
Status: merge_ready to closed
Resolution: N/A to fixed

closed

changed time estimate to 48m

added 48m of time spent

mentioned in issue #31241 (moved)

moved to tpo/core/tor#32185 (closed)