[PATCH] D69054: [LNT] Remove useless import fallback

Thomas Preud'homme via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 16 09:41:29 PDT 2019


thopre created this revision.
thopre added reviewers: cmatthews, hubert.reinterpretcast, kristof.beyls.
thopre added a parent revision: D69053: [LNT] Switch docker image to Python 3.

json module is available since Python 2.6. Since LNT now requires Python
2.7 or later, the fallback for importing simplejson in case json is not
available is useless. Removing it also removes one of the warning from
mypy.


https://reviews.llvm.org/D69054

Files:
  lnt/testing/__init__.py


Index: lnt/testing/__init__.py
===================================================================
--- lnt/testing/__init__.py
+++ lnt/testing/__init__.py
@@ -7,14 +7,10 @@
 """
 
 import datetime
+import json
 import re
 from lnt.util import logger
 
-try:
-    import json
-except ImportError:
-    import simplejson as json
-
 # We define the following constants for use as sample values by
 # convention.
 PASS = 0


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D69054.225250.patch
Type: text/x-patch
Size: 419 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20191016/bc8836e0/attachment-0001.bin>


More information about the llvm-commits mailing list