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

Issue 1284423006: rtpAnalyze matlab tool: filter out RTCP packets (Closed)

Created:
5 years, 4 months ago by hlundin-webrtc
Modified:
5 years, 4 months ago
Reviewers:
tlegrand-webrtc
CC:
webrtc-reviews_webrtc.org
Base URL:
https://chromium.googlesource.com/external/webrtc.git@master
Target Ref:
refs/pending/heads/master
Project:
webrtc
Visibility:
Public.

Description

rtpAnalyze matlab tool: filter out RTCP packets This change relates to the matlab tool rtpAnalyze. With this change, RTP packets with payload types 72 through 76 are removed. In IETF RFC3551, section "Payload Type Definitions", this range is marked as reserved so that RTCP and RTP packets can be reliably distinguished. BUG=webrtc:2692 TBR=tina.legrand@webrtc.org NOTRY=true Committed: https://crrev.com/d84dcbd2ecd28bd57e1f17bc00dc6390cbbd0cda Cr-Commit-Position: refs/heads/master@{#9724}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+12 lines, -0 lines) Patch
M tools/matlab/rtpAnalyze.m View 1 chunk +12 lines, -0 lines 0 comments Download

Messages

Total messages: 4 (1 generated)
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1284423006/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1284423006/1
5 years, 4 months ago (2015-08-18 11:46:06 UTC) #2
commit-bot: I haz the power
Committed patchset #1 (id:1)
5 years, 4 months ago (2015-08-18 11:46:49 UTC) #3
commit-bot: I haz the power
5 years, 4 months ago (2015-08-18 11:46:58 UTC) #4
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/d84dcbd2ecd28bd57e1f17bc00dc6390cbbd0cda
Cr-Commit-Position: refs/heads/master@{#9724}

Powered by Google App Engine
This is Rietveld 408576698