[Python-checkins] CVS: python/dist/src/Tools/idle OutputWindow.py,1.3,1.4

Guido van Rossum guido@cnri.reston.va.us
Tue, 7 Mar 2000 11:25:13 -0500 (EST)


Update of /projects/cvsroot/python/dist/src/Tools/idle
In directory eric:/projects/python/develop/guido/src/Tools/idle

Modified Files:
	OutputWindow.py 
Log Message:
Tweak the goto file/line command (in the right button menu in PyShell
and output windows) so that it if it doesn't succeed with the line
at the cursor, it tries the line before that.  This is handy with
tracebacks, where my natural tendency is to click in the displayed
source line rather than in the file/line indicator just above it.
Now I can indulge this tendency.

I factored out a helper and changed the error handling so that a
non-existing file is treated as if the line didn't match -- this is
handy because some function calls (e.g. "foo.bar(1)") match the grep
pattern.


Index: OutputWindow.py
===================================================================
RCS file: /projects/cvsroot/python/dist/src/Tools/idle/OutputWindow.py,v
retrieving revision 1.3
retrieving revision 1.4
diff -C2 -r1.3 -r1.4
*** OutputWindow.py	1999/04/22 22:28:42	1.3
--- OutputWindow.py	2000/03/07 16:25:11	1.4
***************
*** 68,71 ****
--- 68,90 ----
          # self.text.mark_set("insert", "@%d,%d" % (x, y))
          line = self.text.get("insert linestart", "insert lineend")
+         result = self._file_line_helper(line)
+         if not result:
+             # Try the previous line.  This is handy e.g. in tracebacks,
+             # where you tend to right-click on the displayed source line
+             line = self.text.get("insert -1line linestart",
+                                  "insert -1line lineend")
+             result = self._file_line_helper(line)
+             if not result:
+                 tkMessageBox.showerror(
+                     "No special line",
+                     "The line you point at doesn't look like "
+                     "a valid file name followed by a line number.",
+                     master=self.text)
+                 return
+         filename, lineno = result
+         edit = self.flist.open(filename)
+         edit.gotoline(lineno)
+ 
+     def _file_line_helper(self, line):
          for prog in self.file_line_progs:
              m = prog.search(line)
***************
*** 73,95 ****
                  break
          else:
!             tkMessageBox.showerror("No special line",
!                 "The line you point at doesn't look like "
!                 "a file name followed by a line number.",
!                 master=self.text)
!             return
          filename, lineno = m.group(1, 2)
          try:
              f = open(filename, "r")
              f.close()
!         except IOError, msg:
!             self.text.bell()
!             return
!         edit = self.flist.open(filename)
          try:
!             lineno = int(lineno)
!         except ValueError, msg:
!             self.text.bell()
!             return
!         edit.gotoline(lineno)
  
  # These classes are currently not used but might come in handy
--- 92,106 ----
                  break
          else:
!             return None
          filename, lineno = m.group(1, 2)
          try:
              f = open(filename, "r")
              f.close()
!         except IOError:
!             return None
          try:
!             return filename, int(lineno)
!         except TypeError:
!             return None
  
  # These classes are currently not used but might come in handy