admin/webrtc_m130
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Removing some redundant ostringstreams declarations.

Browse Source 
These shadow a variable in an exterior scope, and cause unneeded overhead.

R=pthatcher@webrtc.org

Review URL: https://codereview.webrtc.org/1714843003 .

Cr-Commit-Position: refs/heads/master@{#11725}
This commit is contained in:
Taylor Brandstetter 2016-02-23 12:58:15 -08:00
parent 71d9721bdc
commit 9788534c77
1 changed files with 0 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
webrtc/api/webrtcsdp.cc
View File

@ -1402,7 +1402,6 @@ void BuildRtpContentAttributes(const MediaContentDescription* media_desc,
} else {
auto track = media_desc->streams().begin();
const std::string& stream_id = track->sync_label;
std::ostringstream os;
InitAttrLine(kAttributeMsid, &os);
os << kSdpDelimiterColon << stream_id << kSdpDelimiterSpace << track->id;
AddLine(os.str(), message);
@ -1457,7 +1456,6 @@ void BuildRtpContentAttributes(const MediaContentDescription* media_desc,
if (track->ssrc_groups[i].ssrcs.empty()) {
continue;
}
std::ostringstream os;
InitAttrLine(kAttributeSsrcGroup, &os);
os << kSdpDelimiterColon << track->ssrc_groups[i].semantics;
std::vector<uint32_t>::const_iterator ssrc =
@ -1480,7 +1478,6 @@ void BuildRtpContentAttributes(const MediaContentDescription* media_desc,
// The appdata consists of the "id" attribute of a MediaStreamTrack,
// which corresponds to the "id" attribute of StreamParams.
const std::string& stream_id = track->sync_label;
std::ostringstream os;
InitAttrLine(kAttributeSsrc, &os);
os << kSdpDelimiterColon << ssrc << kSdpDelimiterSpace
<< kSsrcAttributeMsid << kSdpDelimiterColon << stream_id