From 9228db5f1bee0b715b73ba6f8c11f5eb77d59f62 Mon Sep 17 00:00:00 2001 From: Roger Dingledine Date: Sun, 2 Oct 2016 19:13:35 -0400 Subject: [PATCH] remove confusing comment about conditional consensus download We removed that feature in 0.2.4.2-alpha, but some comments seem to have lingered. I didn't add a changes/ file since this is just internal code cleanup. --- src/or/directory.c | 9 +++------ 1 file changed, 3 insertions(+), 6 deletions(-) diff --git a/src/or/directory.c b/src/or/directory.c index 52b14b9bae..1f894d9fb3 100644 --- a/src/or/directory.c +++ b/src/or/directory.c @@ -495,9 +495,6 @@ MOCK_IMPL(void, directory_get_from_dirserver, ( * sort of dir fetch we'll be doing, so it won't return a bridge * that can't answer our question. */ - /* XXX+++++ Not all bridges handle conditional consensus downloading, - * so, for now, never assume the server supports that. -PP - * Is that assumption still so in 2016? -NM */ const node_t *node = choose_random_dirguard(type); if (node && node->ri) { /* every bridge has a routerinfo. */ @@ -1305,9 +1302,9 @@ compare_strs_(const void **a, const void **b) /** Return the URL we should use for a consensus download. * - * This url depends on whether or not the server we go to - * is sufficiently new to support conditional consensus downloading, - * i.e. GET .../consensus/fpr+fpr+fpr + * Use the "conditional consensus downloading" feature described in + * dir-spec.txt, i.e. + * GET .../consensus/fpr+fpr+fpr * * If 'resource' is provided, it is the name of a consensus flavor to request. */