Suggested by iwakeh on #19571.

Fixed the following number of issues per type:

1131 Indentation
 431 EmptyLineSeparator
  95 CustomImportOrder
  76 OperatorWrap
  54 SeparatorWrap
  25 LocalVariableName
  14 ModifierOrder
  13 MultipleVariableDeclarations
   5 WhitespaceAround
   5 JavadocTagContinuationIndentation
   4 EmptyCatchBlock
   2 VariableDeclarationUsageDistance
   2 MissingSwitchDefault
   1 (... types with single occurrence omitted...)

Copied over from CollecTor.

Implements #19284.

 - Support more than one "directory-signature" line in a vote, which
   may become relevant when authorities start signing votes using more
   than one algorithm.
 - Provide directory signatures in consensuses and votes in a list
   rather than a map to support multiple signatures made using the
   same identity key digest but different algorithms.
 - Be more lenient about digest lengths in directory signatures which
   may be longer or shorter than 20 bytes.

Implements #18875.

While implementing this, make "sha1" constant and deprecate
RelayNetworkStatusVote.getSigningKeyDigest(), because it's remissible
and ambiguous.  Suggested or based on discussions with iwakeh.

Also reduce visibility of all public methods in ParseHelper and of one
public method in TorperfResultsImpl, and remove unused methods in
DirectorySignatureImpl and ExitListEntryImpl.

Implements #19285.

Implements #19051.

Based in parts on very helpful suggestions and feedback by iwakeh.

Implements #16873.

And don't break if x is non-numeric.  Spotted by iwakeh.

Previously, we would have added as many copies of a Torperf line as it
contains keys we don't recognize, which is not really usable.

Implements #17934.

We shouldn't call someStringBuilder.append(someString + "\n"), but
rather someStringBuilder.append(someString).append("\n"), or Java will
internally build another StringBuilder for the + part.

Suggested by iwakeh, implements #17830.

Rather than writing chained if-else statements with String comparison
for equals(), we can now switch on a String instance.

Suggested by iwakeh, implements #17824.

In many cases, the compiler will now infer which parameter types a new
object shall have, which allows us to make our code less verbose.

Suggested by iwakeh, implements #17823.

This patch is loosely based on metrics-web's hidserv module.

More precisely,
 - support Ed25519 certificates and Ed25519 master keys as well as
   SHA-256 digests and Ed25519 signatures thereof in server
   descriptors and extra-info descriptors,
 - parse RSA-1024 signatures of SHA-1 digests of extra-info
   descriptors,
 - parse Ed25519 master keys in votes, and
 - parse Ed25519 and RSA-1024 identity digests in microdescriptors.

This patch is based on metrics-db's bridge descriptor sanitizer.

So far, we did not distinguish between relay and bridge descriptors in
the case of server descriptors and extra-info descriptors.  This works
just fine, because we tried hard to re-use existing keywords in relay
descriptors for sanitized contents in bridge descriptors to facilitate
parsing.

However, some applications process both relay and bridge descriptors
and need to add workarounds for distinguishing between the two.  For
example, they couldn't just read the contents of CollecTor's recent/
directory, because they wouldn't be able to know whether a
ServerDescriptor instance was published by a relay or bridge.  Or even
worse, if an application expects a given directory to contain relay
descriptors and that directory suddenly contains bridge descriptors,
that application wouldn't notice.

This change adds new interfaces to distinguish between relay and
bridge descriptors.  It still supports the existing two interfaces
that don't make this distinction.  Those two interfaces are not
deprecated, because it's okay if an application does not care whether
a descriptor was published by a relay or bridge.

This change is in parts based on a discussion with atagar.  Thanks!

Implements #17000.