Closure compilation config: switch from JSON to Python dict

third_party/closure_compiler/compile_modules.py

 #!/usr/bin/env python
 # Copyright 2014 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 import argparse
+import ast
 from checker import Checker as Checker
 import os
-try:
-  import json
-except:
-  import simplejson as json
-
 
 
 class Module(object):
   def __init__(self, name, sources, depends=[], externs=[]):
     self.name = name
     self.sources = sources
     # TODO(dbeam): support depending on other modules/dependency flattening.
     self.depends = depends
     self.externs = externs
 
@@ skipping 19 matching lines @@
 
   def __init__(self, verbose=False):
     self._verbose = verbose
 
   def parse(self, file_path):
     if file_path in self._cache:
       print "(INFO) Found module file %s in the cache" % file_path
       return self._cache[file_path]
 
     file = open(file_path, "r")
-    data = json.load(file)
+    file_content = file.read()

[Dan Beam, 2014/07/24 18:34:42]

nit: file_contents = open(file_path, "r").read()

[Vitaly Pavlenko, 2014/07/25 22:09:22]

I think that's better:
with open(file_path) as file:
  file_contents = file.read()

I mean, it's unspecified when the opened file should be closed in your case. Of
course, that's not a problem for small scripts, but I could think the general
trend in guides about that should be like I said. What do you think?

+    data = ast.literal_eval(file_content)
     file.close()
 
     if self._verbose:
-      pretty_json = json.dumps(data, indent=2, separators=(',', ': ')).strip()
-      print "(INFO) Layout: " + os.linesep + pretty_json + os.linesep
+      print "(INFO) Layout: " + os.linesep + file_content + os.linesep
 
     self._cache[file_path] = [Module.from_dict(m) for m in data]
     return self._cache[file_path]
 
 
 class ModuleCompiler(object):
   _checker = None
   _parser = None
 
   def __init__(self, verbose=False):
@@ skipping 45 matching lines @@
 
 
 if __name__ == "__main__":
   parser = argparse.ArgumentParser(
       description="Typecheck JavaScript using Closure compiler")
   parser.add_argument("-v", "--verbose", action="store_true",
                       help="Show more information as this script runs")
   parser.add_argument("module_file", nargs=argparse.ONE_OR_MORE,
                       help="Path to a modules file to check")
   main(parser.parse_args())