New rtc dump analyzing tool in Python

tools/py_event_log_analyzer/copy_files.py

+#  Copyright (c) 2016 The WebRTC project authors. All Rights Reserved.
+#
+#  Use of this source code is governed by a BSD-style license
+#  that can be found in the LICENSE file in the root of the source
+#  tree. An additional intellectual property rights grant can be found
+#  in the file PATENTS.  All contributing project authors may
+#  be found in the AUTHORS file in the root of the source tree.
+
+"""File copying script for OS compatibility. Used when building."""
+
+import optparse
+import os
+import shutil
+
+
+def main():
+  parser = optparse.OptionParser()
+  parser.add_option("--input_dir", help="directory with *py files")
+  parser.add_option("--output_dir", help="where to copy *py files")
+  (options, _) = parser.parse_args()

[phoglund, 2016/06/03 08:11:07]

Nit: remove ( ).

[aleloi, 2016/06/09 12:00:44]

The standings among reviewers are currently 1-1 for parentheses vs. no
parentheses. The style guide says "use parentheses sparingly", but it also seems
to encourage parentheses around tuples rather than discourage them. Personally,
I think it adds readability.

Ref to style guide:
https://engdoc.corp.google.com/eng/doc/devguide/py/styleguide.shtml?cl=head#s...

[kwiberg-webrtc, 2016/06/09 12:27:20]

If it helps resolve the impasse, I can change my +1 to a +2. :-) Parentheses
around tuples are good.

+
+  for file_name in os.listdir(options.input_dir):
+    if file_name.endswith(".py"):
+      copy_file(file_name, options.input_dir, options.output_dir)
+
+
+def copy_file(file_name, path_from, path_to):
+  shutil.copyfile(os.path.join(path_from, file_name),
+                  os.path.join(path_to, file_name))
+
+if __name__ == "__main__":
+  main()