On Sat, 28 Jan 2023 16:24:35 +0000 Jon Turney wrote: > On 22/01/2023 12:05, Takashi Yano via Cygwin-apps wrote: > > On Sat, 21 Jan 2023 14:58:55 +0000 > > Jon Turney wrote: > >> You've made some changes in the configuration here (switching from > >> enabling pulseaudio to oss) > >> > >> I think you've also dropped the winmm patch. > >> > >> I don't understand enough to evaluate the correctness those changes, so > >> it would be nice if you could explain that to me. > > > > Thanks for reviewing. The reason why I turnd off pulseaudio > > is that the latency was unacceptably large, despite the fact > > that pulseaudio is preferred over other sound drivers. > > Due to this, ffplay missynchronized sound with video. > > > > Also, I guess OSS was previously disabled because it does not > > work as expected due to a bug of cygwin: > > https://cygwin.com/pipermail/cygwin-patches/2023q1/012134.html > > and winmm was enabled instead. > > > > Now, the bug has been fixed and SDL2 gets working with OSS. > > Therefore, we don't need winmm support anymore. I think > > it is better not to call winmm directly from cygwin apps. > > > > BTW, I tried to modify SDL2 code to minimize the latency of > > pulseaudio and it succeeded. So, I re-enabled the pulseaudio > > and updated the package. > > > > Could you please review the package again? > > Thanks for this explanation, it all makes sense now. > > Please go ahead. Version updated to 2.26.3. -- Takashi Yano