Merge branch 'netparams' into 'main'

NetParams: Document its role as a validated config parameter

See merge request !492

crates/arti-client/src/config.rs

@@ -250,9 +250,6 @@ pub struct TorClientConfig {
 
     /// Facility to override network parameters from the values set in the
     /// consensus.
-    //
-    // TODO: This field seems anomalous and should perhaps be changed somehow.
-    // Maybe NetParams<i32> ought to derive Builder.
     #[builder(
         sub_builder,
         field(

crates/tor-netdoc/src/doc/netstatus.rs

@@ -143,6 +143,9 @@ impl Lifetime {
 /// These are used to describe current settings for the Tor network,
 /// current weighting parameters for path selection, and so on.  They're
 /// encoded with a space-separated K=V format.
+///
+/// A `NetParams<i32>` is part of the validated directory manager configuraton,
+/// where it is built (in the builder-pattern sense) from a transparent HashMap.
 #[derive(Debug, Clone, Default, Eq, PartialEq)]
 pub struct NetParams<T> {
     /// Map from keys to values.