Chromium Code Reviews
|
|
Chromium Code Reviews
Issue
2291283002:
roll libyuv from r1610 to r1616 (Closed)
|
Created:
4 years, 3 months ago by fbarchard1 Modified:
4 years, 3 months ago CC:
chromium-reviews Target Ref:
refs/pending/heads/master Project:
chromium Visibility:
Public. |
Descriptionroll libyuv from r1610 to r1616
Fix for ltcg on VS2012 and earlier - extern declspec must match declaration.
Fixes suggested by mozilla for android/mozilla warnings.
SplitUVPlane and MergeUVPlane functions to allow lighter weight chroma conversions for NV12 and NV21 to/from I420.
Change log:
https://chromium.googlesource.com/libyuv/libyuv/+log/74491ba0..de944ed8
https://chromium.googlesource.com/libyuv/libyuv/+/74491ba0..de944ed8
Includes a fix for ltcg on VS2013 and earlier.
BUG=libyuv:633, libyuv:630, libyuv:629, libyuv:626, libyuv:631
Committed: https://crrev.com/7dcda66fc01e6ccf02855335f8c605c153c07226
Cr-Commit-Position: refs/heads/master@{#415427}
Patch Set 1 #Patch Set 2 : libyuv roll from r1616 to r1619 #Messages
Total messages: 19 (14 generated)
The CQ bit was checked by fbarchard@google.com to run a CQ dry run
Dry run: CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...
brettw@chromium.org changed reviewers: + brettw@chromium.org
lgtm
Description was changed from ========== roll libyuv to r1616 Includes a fix for ltcg on VS2013 and earlier. BUG=libyuv:633 ========== to ========== roll libyuv to r1616 Change log: https://chromium.googlesource.com/libyuv/libyuv/+log/74491ba0..de944ed8 https://chromium.googlesource.com/libyuv/libyuv/+/74491ba0..de944ed8 Includes a fix for ltcg on VS2013 and earlier. BUG=libyuv:633 ==========
Description was changed from ========== roll libyuv to r1616 Change log: https://chromium.googlesource.com/libyuv/libyuv/+log/74491ba0..de944ed8 https://chromium.googlesource.com/libyuv/libyuv/+/74491ba0..de944ed8 Includes a fix for ltcg on VS2013 and earlier. BUG=libyuv:633 ========== to ========== roll libyuv from r1610 to r1616 Fix for ltcg on VS2012 and earlier - extern declspec must match declaration. Fixes suggested by mozilla for android/mozilla warnings. SplitUVPlane and MergeUVPlane functions to allow lighter weight chroma conversions for NV12 and NV21 to/from I420. Change log: https://chromium.googlesource.com/libyuv/libyuv/+log/74491ba0..de944ed8 https://chromium.googlesource.com/libyuv/libyuv/+/74491ba0..de944ed8 Includes a fix for ltcg on VS2013 and earlier. BUG=libyuv:633 ==========
kjellander@chromium.org changed reviewers: + kjellander@chromium.org
lgtm
The CQ bit was unchecked by commit-bot@chromium.org
Dry run: This issue passed the CQ dry run.
The CQ bit was checked by fbarchard@google.com
The CQ bit was unchecked by fbarchard@google.com
Description was changed from ========== roll libyuv from r1610 to r1616 Fix for ltcg on VS2012 and earlier - extern declspec must match declaration. Fixes suggested by mozilla for android/mozilla warnings. SplitUVPlane and MergeUVPlane functions to allow lighter weight chroma conversions for NV12 and NV21 to/from I420. Change log: https://chromium.googlesource.com/libyuv/libyuv/+log/74491ba0..de944ed8 https://chromium.googlesource.com/libyuv/libyuv/+/74491ba0..de944ed8 Includes a fix for ltcg on VS2013 and earlier. BUG=libyuv:633 ========== to ========== roll libyuv from r1610 to r1616 Fix for ltcg on VS2012 and earlier - extern declspec must match declaration. Fixes suggested by mozilla for android/mozilla warnings. SplitUVPlane and MergeUVPlane functions to allow lighter weight chroma conversions for NV12 and NV21 to/from I420. Change log: https://chromium.googlesource.com/libyuv/libyuv/+log/74491ba0..de944ed8 https://chromium.googlesource.com/libyuv/libyuv/+/74491ba0..de944ed8 Includes a fix for ltcg on VS2013 and earlier. BUG=libyuv:633,libyuv:630,libyuv:629,libyuv:626,libyuv:631 ==========
The CQ bit was checked by fbarchard@google.com
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...
Message was sent while issue was closed.
Description was changed from ========== roll libyuv from r1610 to r1616 Fix for ltcg on VS2012 and earlier - extern declspec must match declaration. Fixes suggested by mozilla for android/mozilla warnings. SplitUVPlane and MergeUVPlane functions to allow lighter weight chroma conversions for NV12 and NV21 to/from I420. Change log: https://chromium.googlesource.com/libyuv/libyuv/+log/74491ba0..de944ed8 https://chromium.googlesource.com/libyuv/libyuv/+/74491ba0..de944ed8 Includes a fix for ltcg on VS2013 and earlier. BUG=libyuv:633,libyuv:630,libyuv:629,libyuv:626,libyuv:631 ========== to ========== roll libyuv from r1610 to r1616 Fix for ltcg on VS2012 and earlier - extern declspec must match declaration. Fixes suggested by mozilla for android/mozilla warnings. SplitUVPlane and MergeUVPlane functions to allow lighter weight chroma conversions for NV12 and NV21 to/from I420. Change log: https://chromium.googlesource.com/libyuv/libyuv/+log/74491ba0..de944ed8 https://chromium.googlesource.com/libyuv/libyuv/+/74491ba0..de944ed8 Includes a fix for ltcg on VS2013 and earlier. BUG=libyuv:633,libyuv:630,libyuv:629,libyuv:626,libyuv:631 ==========
Message was sent while issue was closed.
Committed patchset #1 (id:1)
Message was sent while issue was closed.
Description was changed from ========== roll libyuv from r1610 to r1616 Fix for ltcg on VS2012 and earlier - extern declspec must match declaration. Fixes suggested by mozilla for android/mozilla warnings. SplitUVPlane and MergeUVPlane functions to allow lighter weight chroma conversions for NV12 and NV21 to/from I420. Change log: https://chromium.googlesource.com/libyuv/libyuv/+log/74491ba0..de944ed8 https://chromium.googlesource.com/libyuv/libyuv/+/74491ba0..de944ed8 Includes a fix for ltcg on VS2013 and earlier. BUG=libyuv:633,libyuv:630,libyuv:629,libyuv:626,libyuv:631 ========== to ========== roll libyuv from r1610 to r1616 Fix for ltcg on VS2012 and earlier - extern declspec must match declaration. Fixes suggested by mozilla for android/mozilla warnings. SplitUVPlane and MergeUVPlane functions to allow lighter weight chroma conversions for NV12 and NV21 to/from I420. Change log: https://chromium.googlesource.com/libyuv/libyuv/+log/74491ba0..de944ed8 https://chromium.googlesource.com/libyuv/libyuv/+/74491ba0..de944ed8 Includes a fix for ltcg on VS2013 and earlier. BUG=libyuv:633,libyuv:630,libyuv:629,libyuv:626,libyuv:631 Committed: https://crrev.com/7dcda66fc01e6ccf02855335f8c605c153c07226 Cr-Commit-Position: refs/heads/master@{#415427} ==========
Message was sent while issue was closed.
Patchset 1 (id:??) landed as https://crrev.com/7dcda66fc01e6ccf02855335f8c605c153c07226 Cr-Commit-Position: refs/heads/master@{#415427} |
