Opened 14 years ago Closed 14 years ago #116 closed enhancement (fixed)Unnecessary testing of MVs
Description
As reported by Woo-Jin:
In TEncSearch::xTZSearch:
In the above code, m_acMvPredictors were initialized by m_cMvFieldA, m_cMvFieldB and m_cMvFieldC, however, those variables were not initialized thus they are equal to the zero vector. Since the zero-vector testing is done again just after the above code segment, I think that the above code is not needed. (Or it can be initialized correctly to use the actual motion vector candidates used in the AMVP, but the difference seems negligible from quick testing) I prefer simply to delete the above code or set bTestOtherPredictedMV to be zero as:
From "const Bool bTestOtherPredictedMV = 1" to "const Bool bTestOtherPredictedMV = 0"
The above change should reduce the encoding time as you indicated. Change History (4)comment:1 Changed 14 years ago by fbossen
comment:2 Changed 13 years ago by davidf
Updating component after adding WD (Text) tickets comment:3 Changed 13 years ago by davidf
comment:4 Changed 13 years ago by davidf
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
|
Fixed in r423