From bf7735821ce742230cd892c87826ba0a545eceed Mon Sep 17 00:00:00 2001 From: Delirious Lettuce Date: Wed, 24 May 2017 13:22:41 -0600 Subject: [PATCH] Removing '\n' arg negates the need for `wordlist.remove()`, renamed unused variable 'x' to '_' --- onionshare/common.py | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/onionshare/common.py b/onionshare/common.py index cff2224b..508acd36 100644 --- a/onionshare/common.py +++ b/onionshare/common.py @@ -136,11 +136,10 @@ def build_slug(): Returns a random string made from two words from the wordlist, such as "deter-trig". """ with open(get_resource_path('wordlist.txt')) as f: - wordlist = f.read().split('\n') - wordlist.remove('') + wordlist = f.read().split() r = SystemRandom() - return '-'.join(r.choice(wordlist) for x in range(2)) + return '-'.join(r.choice(wordlist) for _ in range(2)) def human_readable_filesize(b):