<div dir="ltr">G'day,<div><br></div><div>On 11 January 2015 at 11:44, Andrew Haines <span dir="ltr"><<a href="mailto:andrewd207@aol.com" target="_blank">andrewd207@aol.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div bgcolor="#FFFFFF" text="#000000"><div class="h5"></div></div></blockquote></div><div>> I played with this for a few minutes and came up with this:</div><div>> MPlayerControl1.OnPlaying has to be assigned for </div><div>> MPlayerControl1.Position to work.<br></div><div><br></div><div>Oh.  Yes, I see...   That was by design - it never occurred to me that someone would want to retrieve the .Position without having .OnPlaying hooked up.</div><div><br></div><div>I put the checks in for FOnPlaying in an attempt to minimise "chatter" with mplayer.   In my testing, I observed degraded playback if that chatter was excessive.  In truth though - "excessive" was chatter at frequencies approaching frame by frame querying, not at the frequency Position is currently updated (twice a second).</div><div><br></div><div>Given the current architecture, what I <b>can't</b> do is obtain the position when the user requests it.  I can either constantly query the position (which is what I currently do when FOnPlaying is assigned) or never ask for position.</div><div><br></div><div>As I see it, we've got 3 options:</div><div><br></div><div>1.  Maintain current code</div><div>2.  Always request Position from mplayer</div><div>3.  Add an Options Flag.  Only request Position if either optRequestPosition or FOnPLaying assigned.</div><div><br></div><div>Now you've raised the issue, I'm not in favour of 1.  2 will introduce overheads - but as we've seen when FOnPlaying is assigned, those overheads aren't excessive.  3 preserves my original caution, but perhaps that's unwarranted.</div><div><br></div><div>Now that there's a few items on the TODO list, let me know which you prefer: 2) or 3) and I'll make the changes.  </div><div><br></div><div>I'll get the .FullScreen property in at the same time (but sorry Pierre:  I completely lack the knowledge or time to investigate mplayer working with GL)</div><div><br></div><div>Many thanks for the feedback</div><div><br></div><div>Mike</div><div><br></div><div><br></div></div>