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 2 years ago and Nikolay Shostak committed 2 years 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

- move VPS and related classes into VideoParameterSet.h/cpp
- move SPS and related classes into SequenceParameterSet.h/cpp
- move PPS and related classes into PictureParameterSet.h/cpp
- rename ProfileLevelTier.h/cpp to ProfileTierLevel.h/cpp and move ProfileTierLevel class there

No other changes except for minor formatting/comment cleanup

Avoid duplicating variables and functions for each reference list

Virtual boundaries can be set in the encoder config without enabling GDR. The change makes sure, GDR constraints are conditioned on GDR being enabled.

Should also fix memory allocation issues with PicHeader.

Roman Guskov authored 2 years ago and Nikolay Shostak committed 2 years ago

Call pop_front() to prevent infinite loop