Opened 13 years ago Closed 12 years ago #563 closed defect (fixed)GolombDecode for ALF params does not match text ge(v)
Description
TDecCAVLC.cpp, line 922 is the signed Golomb decode function which calls the unsigned decode function xReadEpExGolomb on line 2197. Void TDecCavlc::xReadEpExGolomb( UInt& ruiSymbol, UInt uiCount ) { UInt uiSymbol = 0; UInt uiBit = 1; while( uiBit ) { xReadFlag( uiBit ); uiSymbol += uiBit << uiCount++; } uiCount--; while( uiCount-- ) { xReadFlag( uiBit ); uiSymbol += uiBit << uiCount; // This shift is wrong } ruiSymbol = uiSymbol; return; }
Text:
q = −1
}
}
if( codeNum != 0 ) {
}
The function xGolombDecode on line 922 should just be re-written to match the text, and not call xReadEpExGolomb (which can just be removed all together). Change History (3)comment:1 Changed 13 years ago by DefaultCC Plugin
comment:2 Changed 12 years ago by bbross
comment:3 Changed 12 years ago by bbross
This is fixed in JCTVC-I1003 d5 (v6). 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
|
Actually this is a text issue but the text is fixed in JCTVC-I0603_CDText, so HM and CD text are matched.