)]}' { "commit": "da6291e85ddccb81b2b78685e2bb2946eacce63c", "tree": "84e1cb17671e9d5c5832226fed1364d8c0e96f11", "parents": [ "e94b175c421e1acb57e3bfd5c2668aa89fa59d61" ], "author": { "name": "Sam Zackrisson", "email": "saza@webrtc.org", "time": "Fri Aug 05 08:38:02 2022" }, "committer": { "name": "WebRTC LUCI CQ", "email": "webrtc-scoped@luci-project-accounts.iam.gserviceaccount.com", "time": "Fri Aug 05 10:10:47 2022" }, "message": "WebRTC voice engine: Remove duplicate and confusing logs\n\nThe line \"No audio processing module present [...]\" has mislead people several times (see linked bug for one example) and does not add any information that cannot already relatively easily be inferred from platform configuration.\n\nOther removed lines are duplicate (already logged via AudioOptions::ToString()) or never runs (ApplyOptions always returns true + empty #elif).\n\nBug: b/238780321#comment34\nChange-Id: Ie0fbd6675ec963c1180a7f614ec74bba5e850777\nReviewed-on: https://webrtc-review.googlesource.com/c/src/+/270483\nCommit-Queue: Sam Zackrisson \u003csaza@webrtc.org\u003e\nReviewed-by: Alessio Bazzica \u003calessiob@webrtc.org\u003e\nCr-Commit-Position: refs/heads/main@{#37697}\n", "tree_diff": [ { "type": "modify", "old_id": "831e1f5debd75635337506fb2ea1bd1090058509", "old_mode": 33188, "old_path": "media/engine/webrtc_voice_engine.cc", "new_id": "03d9537a6906eb247f877e9d78c6daeeac534b9b", "new_mode": 33188, "new_path": "media/engine/webrtc_voice_engine.cc" }, { "type": "modify", "old_id": "e55a0a34632cd6d4e5a862f84957f10042febfd9", "old_mode": 33188, "old_path": "media/engine/webrtc_voice_engine.h", "new_id": "9cb7ec82eb543eea5e1b98a7c60c76d2c7749bdb", "new_mode": 33188, "new_path": "media/engine/webrtc_voice_engine.h" } ] }