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

Issues Search
    My Issues | Starred     Open | Closed | All

Issue 1557633002: srtp: Fix a warning. (Closed)

Created:
4 years, 11 months ago by Nico
Modified:
4 years, 11 months ago
Reviewers:
joachim, jiayl
CC:
chromium-reviews
Base URL:
https://chromium.googlesource.com/chromium/deps/libsrtp.git@master
Target Ref:
refs/heads/master
Visibility:
Public.

Description

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+4 lines, -1 line) Patch
M README.chromium View 1 chunk +3 lines, -0 lines 0 comments Download
M srtp/test/replay_driver.c View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 8 (3 generated)
Nico
First to stamp wins!
4 years, 11 months ago (2015-12-30 20:12:43 UTC) #2
Nico
Committed patchset #1 (id:1) manually as 8a7662a400df7bdb0dc5cb7c0bf890dfa3481295 (tree was closed).
4 years, 11 months ago (2015-12-30 22:47:48 UTC) #5
Nico
I tbr'd this since it's a simple fix that's blocking a largeish patch. If you ...
4 years, 11 months ago (2015-12-30 22:48:23 UTC) #6
joachim
On 2015/12/30 22:48:23, Nico wrote: > I tbr'd this since it's a simple fix that's ...
4 years, 11 months ago (2016-01-01 18:31:36 UTC) #7
joachim
4 years, 11 months ago (2016-01-01 18:33:04 UTC) #8
Message was sent while issue was closed. 
On 2016/01/01 18:31:36, joachim wrote:
> On 2015/12/30 22:48:23, Nico wrote:
> > I tbr'd this since it's a simple fix that's blocking a largeish patch. If
you
> > have comments I'll address them in a follow-up. Thanks!
> 
> lgtm (if my vote counts), I'll look into submitting this upstream.

Whoops, just noticed you already did this...

Powered by Google App Engine
This is Rietveld 408576698