mirror of
https://github.com/mollyim/webrtc.git
synced 2025-05-19 00:27:51 +01:00
fc52b912a3
Since there is no way to enable/disable these diagnostics at runtime, this CL moves the suppression into the rtc_* templates in order to remove the need to explicitly add the snippet of code needed to suppress it (currently copy/pasted in 144 locations). The diagnostic that causes the most problems is the one about "complex class/struct explicit ctor/dtor" [1] because WebRTC doesn't find it useful enough. Other diagnostics are good (for example the one that warns about using "virtual" instead of "override", but that will be covered by this clang-tidy check [2]) while others are Chromium related so they have never triggered. [1] - https://cs.chromium.org/chromium/src/tools/clang/plugins/FindBadConstructsConsumer.cpp?l=147-167&rcl=b4bebe1aa15dba7ca5fcc6456a81a55665327c3a [2] - https://clang.llvm.org/extra/clang-tidy/checks/modernize-use-override.html Bug: webrtc:163 Change-Id: Icbf27efa5b369100a31e6a32df1a0913729b3b34 Reviewed-on: https://webrtc-review.googlesource.com/c/125088 Commit-Queue: Mirko Bonadei <mbonadei@webrtc.org> Reviewed-by: Karl Wiberg <kwiberg@webrtc.org> Cr-Commit-Position: refs/heads/master@{#26918} |
||
|---|---|---|
| .. | ||
| androidapp | ||
| BUILD.gn | ||
| config_reader.cc | ||
| config_reader.h | ||
| create_network_tester_config.py | ||
| jni.cpp | ||
| network_tester_config.proto | ||
| network_tester_packet.proto | ||
| network_tester_unittest.cc | ||
| packet_logger.cc | ||
| packet_logger.h | ||
| packet_sender.cc | ||
| packet_sender.h | ||
| parse_packet_log.py | ||
| README | ||
| server.cc | ||
| test_controller.cc | ||
| test_controller.h | ||
This file explains how to run the network tester and how to analyze the results. create config files =================== use create_network_tester_config.py to create config files. you can add or change the AddConfig call in the main function to create a the desired network config. run network_tester_server ========================= place the network config file next to the server binary and name it "server_config.dat". the network_tester_server should run on a server with a public IP address. the log file of network_tester_server will be created next to the binary with the name "server_packet_log.dat" run NetworkTesterMobile (apk) ============================= change the IP and the port in the jni.cpp file and set it to the port and IP you run the network_tester_server on. place the network config file as "/mnt/sdcard/network_tester_client_config.dat" on the tester android phone. The log file of network_tester_server will be created at "/mnt/sdcard/network_tester_client_packet_log.dat" analyze the results ==================== run "python parse_packet_log.py -f <log_file_to_parse>" to analyze the log results.