<html><head></head><body><div class="gmail_quote">Le 26 juin 2017 19:20:12 GMT+02:00, Steve Lhomme <robux4@videolabs.io> a écrit :<blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
<pre class="k9mail">---<br /> modules/video_filter/deinterlace/deinterlace.c | 6 +-----<br /> 1 file changed, 1 insertion(+), 5 deletions(-)<br /><br />diff --git a/modules/video_filter/deinterlace/deinterlace.c b/modules/video_filter/deinterlace/deinterlace.c<br />index 2be5422c3a..57df63c813 100644<br />--- a/modules/video_filter/deinterlace/deinterlace.c<br />+++ b/modules/video_filter/deinterlace/deinterlace.c<br />@@ -155,7 +155,7 @@ static void SetFilterMethod( filter_t *p_filter, const char *mode, bool pack )<br /> p_sys->b_half_height = false;<br /> p_sys->b_use_frame_history = false;<br /> <br />- if ( !strcmp( mode, "auto" ) )<br />+ if ( !strcmp( mode, "auto" ) || !strcmp( mode, "x" ) )<br /> {<br /> p_sys->i_mode = DEINTERLACE_X;<br /> }<br />@@ -206,10 +206,6 @@ static void SetFilterMethod( filter_t *p_filter, const char *mode, bool pack )<br /> " for high depth format", mode );<br /> return SetFilterMethod(p_filter, "auto", pack);<br /> }<br />- else if( !strcmp( mode, "x" ) )<br />- {<br />- p_sys->i_mode = DEINTERLACE_X;<br />- }<br /> else if( !strcmp( mode, "phosphor" ) )<br /> {<br /> p_sys->i_mode = DEINTERLACE_PHOSPHOR;</pre></blockquote></div><br clear="all">I don't see the point. It makes it harder to change the default algorithm, and the compiler should be capable of vectoring the code if it helps.<br>
<br>
Also typo.<br>
-- <br>
Rémi Denis-Courmont<br>
Typed on an inconvenient virtual keyboard</body></html>