]> git.bitcoin.ninja Git - youtube-dl/commitdiff
[compat] Do not compare None <= 0
authorPhilipp Hagemeister <phihag@phihag.de>
Tue, 6 Oct 2015 12:28:14 +0000 (14:28 +0200)
committerPhilipp Hagemeister <phihag@phihag.de>
Tue, 6 Oct 2015 12:30:43 +0000 (14:30 +0200)
The result is meaningless (and it emits a warning in cpython2 when called with -3), so handle None before making integer comparisons.

youtube_dl/compat.py

index c36c9c23ff633b82e837cc739e725e8e734f35fa..1ba4ab78cc44af89f5ff528d0db979a8dff5f3b8 100644 (file)
@@ -417,18 +417,18 @@ else:
     _terminal_size = collections.namedtuple('terminal_size', ['columns', 'lines'])
 
     def compat_get_terminal_size(fallback=(80, 24)):
-        columns = compat_getenv('COLUMNS', None)
+        columns = compat_getenv('COLUMNS')
         if columns:
             columns = int(columns)
         else:
             columns = None
-        lines = compat_getenv('LINES', None)
+        lines = compat_getenv('LINES')
         if lines:
             lines = int(lines)
         else:
             lines = None
 
-        if columns <= 0 or lines <= 0:
+        if columns is None or lines is None or columns <= 0 or lines <= 0:
             try:
                 sp = subprocess.Popen(
                     ['stty', 'size'],
@@ -438,9 +438,9 @@ else:
             except Exception:
                 _columns, _lines = _terminal_size(*fallback)
 
-            if columns <= 0:
+            if columns is None or columns <= 0:
                 columns = _columns
-            if lines <= 0:
+            if lines is None or lines <= 0:
                 lines = _lines
         return _terminal_size(columns, lines)