Changes between Initial Version and Version 2 of Ticket #213


Ignore:
Timestamp:
30 Apr 2019, 12:23:04 (5 years ago)
Author:
ksuehring
Comment:

The fix was merged.

Legend:

Unmodified
Added
Removed
Modified

  • Ticket #213

    • Property Status changed from new to closed
    • Property Resolution changed from to fixed
    • Property Milestone changed from to VTM-4.1

  • Ticket #213 – Description

    initial v2  
    111.) The following code to skip over PCM pixels fails to move the destination pixel pointer pRec before continuing to the next block.  Therefore, the destination pointer will be misaligned with the source and the rest of the row will be corrupted.
    22
     3{{{
    34    if( isPCMFilterDisabled && cl.classIdx == AdaptiveLoopFilter::m_ALF_UNUSED_CLASSIDX && transposeIdx == AdaptiveLoopFilter::m_ALF_UNUSED_TRANSPOSIDX )
    45    {
    56        continue;
    67    }
    7 
     8}}}
    89
    9102.) The code below used to determine if a chroma CU is PCM uses the wrong locations.  The locations (i,j) are chroma pixel locations.  The code below treats them as if they were luma pixel locations instead.  So the CU lookup will be wrong.
    1011
    11 
     12{{{
    1213    Position pos(i, j);
    1314    CodingUnit* cu = isDualTree ? cs.getCU(recalcPosition(nChromaFormat, CH_L, CH_C, pos), CH_C) : cs.getCU(pos, CH_L);
     15}}}