commit 2f115411616b3a477aff94bb3def6c15a32704f5 Author: Isis Lovecruft <isis@torproject.org> Date: Sun Jan 5 17:28:19 2014 +0000 Return True if we were able to prepopulate sub hashrings. In bridgedb.Bridges.FilteredBridgeSplitter.prepopulateRings(), we return False if we were unable to prepopulate bridges into the sub hashring for whatever reason. In case some calling function wants to use the boolean return value as a check, we should mirror this logic and return True if prepopulation was successful. --- lib/bridgedb/Bridges.py | 2 ++ 1 file changed, 2 insertions(+) diff --git a/lib/bridgedb/Bridges.py b/lib/bridgedb/Bridges.py index 328cd2e..be88001 100644 --- a/lib/bridgedb/Bridges.py +++ b/lib/bridgedb/Bridges.py @@ -1026,6 +1026,8 @@ class FilteredBridgeSplitter(BridgeHolder): logging.info("Inserted %d bridges into hashring %s!" % (inserted, ringname)) + return True + def dumpAssignments(self, f, description=""): # one ring per filter set # bridges may be present in multiple filter sets