alexfh accepted this revision.
alexfh added a comment.
This revision is now accepted and ready to land.

Please don't forget go add cfe-commits to subscribers when you create a 
differential revision.

LG with one comment.



================
Comment at: clang-tidy/tool/run-clang-tidy.py:242
     # Spin up a bunch of tidy-launching threads.
-    queue = Queue.Queue(max_task)
+    q = queue.Queue(max_task)
     for _ in range(max_task):
----------------
let's give it a more meaningful name, e.g. file_queue or task_queue.


https://reviews.llvm.org/D37138



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [PATCH] D37138: Make r... Alexander Kornienko via Phabricator via cfe-commits

Reply via email to