[vlc-devel] [PATCH] filters: deinterlace: add slice threading (yadif2x)

[Romain Vimont]

On 3/31/20 3:59 PM, Francois Cartegnie wrote:
> +            if( i_thread_count > 1 )
> +            {
> +                if( b_worker )
> +                {
> +                    start = slice_height * (threadctx->i_thread_number - 1);
> +                    end = start + slice_height + 2; /* we always need to copy 2 more lines */

Doesn't this mean that two consecutive slices overlap (causing the same 
region of memory being written by two threads)?

> +                }
> +                else /* parent thread */
> +                {
> +                    start = slice_height * (i_thread_count - 1);
> +                    end = dstp->i_visible_lines;
> +                }
Cheers