Opened 2 years ago

Closed 2 years ago

#1518 closed defect (fixed)

Minor bug in xInitSPS that wrongly checks virtualBoundariesEnabledFlag() instead of virtualBoundariesPresentFlag()

Reported by: saurabh.puri Owned by:
Priority: trivial Milestone:
Component: VTM Version: VTM-14.2
Keywords: RPR Cc: ksuehring, XiangLi, fbossen, jvet@…

Description

In the following check (under xInitSPS() function in EncLib.cpp)

CHECK(sps.getResChangeInClvsEnabledFlag() && sps.getVirtualBoundariesEnabledFlag(), "when the value of sps_res_change_in_clvs_allowed_flag is equal to 1, the value of sps_virtual_boundaries_present_flag shall be equal to 0");

the text and the condition doesn't match.

It may instead be:

CHECK(sps.getResChangeInClvsEnabledFlag() && sps.getVirtualBoundariesPresentFlag(), "when the value of sps_res_change_in_clvs_allowed_flag is equal to 1, the value of sps_virtual_boundaries_present_flag shall be equal to 0");

According to spec, when RPR is enabled, sps_virtual_boundaries_present_flag shall be 0.

Change history (3)

comment:1 Changed 2 years ago by fbossen

Thank you for the report. Can you submit a merge request with the fix?

comment:2 Changed 2 years ago by saurabh.puri

Thanks. Merge request (2134) submitted for the bug.

comment:3 Changed 2 years ago by fbossen

  • Resolution set to fixed
  • Status changed from new to closed
Note: See TracTickets for help on using tickets.