remove some of Python 2.5 compatibility hacks

Author: kmike

_line_profiler.pyx

@@ -127,13 +127,9 @@ cdef class LineProfiler:
         try:
             code = func.__code__
         except AttributeError:
-            try:
-                # Python 2.x
-                code = func.func_code
-            except AttributeError:
-                import warnings
-                warnings.warn("Could not extract a code object for the object %r" % (func,))
-                return
+            import warnings
+            warnings.warn("Could not extract a code object for the object %r" % (func,))
+            return
         if code not in self.code_map:
             self.code_map[code] = {}
             self.functions.append(func)

line_profiler.py

@@ -1,6 +1,5 @@
 #!/usr/bin/env python
 # -*- coding: UTF-8 -*-
-from __future__ import with_statement
 try:
     import cPickle as pickle
 except ImportError:
@@ -23,12 +22,6 @@
 # ===========================================================
 PY3 = sys.version_info[0] == 3
 
-# next:
-try:
-    next_ = next
-except NameError:
-    next_ = lambda obj: obj.next()
-
 # exec (from https://bitbucket.org/gutworth/six/):
 if PY3:
     import builtins
@@ -53,36 +46,10 @@ def exec_(_code_, _globs_=None, _locs_=None):
 def is_generator(f):
     """ Return True if a function is a generator.
     """
-    func_code = f.__code__ if PY3 else f.func_code
+    func_code = f.__code__
     isgen = (func_code.co_flags & CO_GENERATOR) != 0
     return isgen
 
-# Code to exec inside of LineProfiler.__call__ to support PEP-342-style
-# generators in Python 2.5+.
-pep342_gen_wrapper = '''
-def wrap_generator(self, func):
-    """ Wrap a generator to profile it.
-    """
-    def f(*args, **kwds):
-        g = func(*args, **kwds)
-        # The first iterate will not be a .send()
-        self.enable_by_count()
-        try:
-            item = next_(g)
-        finally:
-            self.disable_by_count()
-        input = (yield item)
-        # But any following one might be.
-        while True:
-            self.enable_by_count()
-            try:
-                item = g.send(input)
-            finally:
-                self.disable_by_count()
-            input = (yield item)
-    return f
-'''
-
 class LineProfiler(CLineProfiler):
     """ A profiler that records the execution times of individual lines.
     """
@@ -102,24 +69,27 @@ def __call__(self, func):
         f.__dict__.update(getattr(func, '__dict__', {}))
         return f
 
-    if sys.version_info[:2] >= (2,5):
-        # Delay compilation because the syntax is not compatible with older
-        # Python versions.
-        exec_(pep342_gen_wrapper)
-    else:
-        def wrap_generator(self, func):
-            """ Wrap a generator to profile it.
-            """
-            def f(*args, **kwds):
-                g = func(*args, **kwds)
-                while True:
-                    self.enable_by_count()
-                    try:
-                        item = next_(g)
-                    finally:
-                        self.disable_by_count()
-                    yield item
-            return f
+    def wrap_generator(self, func):
+        """ Wrap a generator to profile it.
+        """
+        def f(*args, **kwds):
+            g = func(*args, **kwds)
+            # The first iterate will not be a .send()
+            self.enable_by_count()
+            try:
+                item = next(g)
+            finally:
+                self.disable_by_count()
+            input = (yield item)
+            # But any following one might be.
+            while True:
+                self.enable_by_count()
+                try:
+                    item = g.send(input)
+                finally:
+                    self.disable_by_count()
+                input = (yield item)
+        return f
 
     def wrap_function(self, func):
         """ Wrap a function to profile it.
@@ -138,11 +108,8 @@ def dump_stats(self, filename):
         object from `get_stats()`.
         """
         lstats = self.get_stats()
-        f = open(filename, 'wb')
-        try:
+        with open(filename, 'wb') as f:
             pickle.dump(lstats, f, pickle.HIGHEST_PROTOCOL)
-        finally:
-            f.close()
 
     def print_stats(self, stream=None):
         """ Show the gathered statistics.
@@ -298,10 +265,7 @@ def magic_lprun(self, parameter_s=''):
     for name in opts.f:
         try:
             funcs.append(eval(name, global_ns, local_ns))
-        except Exception:
-            # "except Exception as e" is not supported in Python 2.5
-            # so we're using a hack to get the exception
-            e = sys.exc_info()[1]
+        except Exception as e:
             raise UsageError('Could not find function %r.\n%s: %s' % (name,
                 e.__class__.__name__, e))
 

setup.py

@@ -1,3 +1,4 @@
+#!/usr/bin/env python
 import os.path
 
 import distutils.errors
@@ -52,7 +53,6 @@
         "Programming Language :: C",
         "Programming Language :: Python",
         'Programming Language :: Python :: 2',
-        'Programming Language :: Python :: 2.5',
         'Programming Language :: Python :: 2.6',
         'Programming Language :: Python :: 2.7',
         'Programming Language :: Python :: 3',