Skip to content

Commit 98b6798

Browse files
committed
Fix getPixel error for P210
1 parent b8820a4 commit 98b6798

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

YUViewLib/src/video/yuv/videoHandlerYUV.cpp

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -383,15 +383,15 @@ std::pair<bool, PixelFormatYUV> convertV210PackedToPlanar(const QByteArray &sour
383383

384384
std::pair<bool, PixelFormatYUV> convertP210ToPlanar(const QByteArray &sourceBuffer,
385385
QByteArray &targetBuffer,
386-
const Size curFrameSize)
386+
const Size /*curFrameSize*/)
387387
{
388388
// The output format is 422 10 bit planar interleaved
389389
auto newFormat = PixelFormatYUV(Subsampling::YUV_422, 10, PlaneOrder::YUV, {}, {}, /*uvInterleaved=*/true);
390390
if (targetBuffer.size() < sourceBuffer.size())
391391
targetBuffer.resize(sourceBuffer.size());
392392
auto* pSrc = (uint16_t*)sourceBuffer.data();
393393
auto* pDest = (uint16_t*)targetBuffer.data();
394-
for (auto i = 0u; i < sourceBuffer.size() >> 1; ++i) {
394+
for (auto i = 0; i < sourceBuffer.size() / 2; ++i) {
395395
*pDest++ = *pSrc++ >> 6;
396396
}
397397
return {true, newFormat};
@@ -449,7 +449,7 @@ yuv_t getPixelValueP210(const QByteArray &sourceBuffer,
449449
{
450450
const auto* p = (uint16_t*)sourceBuffer.data();
451451
const auto* pY = p + pixelPos.y() * curFrameSize.width + pixelPos.x();
452-
const auto* pUV = pY + curFrameSize.width * curFrameSize.height + pixelPos.y() * curFrameSize.width + pixelPos.x() / 2;
452+
const auto* pUV = p + curFrameSize.width * curFrameSize.height + pixelPos.y() * curFrameSize.width + pixelPos.x() / 2;
453453
yuv_t res {};
454454
res.Y = *pY >> 6;
455455
res.U = *pUV >> 6;

0 commit comments

Comments
 (0)