<div dir="ltr"><br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
It looks like the call to video_format_ScaleCropAr should be removed here.<br></blockquote><div><br></div><div>I don't know if it should be removed, i don't understand what it is trying to do.<br><br></div><br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">


With all the changes mentioned above, it looks like I don't have green<br>
lines anymore on thumbnails and videos. I will do more extensive<br>
testing and send a patch next week.<br>
Also, I couldn't reproduce the issue reported by XilasZ on 720p videos.<br></blockquote><div><br></div><div>Here's what i changed to get some details (<span name="Vittorio Giovara" class="">Vittorio's patch + </span>some logs in swscale<span name="Vittorio Giovara" class="">)</span> : <a href="http://xilasz.free.fr/Android/redesign/swscale-debug.patch">http://xilasz.free.fr/Android/redesign/swscale-debug.patch</a><br>
</div><div>With this, i got this output :<br><br>1280x720 video (thumbnailer/vmem asks for 720x405) :<br><br>swscale filter: 1280x738 (1280x720) chroma: I420 -> 720x405 (720x395) chroma: RGBA extend by 1<br>swscale filter: 1280x738 (1280x720) chroma: I420 -> 720x405 (720x395) chroma: RGBA with scaling using Bicubic (good quality)<br>
swscale filter: i_height=738<br>swscale filter: src 1280x772 (1280x738)<br>swscale filter: dst 720x405 (720x405)<br><br>48x48 video (thumbnailer/vmem asks for 405x405) :<br><br>swscale filter: 48x66 (48x48) chroma: I420 -> 405x405 (405x294) chroma: RGBA extend by 1<br>
swscale filter: 48x66 (48x48) chroma: I420 -> 405x405 (405x294) chroma: RGBA with scaling using Bicubic (good quality)<br>swscale filter: i_height=66<br>swscale filter: src 64x100 (48x66)<br>swscale filter: dst 405x405 (405x405)<br>
<br></div><div>we can see that swscale is initialized with correct size/visible_size on input, and a strange visible_size on output, caused by video_format_ScaleCropAr.<br></div><div>But what's bothering me, is the picture_t swscale receives in Filter(filter_t*, picture_t*) :<br>
swscale filter: src 1280x772 (1280x738)<br>swscale filter: src 64x100 (48x66)<br><br></div><div>where are these coming from ?<br></div></div></div></div>