
<html><head></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><br><div><div>Am 10.04.2012 um 10:16 schrieb David Fuhrmann:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; ">Hello again,<div><br></div><div>Currently, the zoom functionality is not really working in macosx vout (see my comment in #5997).</div><div>Therefore, I tried to fix that and changed the code to</div><div><br></div><div><div style="margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0px; font: normal normal normal 11px/normal Menlo; ">vout_display_PlacePicture (&place, source, cfg, <span style="color: #c2009e">false</span>);</div></div><div><div style="margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0px; font: normal normal normal 11px/normal Menlo; ">glViewport (place.x, place.y, place.width, place.height);</div></div><div><br></div><div>for every zoom / crop / ar event and</div><div><br></div><div><div style="margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0px; font: normal normal normal 11px/normal Menlo; ">vout_display_PlacePicture (&place, &vd->source, vd->cfg, <span style="color: #c2009e">false</span>);</div></div><div><div><div style="margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0px; font: normal normal normal 11px/normal Menlo; ">glViewport (place.x, place.y, place.width, place.height);</div></div></div><div><br></div><div>for every window resize (method reshape).</div><div><br></div><div>Now the problem is that the picture is always aligned in the lower left corner when in fullscreen, because <span class="Apple-style-span" style="font-family: Menlo; font-size: 11px; ">cfg->display.*</span> is not updated and always on the standard / initial value.</div><div>I thought that <span class="Apple-style-span" style="font-family: Menlo; font-size: 11px; ">vout_display_SendEventDisplaySize (...)</span> would update these values but this does not really work, even not after felix latest vout commit (4a10b45f3147f524ce09bff7b939d09a3f59f108).</div><div><br></div><div>So, can anyone give me a hint what the correct event would be to update cfg?</div><div>Thanks for your help.</div><div><br></div><div>Best regards,</div><div>David</div>

</div></blockquote></div><br><div>Hello,</div><div><br></div><div>I found the solution for the above problem. vout_display_SendEventDisplaySize seems right, but it was never called actually.</div><div><br></div><div>Now I have a partly working zoom functionality, which leads to a second question:</div><div><br></div><div>When VLC is in fullscreen mode and when I set <span class="Apple-style-span" style="font-family: Menlo; font-size: 11px; ">cfg->is_display_filled</span> to false before (press 'o' to switch to original size), zoom works.</div><div>But without that, is_display_filled is true when the vout module gets the zoom event, even in fullscreen. This means that vout_display_PlacePicture does not look at the zoom values at all, because this happens only in vout_display_GetDefaultDisplaySize which is never called (see src/video_output/display.c line 217 and following).</div><div><br></div><div>Is this the desired behavior? I would expect that the core sets is_display_filled to false once we hit 'z' (and the window is in fullscreen), and sets it to true again if we hit 'o' (reset zoom).</div><div>Another option could be that we change vout_display_PlacePicture.</div><div><br></div><div>What do you think? I would be glad to read your opinions.</div><div><br></div><div>Best regards,</div><div>David</div></body></html>