WWebRTC LUCI CQFormat /modules
c018bae8创建于 2023年4月20日历史提交
文件最后提交记录最后更新时间
Format /modules git ls-files | grep -e "\(\.h\|\.cc\)$" | grep -e "^modules/" | xargs clang-format -i ; git cl format after landing: add to .git-blame-ignore-revs Bug: webrtc:15082 Change-Id: I2c3cd28740062794f8c10e39d8406aadb9e9a35a Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/301620 Reviewed-by: Harald Alvestrand <hta@webrtc.org> Commit-Queue: Harald Alvestrand <hta@webrtc.org> Auto-Submit: Jared Siskin <jtsiskin@meta.com> Cr-Commit-Position: refs/heads/main@{#39901} 3 年前
Remove usage of sprintf in modules sprintf is marked as deprecated with Xcode 14. Bug: chromium:1331345 Change-Id: I834f392bee96e6b6725d5aee469a243dbc6e272e Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/265521 Reviewed-by: Henrik Andreassson <henrika@webrtc.org> Reviewed-by: Henrik Lundin <henrik.lundin@webrtc.org> Commit-Queue: Henrik Lundin <henrik.lundin@webrtc.org> Cr-Commit-Position: refs/heads/main@{#37162} 3 年前
Replace assert() with RTC_DCHECK(). CL partially auto-generated with: git grep -l "\bassert(" | grep "\.[c|h]" | \ xargs sed -i 's/\bassert(/RTC_DCHECK(/g' And with: git grep -l "RTC_DCHECK(false)" | \ xargs sed -i 's/RTC_DCHECK(false)/RTC_NOTREACHED()/g' With some manual changes to include "rtc_base/checks.h" where needed. A follow-up CL will remove assert() from Obj-C code as well and remove the #include of <assert.h>. The choice to replace with RTC_DCHECK is because assert() is because RTC_DCHECK has similar behavior as assert() based on NDEBUG. This CL also contains manual changes to switch from basic RTC_DCHECK to other (preferred) versions like RTC_DCHECK_GT (and similar). Bug: webrtc:6779 Change-Id: I00bed8886e03d685a2f42324e34aef2c9b7a63b0 Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/224846 Reviewed-by: Harald Alvestrand <hta@webrtc.org> Commit-Queue: Mirko Bonadei <mbonadei@webrtc.org> Cr-Commit-Position: refs/heads/master@{#34442} 4 年前
Remove top-level const from parameters in function declarations. This is a safe cleanup change since top-level const applied to parameters in function declarations (that are not also definitions) are ignored by the compiler. Hence, such changes do not change the type of the declared functions and are simply no-ops. Bug: webrtc:13610 Change-Id: Ibafb92c45119a6d8bdb6f9109aa8dad6385163a9 Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/249086 Reviewed-by: Niels Moller <nisse@webrtc.org> Reviewed-by: Harald Alvestrand <hta@webrtc.org> Commit-Queue: Ali Tofigh <alito@webrtc.org> Cr-Commit-Position: refs/heads/main@{#35802} 4 年前