search

yanglw115 / libyuv

Automatically exported from code.google.com/p/libyuv
BSD 3-Clause "New" or "Revised" License
0 stars 0 forks source link

UYVYToNV12 fails on odd height #528

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago 
d:\src\libyuv\libyuv\unit_test\convert_test.cc(1683): error: Value of: dst_uv[i]
  Actual: '\x9B' (155)
Expected: dst_uv_orig[i]
Which is: '\x1F' (31)
[  FAILED  ] LibYUVConvertTest.TestUYVYToNV12 (356 ms)
[----------] 1 test from LibYUVConvertTest (357 ms total)

[----------] Global test environment tear-down
[==========] 1 test from 1 test case ran. (362 ms total)
[  PASSED  ] 0 tests.
[  FAILED  ] 1 test, listed below:
[  FAILED  ] LibYUVConvertTest.TestUYVYToNV12

YUY2 to NV12 is okay though.
[ RUN      ] LibYUVConvertTest.TestYUY2ToNV12
[       OK ] LibYUVConvertTest.TestYUY2ToNV12 (0 ms)

Original issue reported on code.google.com by fbarch...@chromium.org on 7 Dec 2015 at 7:34

GoogleCodeExporter commented 8 years ago 
The following revision refers to this bug:
  https://chromium.googlesource.com/libyuv/libyuv.git/+/48a919d86e23f767d77c846ed59d21bea3e1ba7d

commit 48a919d86e23f767d77c846ed59d21bea3e1ba7d
Author: Frank Barchard <fbarchard@google.com>
Date: Mon Dec 07 19:39:48 2015

Bug fix for UYVYToNV12 odd height

TBR=harryjin@google.com
BUG=libyuv:528

Review URL: https://codereview.chromium.org/1506973002 .

[modify] 
http://crrev.com/48a919d86e23f767d77c846ed59d21bea3e1ba7d/source/planar_function
s.cc

Original comment by bugdroid1@chromium.org on 7 Dec 2015 at 7:40

GoogleCodeExporter commented 8 years ago 
There were 2 issue
y and uv were swapped
odd width in code that handles odd height, was not supported

Original comment by fbarch...@chromium.org on 7 Dec 2015 at 8:11