Opened 13 years ago Closed 13 years ago #478 closed defect (fixed)2nd & 4th "If" statement in combine list initialization is vague and not precise.
Description
The language of the 2nd and 4th "if" statement in step 3 of subclause 8.3.4.3 is vague and not precise, as it is really difficult to know what "first occurance of the reference picture" actually means.
I propose editorial to change Step 3 of Subclause 8.3.4.3 as follows:
Replace
.....
with
......
And similarly, replace
– If the entry RefPicListL1[ refIdxL1 ] is the first occurance of the reference picture,
.....
with
– If the element RefPicList1[ refIdxL1 ] is not in the reference picture list RefPicList2,
......
Also note that there is a defect in the List name RefPicList0 and RefPicList1, they sometimes appear as RefPicListL0 and RefPicListL1.
While we are at it, we may as well change the following variable names....... as "LC" and "lc" are not good variables as they are used elsewhere for other tools.
PredLCToPredLx to PredL2ToPredLx Change History (7)comment:1 Changed 13 years ago by DefaultCC Plugin
comment:2 Changed 13 years ago by suzukiyoscomment:3 Changed 13 years ago by tktan
Also need to add intitalization of variable refIdxLC to 0 before Step 3 of Subclause 8.3.4.3. comment:4 Changed 13 years ago by suzukiyos
Also then, the following typos in subclause 8.3.4.3 need to be corrected.
In the last two paragraphs in 8.3.4.3 (When refIdxLC is grater/less than....), "num_com_ref_list_active_minus1" should be changed to num_ref_idx_l2_active_minus1. comment:5 Changed 13 years ago by bbross
Combined list was removed in d7 HLS text but the following variables should definitely be renamed:
RefPicListLx should be RefPicListx.
Furthermore, the index cIdx in RefPicListL0/1[cIdx] is very confusing because it is used for color component in most parts of the draft text. It should be replaced by rIdx.
The indexing with parentheses should be replaced with indexing by brackets. comment:6 Changed 13 years ago by bbross
Ticket #481 has been marked as a duplicate of this ticket. comment:7 Changed 13 years ago by bbross
Fixed in JCTVC-I1003 d4. 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
|
Additionally, the following variables which use "lc" may be changed as well;
ref_idx_lc to ref_idx_l2
LcToLx to L2ToLx
And, two terms, that are "lists combination" and "list combination" are used in the WD text and their meaning is same. We would like to choose one of them. "lists combination" is probably better.