From 3a8a0ebfc91ae1c8cd54685930330cdaca2a1398 Mon Sep 17 00:00:00 2001
From: Ben Darnell <ben@bendarnell.com>
Date: Sat, 19 Jan 2013 13:38:48 -0500
Subject: [PATCH] Remove conditional imports for py25.

ssl, json, and multiprocessing are now ubiquitous.

Also misc pyflakes cleanups (unused imports)
---
 tornado/auth.py                 |  1 -
 tornado/concurrent.py           |  2 --
 tornado/escape.py               | 34 ++++-----------------------------
 tornado/httpserver.py           |  6 +-----
 tornado/httputil.py             |  1 -
 tornado/ioloop.py               |  1 -
 tornado/iostream.py             |  6 +-----
 tornado/netutil.py              |  6 +-----
 tornado/platform/epoll.py       |  1 -
 tornado/platform/twisted.py     |  1 -
 tornado/process.py              |  7 +------
 tornado/simple_httpclient.py    |  7 +------
 tornado/test/httpclient_test.py |  1 -
 tornado/test/httpserver_test.py | 10 ++--------
 tornado/test/iostream_test.py   |  7 +------
 tornado/test/template_test.py   |  1 -
 tornado/test/testing_test.py    |  1 -
 tornado/testing.py              |  1 -
 tornado/wsgi.py                 |  3 +--
 19 files changed, 13 insertions(+), 84 deletions(-)

diff --git a/tornado/auth.py b/tornado/auth.py
index 6fa65442c..d158410a1 100644
--- a/tornado/auth.py
+++ b/tornado/auth.py
@@ -51,7 +51,6 @@ import binascii
 import hashlib
 import hmac
 import time
-import urllib
 import uuid
 
 from tornado import httpclient
diff --git a/tornado/concurrent.py b/tornado/concurrent.py
index 14786b03b..8af16d73e 100644
--- a/tornado/concurrent.py
+++ b/tornado/concurrent.py
@@ -16,10 +16,8 @@
 from __future__ import absolute_import, division, print_function, with_statement
 
 import functools
-import sys
 
 from tornado.stack_context import ExceptionStackContext
-from tornado.util import raise_exc_info
 
 try:
     from concurrent import futures
diff --git a/tornado/escape.py b/tornado/escape.py
index 6a2cbbb01..8b3e67b4c 100644
--- a/tornado/escape.py
+++ b/tornado/escape.py
@@ -30,10 +30,7 @@ from tornado.util import bytes_type, unicode_type, basestring_type, u
 try:
     from urllib.parse import parse_qs  # py3
 except ImportError:
-    try:
-        from urlparse import parse_qs  # Python 2.6+
-    except ImportError:
-        from cgi import parse_qs
+    from urlparse import parse_qs  # Python 2.6+
 
 try:
     import htmlentitydefs  # py2
@@ -45,30 +42,7 @@ try:
 except ImportError:
     import urllib as urllib_parse  # py2
 
-# json module is in the standard library as of python 2.6; fall back to
-# simplejson if present for older versions.
-try:
-    import json
-    assert hasattr(json, "loads") and hasattr(json, "dumps")
-    _json_decode = json.loads
-    _json_encode = json.dumps
-except Exception:
-    try:
-        import simplejson
-        _json_decode = lambda s: simplejson.loads(_unicode(s))
-        _json_encode = lambda v: simplejson.dumps(v)
-    except ImportError:
-        try:
-            # For Google AppEngine
-            from django.utils import simplejson
-            _json_decode = lambda s: simplejson.loads(_unicode(s))
-            _json_encode = lambda v: simplejson.dumps(v)
-        except ImportError:
-            def _json_decode(s):
-                raise NotImplementedError(
-                    "A JSON parser is required, e.g., simplejson at "
-                    "http://pypi.python.org/pypi/simplejson/")
-            _json_encode = _json_decode
+import json
 
 try:
     unichr
@@ -98,12 +72,12 @@ def json_encode(value):
     # the javscript.  Some json libraries do this escaping by default,
     # although python's standard library does not, so we do it here.
     # http://stackoverflow.com/questions/1580647/json-why-are-forward-slashes-escaped
-    return _json_encode(recursive_unicode(value)).replace("</", "<\\/")
+    return json.dumps(recursive_unicode(value)).replace("</", "<\\/")
 
 
 def json_decode(value):
     """Returns Python objects for the given JSON string."""
-    return _json_decode(to_basestring(value))
+    return json.loads(to_basestring(value))
 
 
 def squeeze(value):
diff --git a/tornado/httpserver.py b/tornado/httpserver.py
index ae063fedd..9498ea01f 100644
--- a/tornado/httpserver.py
+++ b/tornado/httpserver.py
@@ -27,6 +27,7 @@ This module also defines the `HTTPRequest` class which is exposed via
 from __future__ import absolute_import, division, print_function, with_statement
 
 import socket
+import ssl
 import time
 
 from tornado.escape import native_str, parse_qs_bytes
@@ -37,11 +38,6 @@ from tornado.netutil import TCPServer
 from tornado import stack_context
 from tornado.util import bytes_type
 
-try:
-    import ssl  # Python 2.6+
-except ImportError:
-    ssl = None
-
 try:
     import Cookie  # py2
 except ImportError:
diff --git a/tornado/httputil.py b/tornado/httputil.py
index e01550626..ceed539c2 100644
--- a/tornado/httputil.py
+++ b/tornado/httputil.py
@@ -18,7 +18,6 @@
 
 from __future__ import absolute_import, division, print_function, with_statement
 
-import urllib
 import re
 
 from tornado.escape import native_str, parse_qs_bytes, utf8
diff --git a/tornado/ioloop.py b/tornado/ioloop.py
index 4731564cd..362f68edd 100644
--- a/tornado/ioloop.py
+++ b/tornado/ioloop.py
@@ -36,7 +36,6 @@ import logging
 import numbers
 import os
 import select
-import sys
 import threading
 import time
 import traceback
diff --git a/tornado/iostream.py b/tornado/iostream.py
index 4aca7b11f..cf5307dd6 100644
--- a/tornado/iostream.py
+++ b/tornado/iostream.py
@@ -31,6 +31,7 @@ import errno
 import numbers
 import os
 import socket
+import ssl
 import sys
 import re
 
@@ -39,11 +40,6 @@ from tornado.log import gen_log, app_log
 from tornado import stack_context
 from tornado.util import bytes_type
 
-try:
-    import ssl  # Python 2.6+
-except ImportError:
-    ssl = None
-
 try:
     from tornado.platform.posix import _set_nonblocking
 except ImportError:
diff --git a/tornado/netutil.py b/tornado/netutil.py
index ed768cc96..f54c960da 100644
--- a/tornado/netutil.py
+++ b/tornado/netutil.py
@@ -21,6 +21,7 @@ from __future__ import absolute_import, division, print_function, with_statement
 import errno
 import os
 import socket
+import ssl
 import stat
 
 from tornado import process
@@ -30,11 +31,6 @@ from tornado.iostream import IOStream, SSLIOStream
 from tornado.log import app_log
 from tornado.platform.auto import set_close_exec
 
-try:
-    import ssl  # Python 2.6+
-except ImportError:
-    ssl = None
-
 
 class TCPServer(object):
     r"""A non-blocking, single-threaded TCP server.
diff --git a/tornado/platform/epoll.py b/tornado/platform/epoll.py
index 2b20a41bd..790b28692 100644
--- a/tornado/platform/epoll.py
+++ b/tornado/platform/epoll.py
@@ -16,7 +16,6 @@
 """EPoll-based IOLoop implementation for Linux systems."""
 from __future__ import absolute_import, division, print_function, with_statement
 
-import os
 import select
 
 from tornado.ioloop import PollIOLoop
diff --git a/tornado/platform/twisted.py b/tornado/platform/twisted.py
index 07db8085c..c61019ac7 100644
--- a/tornado/platform/twisted.py
+++ b/tornado/platform/twisted.py
@@ -68,7 +68,6 @@ from __future__ import absolute_import, division, print_function, with_statement
 
 import functools
 import datetime
-import time
 
 from twisted.internet.posixbase import PosixReactorBase
 from twisted.internet.interfaces import \
diff --git a/tornado/process.py b/tornado/process.py
index 90294b4c3..96c174ec3 100644
--- a/tornado/process.py
+++ b/tornado/process.py
@@ -19,7 +19,7 @@
 from __future__ import absolute_import, division, print_function, with_statement
 
 import errno
-import functools
+import multiprocessing
 import os
 import signal
 import subprocess
@@ -33,11 +33,6 @@ from tornado.iostream import PipeIOStream
 from tornado.log import gen_log
 from tornado import stack_context
 
-try:
-    import multiprocessing  # Python 2.6+
-except ImportError:
-    multiprocessing = None
-
 try:
     long  # py2
 except NameError:
diff --git a/tornado/simple_httpclient.py b/tornado/simple_httpclient.py
index 5b3c90287..24df4c396 100644
--- a/tornado/simple_httpclient.py
+++ b/tornado/simple_httpclient.py
@@ -17,19 +17,14 @@ import functools
 import os.path
 import re
 import socket
+import ssl
 import sys
-import time
 
 try:
     from io import BytesIO  # python 3
 except ImportError:
     from cStringIO import StringIO as BytesIO  # python 2
 
-try:
-    import ssl  # python 2.6+
-except ImportError:
-    ssl = None
-
 try:
     import urlparse  # py2
 except ImportError:
diff --git a/tornado/test/httpclient_test.py b/tornado/test/httpclient_test.py
index d4e0579a7..02509fbca 100644
--- a/tornado/test/httpclient_test.py
+++ b/tornado/test/httpclient_test.py
@@ -6,7 +6,6 @@ import base64
 import binascii
 from contextlib import closing
 import functools
-import re
 import sys
 
 from tornado.escape import utf8
diff --git a/tornado/test/httpserver_test.py b/tornado/test/httpserver_test.py
index b16e4c6a8..b8fb87621 100644
--- a/tornado/test/httpserver_test.py
+++ b/tornado/test/httpserver_test.py
@@ -17,14 +17,10 @@ import datetime
 import os
 import shutil
 import socket
+import ssl
 import sys
 import tempfile
 
-try:
-    import ssl
-except ImportError:
-    ssl = None
-
 
 class HandlerBaseTestCase(AsyncHTTPTestCase):
     def get_app(self):
@@ -142,7 +138,7 @@ class BadSSLOptionsTest(unittest.TestCase):
         })
 
         # This actually works because both files exist
-        server = HTTPServer(application, ssl_options={
+        HTTPServer(application, ssl_options={
            "certfile": existing_certificate,
            "keyfile": existing_certificate
         })
@@ -431,8 +427,6 @@ class KeepAliveTest(AsyncHTTPTestCase):
     connection reuse and closing.
     """
     def get_app(self):
-        test = self
-
         class HelloHandler(RequestHandler):
             def get(self):
                 self.finish('Hello world')
diff --git a/tornado/test/iostream_test.py b/tornado/test/iostream_test.py
index da0930349..a76e60219 100644
--- a/tornado/test/iostream_test.py
+++ b/tornado/test/iostream_test.py
@@ -12,13 +12,8 @@ import logging
 import os
 import platform
 import socket
+import ssl
 import sys
-import time
-
-try:
-    import ssl
-except ImportError:
-    ssl = None
 
 skipIfNoSSL = unittest.skipIf(ssl is None, "ssl module not present")
 
diff --git a/tornado/test/template_test.py b/tornado/test/template_test.py
index a69004c32..e39025663 100644
--- a/tornado/test/template_test.py
+++ b/tornado/test/template_test.py
@@ -5,7 +5,6 @@ import traceback
 
 from tornado.escape import utf8, native_str, to_unicode
 from tornado.template import Template, DictLoader, ParseError, Loader
-from tornado.testing import ExpectLog
 from tornado.test.util import unittest
 from tornado.util import u, bytes_type, ObjectDict, unicode_type
 
diff --git a/tornado/test/testing_test.py b/tornado/test/testing_test.py
index 428b8a101..75f07093a 100644
--- a/tornado/test/testing_test.py
+++ b/tornado/test/testing_test.py
@@ -1,7 +1,6 @@
 #!/usr/bin/env python
 
 from __future__ import absolute_import, division, print_function, with_statement
-import time
 from tornado.testing import AsyncTestCase
 from tornado.test.util import unittest
 
diff --git a/tornado/testing.py b/tornado/testing.py
index 0e3db883a..cc31e8c8f 100644
--- a/tornado/testing.py
+++ b/tornado/testing.py
@@ -43,7 +43,6 @@ import re
 import signal
 import socket
 import sys
-import time
 
 try:
     from io import StringIO  # py3
diff --git a/tornado/wsgi.py b/tornado/wsgi.py
index 5cfcedc3c..6c3d6a8b8 100644
--- a/tornado/wsgi.py
+++ b/tornado/wsgi.py
@@ -34,13 +34,12 @@ from __future__ import absolute_import, division, print_function, with_statement
 import sys
 import time
 import tornado
-import urllib
 
 from tornado import escape
 from tornado import httputil
 from tornado.log import access_log
 from tornado import web
-from tornado.escape import native_str, utf8, parse_qs_bytes
+from tornado.escape import native_str, parse_qs_bytes
 from tornado.util import bytes_type, unicode_type
 
 try:
-- 
2.47.2