Previously some of the comments here used /*.

This makes out-of-tree doxygen builds work.

Closes ticket 32113.

Running doxygen with latex gave us all manner of unicode issues,
slowed down the "make doxygen" target by a lot, and added several
latex dependencies... all to produce a 4000-page reference manual
which is probably not what anybody wanted.

Closes ticket 32099.

I have left the settings at their old values, except when they
referred to nonexistent files.

Closes ticket 32110

Doxygen thinks that saying #foo is linking to a "foo" anchor
someplace.

Doxygen seems to interpret anything starting with \ as a command,
even when we don't mean it that way.

In 8 places, our \file declarations didn't match the actual files
they were in.

fix three more instances in the man page that we forgot to fix
in earlier attempts

Merging without review, as this is an editorial fix in a comment.

"Ours" merge to avoid takign ".enable_practracker_in_hooks"

David Goulet authored 5 years ago and Nick Mathewson committed 5 years ago

Found by Coverity.

Fixes #31682

Signed-off-by: David Goulet <dgoulet@torproject.org>

George Kadianakis authored 5 years ago and Nick Mathewson committed 5 years ago

Code adapted from Rob's proposed patch in #30344.

Also add a comment in connection_mark_for_close_internal_() on why we should
not be adding extra code there without a very good reason.

Neel Chauhan authored 5 years ago and David Goulet committed 5 years ago

Unit test for #31652 where if we are over the retry limit for the IP but we
have an established circuit, we don't remove the IP.

Part of #31652

When considering introduction point of a service's descriptor, do not remove
an intro point that has an established or pending circuit.

Fixes #31652

Signed-off-by: David Goulet <dgoulet@torproject.org>