Reply to thread

Message

<blockquote data-quote="flyinj" data-source="post: 646846" data-attributes="member: 99224">This Patch attempts to fix two things:1) The Problem with MPC decoder filter reporting wrong aspect ratio2) A potential bug in the handling crop settings by Normal zoom modeConcerning 1):&nbsp;The solution was using ar information that is available from internal demuxer using the g_player singleton.&nbsp; &nbsp; This is being done in the InternalPresenter method where arwidth arheight arguments are replaced byg_player.getvideostreamformat information&nbsp; &nbsp; Although one doesnt make good usage of available API using this method, one doesnt rely on other ppls decoder software&nbsp; passing our own data correctly back to ourselfes <img src="data:image/gif;base64,R0lGODlhAQABAIAAAAAAAP///yH5BAEAAAAALAAAAAABAAEAAAIBRAA7" class="smilie smilie--sprite smilie--sprite7" alt=":p" title="Stick Out Tongue&nbsp; &nbsp; :p" loading="lazy" data-shortname=":p" />Concerning 2):&nbsp;The previous behavior depended on Pixelratio, which appeared to me as the Pixelratio of the screen Pixels (apparently determined&nbsp;&nbsp; by that nice interface which displays a shloud-be-square on the screen). But it shloud depend of the Aspect ratio of Image Pixels, which it does now...I made this against Revision 26135</blockquote>

[QUOTE="flyinj, post: 646846, member: 99224"] This Patch attempts to fix two things: 1) The Problem with MPC decoder filter reporting wrong aspect ratio 2) A potential bug in the handling crop settings by Normal zoom mode Concerning 1): The solution was using ar information that is available from internal demuxer using the g_player singleton. This is being done in the InternalPresenter method where arwidth arheight arguments are replaced by g_player.getvideostreamformat information Although one doesnt make good usage of available API using this method, one doesnt rely on other ppls decoder software passing our own data correctly back to ourselfes :P Concerning 2): The previous behavior depended on Pixelratio, which appeared to me as the Pixelratio of the screen Pixels (apparently determined by that nice interface which displays a shloud-be-square on the screen). But it shloud depend of the Aspect ratio of Image Pixels, which it does now... I made this against Revision 26135 [/QUOTE]

Verification