Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(383)

Unified Diff: webrtc/rtc_tools/video_analysis.py

Issue 2966743002: fix comment length (Closed)
Patch Set: Created 3 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webrtc/rtc_tools/video_analysis.py
diff --git a/webrtc/rtc_tools/video_analysis.py b/webrtc/rtc_tools/video_analysis.py
index 885c2f542b653752984e033d0a3716f8f1d65844..0a4a8eb640930f750ce3b22e702e492b3489fc33 100755
--- a/webrtc/rtc_tools/video_analysis.py
+++ b/webrtc/rtc_tools/video_analysis.py
@@ -408,7 +408,8 @@ def FlipAndCropRecordings(options, test_file_name, test_file_location,
def CompareVideos(options, cropped_ref_file, cropped_test_file):
- """Runs the compare_video.py script from src/webrtc/rtc_tools using the file path.
+ """Runs the compare_video.py script from src/webrtc/rtc_tools using the file
mbonadei 2017/06/30 11:58:15 I think this violates coding standards. It should
kjellander_webrtc 2017/06/30 12:05:16 Yepp, first line should fit on one line. Anything
+ path.
Uses the path from recording_result and writes the output to a file named
<options.app_name + '_' + CURRENT_TIME + '_result.txt> in the reference video
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698