id	summary	reporter	owner	description	type	status	priority	milestone	component	version	resolution	keywords	cc
1260	Filtering process for chroma block edges doesn't match spec	jackh		"Assuming ticket #1254 to be correct, the following piece of code in TComLoopFilter::xEdgeFilterChroma() doesn't match the spec:

{{{
        iQP = ((iQP_P + iQP_Q + 1) >> 1) + chromaQPOffset;
        if      (iQP >= chromaQPMappingTableSize) iQP -=6;
        else if (iQP >= 0) iQP = getScaledChromaQP(iQP, pcPicYuvRec->getChromaFormat());
}}}

It misses out the case when ChromaArrayType != 1 and iQP >= chromaQPMappingTableSize. A possible solution to this might be:

{{{
        iQP = ((iQP_P + iQP_Q + 1) >> 1) + chromaQPOffset;
        if      (iQP >= chromaQPMappingTableSize) {
          if (pcPicYuvRec->getChromaFormat()==CHROMA_420) iQP -=6;
          else if (iQP>51) iQP=51;
        }
        else if (iQP >= 0) iQP = getScaledChromaQP(iQP, pcPicYuvRec->getChromaFormat());
}}}"	defect	closed	minor	HM+RExt-4.2	HM RExt	RExt-4.1 (HM-12.0)	fixed		davidf karlsharman jct-vc@…