On Monday, August 31, 2015,  <<a href="mailto:Joerg-Cyril.Hoehle@t-systems.com">Joerg-Cyril.Hoehle@t-systems.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Bruno,<br>
<br>
>If you ever have time, please take a look.<br>
+            wma->dwCurrVideoFrame = wma->dwCurrAudioBlock = 0;<br>
I'm sorry I'd reject this patch because it's missing integration for the MCI_FROM keyword.<br>
<br>
Example using MS' clock.avi:<br>
OPEN clock.avi alias c<br>
WINDOW c State Show ; may still be needed in Wine<br>
PLAY c FROM 2 TO 5 repeat<br>
should open the window and repeat stepping the clock from 3 to 6 o'clock.<br>
(You remember my MCI shell in bugzilla, don't you?)<br>
<br>
You could argue that there's no need to bother for MCI_FROM, because it's not yet supported with audio output (see mmoutput.c). However, play c FROM x to y currently works with video only output (e.g. clock.avi), thus new patches should at least support what's already in place.<br>
<br>
BTW (not in your patch):<br>
         FIXME("Should recompute dwCurrAudioBlock, except unsynchronized sound & video\n");<br>
Typo, should be "expect"<br>
<br>
Supporting sound with MCI_FROM non-zero would be the matter of another patch for a volunteer...<br>
<br>
In summary, I believe that a better patch would either add dwFromVideoFrame to WINE_MCIAVI or remember dwCurrentVideoFrame (but see below) at the entrance of MCIAVI_Player and use that.<br>
<br>
Note that I'd expect any patch to handle this scenario correctly:<br>
Play c FROM 2 TO 5 repeat<br>
Play c TO 8 repeat<br>
In native, the second play would *continue* playing from the current position upto 9 o'clock, then resume from 0 (not 3 o'clock, because 0 is the default FROM value in the second play command).<br>
<br>
Well, that's probably too much for a single patch, since it would likely require keeping the player thread alive, just changing its start/repeat and end positions, rather than restarting a new one.</blockquote><div><br></div><div>Thank you very much for your review, i knew you would be interested. I did use the mcishell to test with other avi files. I'll try to address the issues mentioned and resend, I was not aware of the from argument. </div>