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

Unified Diff: platform_tools/android/bin/gyp_to_android.py

Issue 142173002: Add self tests for gyp_to_android. (Closed) Base URL: https://skia.googlesource.com/skia.git@GYP2
Patch Set: Created 6 years, 11 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 | tools/tests/gyp_to_android_tests.py » ('j') | tools/tests/gyp_to_android_tests.py » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: platform_tools/android/bin/gyp_to_android.py
diff --git a/platform_tools/android/bin/gyp_to_android.py b/platform_tools/android/bin/gyp_to_android.py
index 9d887c4c2d5320a0c675f7eb1a4c189d33ee5214..1e789e912bb83303d121a4fb0af962a1e8ec3262 100644
--- a/platform_tools/android/bin/gyp_to_android.py
+++ b/platform_tools/android/bin/gyp_to_android.py
@@ -413,6 +413,7 @@ def main():
WriteAndroidMk(common, arm_var_dict, arm_neon_var_dict, x86_var_dict,
default_var_dict)
+ return 0
epoger 2014/01/21 20:00:51 How would main() ever return anything but 0?
scroggo 2014/01/22 16:36:48 It would not. Removed.
scroggo 2014/01/22 19:32:50 D'oh! I thought I removed it. Removed in next patc
if __name__ == '__main__':
main()
« no previous file with comments | « no previous file | tools/tests/gyp_to_android_tests.py » ('j') | tools/tests/gyp_to_android_tests.py » ('J')

Powered by Google App Engine
This is Rietveld 408576698