Chromium Code Reviews

Issue 2589773002: CodecInst operator<< (Closed)

Created:
4 years ago by kwiberg-webrtc
Modified:
4 years ago
Reviewers:
ossu
CC:
webrtc-reviews_webrtc.org, interface-changes_webrtc.org, yujie_mao (webrtc), tterriberry_mozilla.com, qiang.lu, niklas.enbom, peah-webrtc
Target Ref:
refs/heads/master
Project:
webrtc
Visibility:
Public.

Description

CodecInst operator<< I always find myself re-writing this function for debug purposes. It'd save so much time if it already existed... BUG=none Review-Url: https://codereview.webrtc.org/2589773002 Cr-Commit-Position: refs/heads/master@{#15677} Committed: https://chromium.googlesource.com/external/webrtc/+/5bf4e0810cf4b1580726596e5d3d8bd44e018b55

Patch Set 1 #

Unified diffs Side-by-side diffs Stats (+11 lines, -0 lines)
M webrtc/common_types.h View 2 chunks +11 lines, -0 lines 0 comments

Messages

Total messages: 12 (8 generated)
kwiberg-webrtc
4 years ago (2016-12-19 13:52:20 UTC) #6
ossu
Oh, go ahead! It's almost Christmas! lgtm
4 years ago (2016-12-19 13:59:24 UTC) #7
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/2589773002/1
4 years ago (2016-12-19 14:02:30 UTC) #9
commit-bot: I haz the power
4 years ago (2016-12-19 14:04:07 UTC) #12
Message was sent while issue was closed.
Committed patchset #1 (id:1) as
https://chromium.googlesource.com/external/webrtc/+/5bf4e0810cf4b1580726596e5...

Powered by Google App Engine