<div dir="ltr"><div><div>Hi,<br><br></div>I see it on the <a href="https://wiki.winehq.org/Developer_Hints#Using_only_C89-compliant_code">https://wiki.winehq.org/Developer_Hints#Using_only_C89-compliant_code</a> page.<br></div><div>It mentions variables being earlier, but that seemed to slow things down in this instance.<br></div><div><br></div>Some comments I left in because I'm not sure if they're the right thing to do. Will convert.<br><br><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, May 20, 2022 at 3:46 AM Bernhard Kölbl <<a href="mailto:besentv@gmail.com">besentv@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="auto">Hi Zlice,<div dir="auto"><br></div><div dir="auto">thanks for your patch. Unfortunately you're missing out on quite a few requirements for this to get accepted. I'd recommend to refactor your submission using this guide as reference: <a href="https://wiki.winehq.org/Submitting_Patches" target="_blank">https://wiki.winehq.org/Submitting_Patches</a> and to resend it afterwards.</div><div dir="auto"><br></div><div dir="auto">E.g. you left in C++ comments, which aren't allowed.</div><div dir="auto"><br></div><div dir="auto">Thanks,</div><div dir="auto">Bernhard</div><div dir="auto"><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">Zlice Z <<a href="mailto:zlice555@gmail.com" target="_blank">zlice555@gmail.com</a>> schrieb am Fr., 20. Mai 2022, 09:22:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div><div><div><div><div>Sent a patch yesterday that was mediocre and barely helps.<br><br></div>commit <a href="https://github.com/wine-mirror/wine/commit/5144b27661fcd6705353d832e0383085f8afe842#" rel="noreferrer" target="_blank">https://github.com/wine-mirror/wine/commit/5144b27661fcd6705353d832e0383085f8afe842#</a><br><br></div>removed flushing from parser, which after commit bisecting turns out to be the cause of Fallout 3 radio chugging every new song.<br><br></div>- re-introduced parser flushing (self explanatory, commit said it's not worth it but it helps here)<br><br></div>- moved some variable initialization in wm_reader.c to after initial conditional returns. these were in a switch-case before in the middle of the function and it seems to be part of the stuttering, wasn't moved in the mentioned commit though<br><br></div>- commented out and left my name by some EOS sets. they used to be checked for EOS 'events' but those references have been changed to 'buffer' and idk what the right action is. wine seemed to randomly seg fault on exit with leaving them on though. suspect they incorrectly guess and set 'end of stream' ?<br></div>
</blockquote></div>
</blockquote></div>