mirror of
https://github.com/FFmpeg/FFmpeg.git
synced 2025-01-24 13:56:33 +02:00
hevc: deobfuscate slice/tile boundary handling for DBF
Use named constants instead of magic numbers, avoid using variables with inverse meaning from what their name implies.
This commit is contained in:
parent
5501971578
commit
70211539a3
@ -1420,9 +1420,7 @@ do {
|
||||
}
|
||||
}
|
||||
if (!s->sh.disable_deblocking_filter_flag) {
|
||||
ff_hevc_deblocking_boundary_strengths(s, x0, y0, log2_trafo_size,
|
||||
lc->slice_or_tiles_up_boundary,
|
||||
lc->slice_or_tiles_left_boundary);
|
||||
ff_hevc_deblocking_boundary_strengths(s, x0, y0, log2_trafo_size);
|
||||
if (s->pps->transquant_bypass_enable_flag &&
|
||||
lc->cu.cu_transquant_bypass_flag)
|
||||
set_deblocking_bypass(s, x0, y0, log2_trafo_size);
|
||||
@ -1448,11 +1446,8 @@ static int hls_pcm_sample(HEVCContext *s, int x0, int y0, int log2_cb_size)
|
||||
const uint8_t *pcm = skip_bytes(&lc->cc, (length + 7) >> 3);
|
||||
int ret;
|
||||
|
||||
if (!s->sh.disable_deblocking_filter_flag) {
|
||||
ff_hevc_deblocking_boundary_strengths(s, x0, y0, log2_cb_size,
|
||||
lc->slice_or_tiles_up_boundary,
|
||||
lc->slice_or_tiles_left_boundary);
|
||||
}
|
||||
if (!s->sh.disable_deblocking_filter_flag)
|
||||
ff_hevc_deblocking_boundary_strengths(s, x0, y0, log2_cb_size);
|
||||
|
||||
ret = init_get_bits(&gb, pcm, length);
|
||||
if (ret < 0)
|
||||
@ -2100,9 +2095,7 @@ static int hls_coding_unit(HEVCContext *s, int x0, int y0, int log2_cb_size)
|
||||
intra_prediction_unit_default_value(s, x0, y0, log2_cb_size);
|
||||
|
||||
if (!s->sh.disable_deblocking_filter_flag)
|
||||
ff_hevc_deblocking_boundary_strengths(s, x0, y0, log2_cb_size,
|
||||
lc->slice_or_tiles_up_boundary,
|
||||
lc->slice_or_tiles_left_boundary);
|
||||
ff_hevc_deblocking_boundary_strengths(s, x0, y0, log2_cb_size);
|
||||
} else {
|
||||
if (s->sh.slice_type != I_SLICE)
|
||||
lc->cu.pred_mode = ff_hevc_pred_mode_decode(s);
|
||||
@ -2185,9 +2178,7 @@ static int hls_coding_unit(HEVCContext *s, int x0, int y0, int log2_cb_size)
|
||||
return ret;
|
||||
} else {
|
||||
if (!s->sh.disable_deblocking_filter_flag)
|
||||
ff_hevc_deblocking_boundary_strengths(s, x0, y0, log2_cb_size,
|
||||
lc->slice_or_tiles_up_boundary,
|
||||
lc->slice_or_tiles_left_boundary);
|
||||
ff_hevc_deblocking_boundary_strengths(s, x0, y0, log2_cb_size);
|
||||
}
|
||||
}
|
||||
}
|
||||
@ -2265,9 +2256,6 @@ static void hls_decode_neighbour(HEVCContext *s, int x_ctb, int y_ctb,
|
||||
int ctb_addr_rs = s->pps->ctb_addr_ts_to_rs[ctb_addr_ts];
|
||||
int ctb_addr_in_slice = ctb_addr_rs - s->sh.slice_addr;
|
||||
|
||||
int tile_left_boundary, tile_up_boundary;
|
||||
int slice_left_boundary, slice_up_boundary;
|
||||
|
||||
s->tab_slice_address[ctb_addr_rs] = s->sh.slice_addr;
|
||||
|
||||
if (s->pps->entropy_coding_sync_enabled_flag) {
|
||||
@ -2287,25 +2275,25 @@ static void hls_decode_neighbour(HEVCContext *s, int x_ctb, int y_ctb,
|
||||
|
||||
lc->end_of_tiles_y = FFMIN(y_ctb + ctb_size, s->sps->height);
|
||||
|
||||
lc->boundary_flags = 0;
|
||||
if (s->pps->tiles_enabled_flag) {
|
||||
tile_left_boundary = x_ctb > 0 &&
|
||||
s->pps->tile_id[ctb_addr_ts] == s->pps->tile_id[s->pps->ctb_addr_rs_to_ts[ctb_addr_rs - 1]];
|
||||
slice_left_boundary = x_ctb > 0 &&
|
||||
s->tab_slice_address[ctb_addr_rs] == s->tab_slice_address[ctb_addr_rs - 1];
|
||||
tile_up_boundary = y_ctb > 0 &&
|
||||
s->pps->tile_id[ctb_addr_ts] == s->pps->tile_id[s->pps->ctb_addr_rs_to_ts[ctb_addr_rs - s->sps->ctb_width]];
|
||||
slice_up_boundary = y_ctb > 0 &&
|
||||
s->tab_slice_address[ctb_addr_rs] == s->tab_slice_address[ctb_addr_rs - s->sps->ctb_width];
|
||||
if (x_ctb > 0 && s->pps->tile_id[ctb_addr_ts] != s->pps->tile_id[s->pps->ctb_addr_rs_to_ts[ctb_addr_rs - 1]])
|
||||
lc->boundary_flags |= BOUNDARY_LEFT_TILE;
|
||||
if (x_ctb > 0 && s->tab_slice_address[ctb_addr_rs] != s->tab_slice_address[ctb_addr_rs - 1])
|
||||
lc->boundary_flags |= BOUNDARY_LEFT_SLICE;
|
||||
if (y_ctb > 0 && s->pps->tile_id[ctb_addr_ts] != s->pps->tile_id[s->pps->ctb_addr_rs_to_ts[ctb_addr_rs - s->sps->ctb_width]])
|
||||
lc->boundary_flags |= BOUNDARY_UPPER_TILE;
|
||||
if (y_ctb > 0 && s->tab_slice_address[ctb_addr_rs] != s->tab_slice_address[ctb_addr_rs - s->sps->ctb_width])
|
||||
lc->boundary_flags |= BOUNDARY_UPPER_SLICE;
|
||||
} else {
|
||||
tile_left_boundary =
|
||||
tile_up_boundary = 1;
|
||||
slice_left_boundary = ctb_addr_in_slice > 0;
|
||||
slice_up_boundary = ctb_addr_in_slice >= s->sps->ctb_width;
|
||||
if (!ctb_addr_in_slice > 0)
|
||||
lc->boundary_flags |= BOUNDARY_LEFT_SLICE;
|
||||
if (ctb_addr_in_slice < s->sps->ctb_width)
|
||||
lc->boundary_flags |= BOUNDARY_UPPER_SLICE;
|
||||
}
|
||||
lc->slice_or_tiles_left_boundary = (!slice_left_boundary) + (!tile_left_boundary << 1);
|
||||
lc->slice_or_tiles_up_boundary = (!slice_up_boundary + (!tile_up_boundary << 1));
|
||||
lc->ctb_left_flag = ((x_ctb > 0) && (ctb_addr_in_slice > 0) && tile_left_boundary);
|
||||
lc->ctb_up_flag = ((y_ctb > 0) && (ctb_addr_in_slice >= s->sps->ctb_width) && tile_up_boundary);
|
||||
|
||||
lc->ctb_left_flag = ((x_ctb > 0) && (ctb_addr_in_slice > 0) && !(lc->boundary_flags & BOUNDARY_LEFT_TILE));
|
||||
lc->ctb_up_flag = ((y_ctb > 0) && (ctb_addr_in_slice >= s->sps->ctb_width) && !(lc->boundary_flags & BOUNDARY_UPPER_TILE));
|
||||
lc->ctb_up_right_flag = ((y_ctb > 0) && (ctb_addr_in_slice+1 >= s->sps->ctb_width) && (s->pps->tile_id[ctb_addr_ts] == s->pps->tile_id[s->pps->ctb_addr_rs_to_ts[ctb_addr_rs+1 - s->sps->ctb_width]]));
|
||||
lc->ctb_up_left_flag = ((x_ctb > 0) && (y_ctb > 0) && (ctb_addr_in_slice-1 >= s->sps->ctb_width) && (s->pps->tile_id[ctb_addr_ts] == s->pps->tile_id[s->pps->ctb_addr_rs_to_ts[ctb_addr_rs-1 - s->sps->ctb_width]]));
|
||||
}
|
||||
|
@ -740,8 +740,13 @@ typedef struct HEVCLocalContext {
|
||||
PredictionUnit pu;
|
||||
NeighbourAvailable na;
|
||||
|
||||
uint8_t slice_or_tiles_left_boundary;
|
||||
uint8_t slice_or_tiles_up_boundary;
|
||||
#define BOUNDARY_LEFT_SLICE (1 << 0)
|
||||
#define BOUNDARY_LEFT_TILE (1 << 1)
|
||||
#define BOUNDARY_UPPER_SLICE (1 << 2)
|
||||
#define BOUNDARY_UPPER_TILE (1 << 3)
|
||||
/* properties of the boundary of the current CTB for the purposes
|
||||
* of the deblocking filter */
|
||||
int boundary_flags;
|
||||
} HEVCLocalContext;
|
||||
|
||||
typedef struct HEVCContext {
|
||||
@ -970,9 +975,7 @@ void ff_hevc_luma_mv_mvp_mode(HEVCContext *s, int x0, int y0,
|
||||
void ff_hevc_set_qPy(HEVCContext *s, int xC, int yC, int xBase, int yBase,
|
||||
int log2_cb_size);
|
||||
void ff_hevc_deblocking_boundary_strengths(HEVCContext *s, int x0, int y0,
|
||||
int log2_trafo_size,
|
||||
int slice_or_tiles_up_boundary,
|
||||
int slice_or_tiles_left_boundary);
|
||||
int log2_trafo_size);
|
||||
int ff_hevc_cu_qp_delta_sign_flag(HEVCContext *s);
|
||||
int ff_hevc_cu_qp_delta_abs(HEVCContext *s);
|
||||
void ff_hevc_hls_filter(HEVCContext *s, int x, int y);
|
||||
|
@ -584,10 +584,9 @@ static int boundary_strength(HEVCContext *s, MvField *curr,
|
||||
}
|
||||
|
||||
void ff_hevc_deblocking_boundary_strengths(HEVCContext *s, int x0, int y0,
|
||||
int log2_trafo_size,
|
||||
int slice_or_tiles_up_boundary,
|
||||
int slice_or_tiles_left_boundary)
|
||||
int log2_trafo_size)
|
||||
{
|
||||
HEVCLocalContext *lc = &s->HEVClc;
|
||||
MvField *tab_mvf = s->ref->tab_mvf;
|
||||
int log2_min_pu_size = s->sps->log2_min_pu_size;
|
||||
int log2_min_tu_size = s->sps->log2_min_tb_size;
|
||||
@ -616,11 +615,11 @@ void ff_hevc_deblocking_boundary_strengths(HEVCContext *s, int x0, int y0,
|
||||
bs = boundary_strength(s, curr, curr_cbf_luma,
|
||||
top, top_cbf_luma, top_refPicList, 1);
|
||||
if (!s->sh.slice_loop_filter_across_slices_enabled_flag &&
|
||||
(slice_or_tiles_up_boundary & 1) &&
|
||||
lc->boundary_flags & BOUNDARY_UPPER_SLICE &&
|
||||
(y0 % (1 << s->sps->log2_ctb_size)) == 0)
|
||||
bs = 0;
|
||||
else if (!s->pps->loop_filter_across_tiles_enabled_flag &&
|
||||
(slice_or_tiles_up_boundary & 2) &&
|
||||
lc->boundary_flags & BOUNDARY_UPPER_TILE &&
|
||||
(y0 % (1 << s->sps->log2_ctb_size)) == 0)
|
||||
bs = 0;
|
||||
if (bs)
|
||||
@ -675,11 +674,11 @@ void ff_hevc_deblocking_boundary_strengths(HEVCContext *s, int x0, int y0,
|
||||
bs = boundary_strength(s, curr, curr_cbf_luma,
|
||||
left, left_cbf_luma, left_refPicList, 1);
|
||||
if (!s->sh.slice_loop_filter_across_slices_enabled_flag &&
|
||||
(slice_or_tiles_left_boundary & 1) &&
|
||||
lc->boundary_flags & BOUNDARY_LEFT_SLICE &&
|
||||
(x0 % (1 << s->sps->log2_ctb_size)) == 0)
|
||||
bs = 0;
|
||||
else if (!s->pps->loop_filter_across_tiles_enabled_flag &&
|
||||
(slice_or_tiles_left_boundary & 2) &&
|
||||
lc->boundary_flags & BOUNDARY_LEFT_TILE &&
|
||||
(x0 % (1 << s->sps->log2_ctb_size)) == 0)
|
||||
bs = 0;
|
||||
if (bs)
|
||||
|
Loading…
x
Reference in New Issue
Block a user