commit bb3de90a82ff927c5005fbadce3b2ea975a15aae Author: Damian Johnson atagar@torproject.org Date: Sun Aug 11 13:49:30 2013 -0700

Dropping document_handler from get_consensus()

Oops, the document_handler is already a query parameter. There's no benefit to having a default value for it in get_consensus(), and that had the drawback of preventing us from asserting a default value. --- stem/descriptor/remote.py | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/stem/descriptor/remote.py b/stem/descriptor/remote.py index 95934e1..40d9afe 100644 --- a/stem/descriptor/remote.py +++ b/stem/descriptor/remote.py @@ -517,7 +517,7 @@ class DescriptorDownloader(object):

return self.query('/tor/micro/d/%s.z' % '-'.join(hashes), **query_args)

- def get_consensus(self, authority_v3ident = None, document_handler = stem.descriptor.DocumentHandler.ENTRIES, **query_args): + def get_consensus(self, authority_v3ident = None, **query_args): """ Provides the present router status entries.

@@ -525,8 +525,6 @@ class DescriptorDownloader(object): to get the consensus, see `'v3ident' in tor's config.c https://gitweb.torproject.org/tor.git/blob/f631b73:/src/or/config.c#l816`_ for the values. - :param stem.descriptor.__init__.DocumentHandler document_handler: method in - which to parse the :class:`~stem.descriptor.networkstatus.NetworkStatusDocumentV3` :param query_args: additional arguments for the :class:`~stem.descriptor.remote.Query` constructor

@@ -539,7 +537,7 @@ class DescriptorDownloader(object): if authority_v3ident: resource += '/%s' % authority_v3ident

- return self.query(resource + '.z', document_handler = document_handler, **query_args) + return self.query(resource + '.z', **query_args)

def get_key_certificates(self, authority_v3idents = None, **query_args): """