Rev 4767: Revert incomplete fix for bug #353370 in http://bazaar.launchpad.net/~vila/bzr/integration

[Vincent Ladeuil]

At http://bazaar.launchpad.net/~vila/bzr/integration

------------------------------------------------------------
revno: 4767
revision-id: v.ladeuil+lp at free.fr-20091023163103-x1h0sbrz61r0bqzx
parent: pqm at pqm.ubuntu.com-20091022162721-r0to3zua8p8oprq1
committer: Vincent Ladeuil <v.ladeuil+lp at free.fr>
branch nick: integration
timestamp: Fri 2009-10-23 18:31:03 +0200
message:
  Revert incomplete fix for bug #353370
-------------- next part --------------
=== modified file 'NEWS'
--- a/NEWS	2009-10-22 08:49:42 +0000
+++ b/NEWS	2009-10-23 16:31:03 +0000
@@ -25,10 +25,6 @@
 * ``bzr+http`` servers no longer give spurious jail break errors when
   serving branches inside a shared repository.  (Andrew Bennetts, #348308)
 
-* ``osutils.terminal_width()`` will now return a very large value
-  to avoid truuncated lines when using pagers (for example).
-  (Joke de Buhr, Vincent Ladeuil, #353370)
-
 * TreeTransform.adjust_path updates the limbo paths of descendants of adjusted
   files.  (Aaron Bentley)
 

=== modified file 'bzrlib/osutils.py'
--- a/bzrlib/osutils.py	2009-10-22 15:19:59 +0000
+++ b/bzrlib/osutils.py	2009-10-23 16:31:03 +0000
@@ -1296,15 +1296,6 @@
 
 def terminal_width():
     """Return estimated terminal width."""
-    isatty = getattr(sys.stdout, 'isatty', None)
-    if  isatty is None or not isatty():
-        # If it's not a tty, the width makes no sense. We just use a value bug
-        # enough to avoid truncations. When the output is redirected, the
-        # pagers can then handle that themselves. A cleaner implementation
-        # would be to fix the callers to not try to format at all in these
-        # circumstances.
-        return 65536
-
     if sys.platform == 'win32':
         return win32utils.get_console_size()[0]
     width = 0