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

Issue 1574253004: refactor ARGBToI422 using ARGBToI420 internally (Closed)

Created:
4 years, 11 months ago by fbarchard1
Modified:
4 years, 11 months ago
Reviewers:
harryjin, fbarchard
Base URL:
https://chromium.googlesource.com/libyuv/libyuv@master
Target Ref:
refs/heads/master
Project:
libyuv
Visibility:
Public.

Description

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+84 lines, -364 lines) Patch
M README.chromium View 1 chunk +1 line, -1 line 0 comments Download
M include/libyuv/row.h View 5 chunks +0 lines, -15 lines 0 comments Download
M include/libyuv/version.h View 1 chunk +1 line, -1 line 0 comments Download
M source/convert_from_argb.cc View 14 chunks +80 lines, -95 lines 0 comments Download
M source/row_any.cc View 1 chunk +0 lines, -4 lines 0 comments Download
M source/row_common.cc View 2 chunks +0 lines, -44 lines 0 comments Download
M source/row_gcc.cc View 1 chunk +0 lines, -53 lines 0 comments Download
M source/row_neon.cc View 1 chunk +0 lines, -49 lines 0 comments Download
M source/row_neon64.cc View 1 chunk +0 lines, -44 lines 0 comments Download
M source/row_win.cc View 1 chunk +0 lines, -58 lines 0 comments Download
M unit_test/convert_test.cc View 1 chunk +2 lines, -0 lines 0 comments Download

Messages

Total messages: 6 (2 generated)
fbarchard1
4 years, 11 months ago (2016-01-13 00:14:34 UTC) #1
fbarchard
Removing ARGBToUV422 row functions and use ARGBToUV with stride of 0. Slightly less efficient, but ...
4 years, 11 months ago (2016-01-13 00:19:15 UTC) #3
harryjin
lgtm
4 years, 11 months ago (2016-01-13 00:22:22 UTC) #4
fbarchard1
4 years, 11 months ago (2016-01-13 01:05:53 UTC) #6
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
081475b3c86a049c318cb8182e0b12712ff2b40a (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698