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

Issue 66503002: Updating Opus 1.1 Beta (Closed)

Created:
7 years, 1 month ago by tlegrand1
Modified:
7 years ago
Reviewers:
Sergey Ulanov
CC:
chromium-reviews
Base URL:
https://chromium.googlesource.com/chromium/deps/opus.git@master
Visibility:
Public.

Description

Updating Opus 1.1 Beta Removing two files not needed, that where missing license headers. Now passes src/tools/checklicenses/checklicenses.py. BUG=321557 R=sergeyu@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=237544

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -78 lines) Patch
D autogen.sh View 1 chunk +0 lines, -11 lines 0 comments Download
D tests/test_opus_padding.c View 1 chunk +0 lines, -67 lines 0 comments Download

Messages

Total messages: 8 (0 generated)
tlegrand1
Hi Sergey, The following two files have the wrong license headers, but since they are ...
7 years, 1 month ago (2013-11-19 13:35:51 UTC) #1
Sergey Ulanov
LGTM. Is there a bug that tracks checklicenses for Opus?
7 years, 1 month ago (2013-11-20 04:47:21 UTC) #2
tlegrand1
On 2013/11/20 04:47:21, Sergey Ulanov wrote: > LGTM. Is there a bug that tracks checklicenses ...
7 years, 1 month ago (2013-11-20 10:18:39 UTC) #3
tlegrand1
Committed patchset #1 manually as r8a67201 (presubmit successful).
7 years, 1 month ago (2013-11-20 12:08:05 UTC) #4
tlegrand1
On 2013/11/20 12:08:05, tlegrand1 wrote: > Committed patchset #1 manually as r8a67201 (presubmit successful). I ...
7 years, 1 month ago (2013-11-21 10:13:51 UTC) #5
Sergey Ulanov
On 2013/11/21 10:13:51, tlegrand1 wrote: > On 2013/11/20 12:08:05, tlegrand1 wrote: > > Committed patchset ...
7 years, 1 month ago (2013-11-22 00:51:05 UTC) #6
cmp
On 2013/11/22 00:51:05, Sergey Ulanov wrote: > On 2013/11/21 10:13:51, tlegrand1 wrote: > > On ...
7 years ago (2013-11-24 18:59:56 UTC) #7
tlegrand1
7 years ago (2013-11-27 08:37:36 UTC) #8
Message was sent while issue was closed.
Committed patchset #1 manually as r237544 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698