[pypy-commit] pypy default: Backed out changeset 44c4e9c635ea: this breaks the loading of cource code in

antocuni noreply at buildbot.pypy.org
Thu Nov 29 15:57:07 CET 2012


Author: Antonio Cuni <anto.cuni at gmail.com>
Branch: 
Changeset: r59122:3282301eb9af
Date: 2012-11-29 14:54 +0000
http://bitbucket.org/pypy/pypy/changeset/3282301eb9af/

Log:	Backed out changeset 44c4e9c635ea: this breaks the loading of cource
	code in the jitviewer: it's better for directly implement the
	feature there

diff --git a/pypy/tool/jitlogparser/parser.py b/pypy/tool/jitlogparser/parser.py
--- a/pypy/tool/jitlogparser/parser.py
+++ b/pypy/tool/jitlogparser/parser.py
@@ -1,5 +1,4 @@
 import re, sys
-import os
 
 from pypy.jit.metainterp.resoperation import opname
 from pypy.jit.tool.oparser import OpParser
@@ -161,27 +160,19 @@
             if op.name == 'debug_merge_point':
                 self.inline_level = int(op.args[0])
                 parsed = parse_code_data(op.args[2][1:-1])
-                (self.name, self.bytecode_name, filename,
+                (self.name, self.bytecode_name, self.filename,
                  self.startlineno, self.bytecode_no) = parsed
                 break
         else:
             self.inline_level = 0
             parsed = parse_code_data(loopname)
-            (self.name, self.bytecode_name, filename,
+            (self.name, self.bytecode_name, self.filename,
              self.startlineno, self.bytecode_no) = parsed
-        self.filename = self.normalize_filename(filename)
         self.operations = operations
         self.storage = storage
-        self.code = storage.disassemble_code(filename, self.startlineno,
+        self.code = storage.disassemble_code(self.filename, self.startlineno,
                                              self.name)
 
-    def normalize_filename(self, filename):
-        home = os.path.expanduser('~') + os.path.sep
-        if filename is not None and filename.startswith(home):
-            filename = filename.replace(home, '~' + os.path.sep)
-        return filename
-
-
     def repr(self):
         if self.filename is None:
             return self.bytecode_name
diff --git a/pypy/tool/jitlogparser/test/test_parser.py b/pypy/tool/jitlogparser/test/test_parser.py
--- a/pypy/tool/jitlogparser/test/test_parser.py
+++ b/pypy/tool/jitlogparser/test/test_parser.py
@@ -354,14 +354,3 @@
     f = Function.from_operations(loop.operations, LoopStorage())
     assert f.chunks[-1].filename == 'x.py'
     assert f.filename is None
-
-def test_normalize_filename():
-    import os.path
-    abspath = os.path.expanduser('~/foo.py')
-    ops = parse('''
-    [i0]
-    debug_merge_point(0, 0, "<code object stuff. file '%s'. line 200> #10 ADD")
-    ''' % abspath)
-    res = Function.from_operations(ops.operations, LoopStorage())
-    assert res.filename == '~/foo.py'
-    assert res.chunks[0].filename == '~/foo.py'


More information about the pypy-commit mailing list