Reuse getSEIDecodingUnitInfoDuiIdx to avoid code duplication

Includes:
- fix syntax element names for tracing
- add validation checks
- move validation check to correct place
- define `enum HrdType` for indexing by `NAL` or `VCL`
- consistently use `bp` variable name for BP SEI objects
- use `static_vector` instead of having separate variable for array size

JVET-AD0057 item 2: disallow the post-filter hint SEI message to be present in both a prefix SEI NALU and a suffix SEI NALU in the same picture unit

Fixed some parts of code that are reported as 'use-of-uninitialized-value'
by MemorySanitizer (using clang++-16)

Sees authored 1 year ago and Nikolay Shostak committed 1 year ago

parseSEImessage() may parse more than one SEI message.
All of them has to be added to the list `m_accessUnitSeiPayLoadTypes`

Sees authored 2 years ago and Nikolay Shostak committed 2 years ago

It is checked by checkConformanceForEDRAP()

Sees authored 2 years ago and Nikolay Shostak committed 2 years ago

Currently for pictures which follow EDRAP m_latestEDRAPPOC is always MAX_INT.

There was similar MR for DRAP POC: jvet/VVCSoftware_VTM!2408

When decoder parses SEI message of unknown type,
it does not add it to list of SEIs.

But then try to access it by calling back() of the list of SEIs.

If list is empty decoder crashes.

Otherwise (if list is not empty) decoder gets wrong SEI message (one of previously added to the list)

Possible fix is to make xReadSEImessage() and parseSEImessage() report if new SEI message object was added to the list