mutt_ldap.py: Rename sys to _sys on import
authorW. Trevor King <wking@tremily.us>
Thu, 24 Jan 2013 22:18:58 +0000 (17:18 -0500)
committerW. Trevor King <wking@tremily.us>
Thu, 24 Jan 2013 22:18:58 +0000 (17:18 -0500)
Keeping the namespace clean when __name__ == '__main__' is not really
important, but this way all the imports are consistently renamed.
This future-proofs the code in case those modules ever need to get
shifted to module-wide imports.

mutt_ldap.py

index 21a1174da30e1fd3f065852189f2816987222f07..dab031c76fdd6e041f767b08a90d5f55848443c7 100755 (executable)
@@ -268,7 +268,7 @@ def format_entry(entry):
 if __name__ == '__main__':
     import codecs as _codecs
     import locale as _locale
-    import sys
+    import sys as _sys
 
     default_encoding = _locale.getpreferredencoding(do_setlocale=True)
     for key in ['output-encoding', 'argv-encoding']:
@@ -278,17 +278,18 @@ if __name__ == '__main__':
 
     # HACK: convert sys.stdout to Unicode (not needed in Python 3)
     output_encoding = CONFIG.get('system', 'output-encoding')
-    sys.stdout = _codecs.getwriter(output_encoding)(sys.stdout)
+    _sys.stdout = _codecs.getwriter(output_encoding)(_sys.stdout)
 
     # HACK: convert sys.argv to Unicode (not needed in Python 3)
     argv_encoding = CONFIG.get('system', 'argv-encoding')
-    sys.argv = [unicode(arg, argv_encoding) for arg in sys.argv]
+    _sys.argv = [unicode(arg, argv_encoding) for arg in _sys.argv]
 
-    if len(sys.argv) < 2:
-        sys.stderr.write(u'{0}: no search string given\n'.format(sys.argv[0]))
-        sys.exit(1)
+    if len(_sys.argv) < 2:
+        _sys.stderr.write(
+            u'{0}: no search string given\n'.format(_sys.argv[0]))
+        _sys.exit(1)
 
-    query = u' '.join(sys.argv[1:])
+    query = u' '.join(_sys.argv[1:])
 
     if CONFIG.getboolean('cache', 'enable'):
         connection_class = CachedLDAPConnection