Fix a bug in vp8_clamp_mv function
Scott fixed the bug in MV clamping function in encoder, which could cause artifacts. Change-Id: Id05f2794c43c31cdd45e66179c8811f3ee452cb9
This commit is contained in:
parent
eafdc5e10a
commit
9c62f94129
@ -637,10 +637,10 @@ void vp8_pick_inter_mode(VP8_COMP *cpi, MACROBLOCK *x, int recon_yoffset, int re
|
|||||||
|
|
||||||
/* adjust mvp to make sure it is within MV range */
|
/* adjust mvp to make sure it is within MV range */
|
||||||
vp8_clamp_mv(&mvp,
|
vp8_clamp_mv(&mvp,
|
||||||
best_ref_mv.as_mv.row - MAX_FULL_PEL_VAL,
|
|
||||||
best_ref_mv.as_mv.row + MAX_FULL_PEL_VAL,
|
|
||||||
best_ref_mv.as_mv.col - MAX_FULL_PEL_VAL,
|
best_ref_mv.as_mv.col - MAX_FULL_PEL_VAL,
|
||||||
best_ref_mv.as_mv.col + MAX_FULL_PEL_VAL);
|
best_ref_mv.as_mv.col + MAX_FULL_PEL_VAL,
|
||||||
|
best_ref_mv.as_mv.row - MAX_FULL_PEL_VAL,
|
||||||
|
best_ref_mv.as_mv.row + MAX_FULL_PEL_VAL);
|
||||||
}
|
}
|
||||||
|
|
||||||
switch (this_mode)
|
switch (this_mode)
|
||||||
|
@ -1878,10 +1878,10 @@ void vp8_rd_pick_inter_mode(VP8_COMP *cpi, MACROBLOCK *x, int recon_yoffset, int
|
|||||||
|
|
||||||
/* adjust mvp to make sure it is within MV range */
|
/* adjust mvp to make sure it is within MV range */
|
||||||
vp8_clamp_mv(&mvp,
|
vp8_clamp_mv(&mvp,
|
||||||
best_ref_mv.as_mv.row - MAX_FULL_PEL_VAL,
|
|
||||||
best_ref_mv.as_mv.row + MAX_FULL_PEL_VAL,
|
|
||||||
best_ref_mv.as_mv.col - MAX_FULL_PEL_VAL,
|
best_ref_mv.as_mv.col - MAX_FULL_PEL_VAL,
|
||||||
best_ref_mv.as_mv.col + MAX_FULL_PEL_VAL);
|
best_ref_mv.as_mv.col + MAX_FULL_PEL_VAL,
|
||||||
|
best_ref_mv.as_mv.row - MAX_FULL_PEL_VAL,
|
||||||
|
best_ref_mv.as_mv.row + MAX_FULL_PEL_VAL);
|
||||||
}
|
}
|
||||||
|
|
||||||
// Check to see if the testing frequency for this mode is at its max
|
// Check to see if the testing frequency for this mode is at its max
|
||||||
|
Loading…
x
Reference in New Issue
Block a user