Opened 4 years ago

Last modified 4 years ago

#1163 new defect

Undefined variable in Derivation process for collocated motion vectors

Reported by: laurent.birtz@… Owned by:
Priority: minor Milestone: D10
Component: Text Version: D10 (L1003) v33
Keywords: Cc: bbross, wjhan, jct-vc@…

Description

In "Derivation process for collocated motion vectors":

Inputs to this process are:

  • a reference index refIdxLX, with X being 0 or 1.

The arrays predFlagLXCol[ x ][ y ], mvLXCol[ x ][ y ], and refIdxLXCol[ x ][ y ] are set equal to the corresponding arrays of the collocated picture specified by colPic, PredFlagLX[ x ][ y ], MvLX[ x ][ y ], and RefIdxLX[ x ][ y ], respectively, with X being the value of X this process is invoked for.

If predFlagL0Col[ xColPb ][ yColPb ] is equal to 0, ...

Problem:

If the process is invoked with refIdxL1, then X=1 and predFlagL0Col isn't defined since predFlagLXCol is assigned only for "X being the value of X this process is invoked for".

Is my understanding correct?

Change History (2)

comment:1 Changed 4 years ago by DefaultCC Plugin

  • Cc bbross wjhan jct-vc@… added

comment:2 Changed 4 years ago by yk

I think you are right and changing

The arrays predFlagLXCol[ x ][ y ], mvLXCol[ x ][ y ], and refIdxLXCol[ x ][ y ] are set equal to the corresponding arrays of the collocated picture specified by colPic, PredFlagLX[ x ][ y ], MvLX[ x ][ y ], and RefIdxLX[ x ][ y ], respectively, with X being the value of X this process is invoked for.

to

The arrays predFlagL0Col[ x ][ y ], mvL0Col[ x ][ y ], and refIdxL0Col[ x ][ y ] are set equal to PredFlagL0[ x ][ y ], MvL0[ x ][ y ], and RefIdxL0[ x ][ y ], respectively, of the collocated picture specified by colPic, and the arrays predFlagL1Col[ x ][ y ], mvL1Col[ x ][ y ], and refIdxL1Col[ x ][ y ] are set equal to PredFlagL1[ x ][ y ], MvL1[ x ][ y ], and RefIdxL1[ x ][ y ], respectively, of the collocated picture specified by colPic..

should resolve this issue.

BR, YK

Note: See TracTickets for help on using tickets.

This list contains all users that will be notified about changes made to this ticket.

These roles will be notified: Reporter, Owner, Subscriber, Participant

  • Benjamin Bross(Subscriber)
  • David Flynn(Always)
  • jct-vc@…(Subscriber)
  • Karl Sharman(Always)
  • Karsten Suehring(Always)
  • laurent.birtz@…(Reporter)
  • Woo-Jin Han(Subscriber)
  • Ye-Kui Wang(Participant)