Read, never written:

for(Int i=0; i<uiCode; i++)
{

UInt ignore;
READ_CODE(8,ignore,"slice_header_extension_data_byte");

}

This is an extension mechanism, which is already used in the multi-layer extensions (syntax element name in the spec: slice_segment_header_extension_data_byte).

We don't have meaningful semantics defined for profiles specified in Annex A, so there is no point in writing data into this section (except if you want to test that a decoder properly ignores the content according to the spec). But such a test should be a conformance bitstream, not a common HM functionality.

You are comparing different parts of T(Enc/Dec)CAVLC.

TEncCAVLC @ 284:

for (Int chromaQpAdjustmentIndex = 1; chromaQpAdjustmentIndex <= pcPPS->getChromaQpAdjTableSize(); chromaQpAdjustmentIndex++)
{

WRITE_SVLC(pcPPS->getChromaQpAdjTableAt(chromaQpAdjustmentIndex).u.comp.CbOffset, "cb_qp_adjustnemt[i]");
WRITE_SVLC(pcPPS->getChromaQpAdjTableAt(chromaQpAdjustmentIndex).u.comp.CrOffset, "cr_qp_adjustnemt[i]");

}

TDecCAVLC @ 363:

for (Int chromaQpAdjustmentIndex = 1; chromaQpAdjustmentIndex <= (tableSizeMinus1 + 1); chromaQpAdjustmentIndex++)
{

Int cbOffset;
Int crOffset;
READ_SVLC(cbOffset, "cb_qp_adjustnemt[i]");
READ_SVLC(crOffset, "cr_qp_adjustnemt[i]");
pcPPS->setChromaQpAdjTableAt(chromaQpAdjustmentIndex, cbOffset, crOffset);

}

Yes you're right.

Me set the tag incorrectly.

My comment is about HM-dev, revision 4243. There is no reading of that kind there.

In HM-dev, revision 4243, the line numbers for the two highlighted sections are practically the same:

TEncCAVLC @ line 280

https://hevc.hhi.fraunhofer.de/trac/hevc/browser/branches/HM-dev/source/Lib/TLibEncoder/TEncCavlc.cpp?rev=4243

TDecCAVLC @ line 363

https://hevc.hhi.fraunhofer.de/trac/hevc/browser/branches/HM-dev/source/Lib/TLibDecoder/TDecCAVLC.cpp?rev=4243

Agreed again. This is due to stupid automatic parser, who cannot treat "cb_qp_adjustnemt" and "cb_qp_adjustment" as the same %)

Replying to karlsharman:

In HM-dev, revision 4243, the line numbers for the two highlighted sections are practically the same:

TEncCAVLC @ line 280

https://hevc.hhi.fraunhofer.de/trac/hevc/browser/branches/HM-dev/source/Lib/TLibEncoder/TEncCavlc.cpp?rev=4243

TDecCAVLC @ line 363

https://hevc.hhi.fraunhofer.de/trac/hevc/browser/branches/HM-dev/source/Lib/TLibDecoder/TDecCAVLC.cpp?rev=4243

Fixed spelling mistake in debug trace string in r4244.