The next Cocoa port release will be exposing SPU synchronizer functionality to the user. This patch allows compilation of the necessary PCSX-2 Synchronizer code for the Cocoa port.
Can't you define HAVE_LIBSOUNDTOUCH instead?
I would strongly prefer not to just define HAVE_LIBSOUNDTOUCH for metaspu.cpp, because this will change the #include paths in Timestretcher.cpp.
From Timestretcher.cpp --
The reason why I didn't define HAVE_LIBSOUNDTOUCH is because it is kind of a misnomer -- all ports will have it because the entire library is included with DeSmuME.
The bit in Timestretcher.cpp using an external vs internal SoundTouch library confuses me. I'd assume that all ports should be using the one included with DeSmuME. I thought that, as a general case, it would be better to use the included library instead of an external one, since there's probably some reason why it's in there.
It depends on the platform on *nix we usually prefer to use system libraries and shipping private copies of libraries is considered bad practice. It looks like your plan is to use the shipped one instead, I'll apply your patch then.
Sure thing about the system libraries issue. ;-D
Yeah, I'm using the shipped library because I think that is the safer route. I honestly don't know if there are any DeSmuME-specific changes to that code, or possibly any future planned changes.
i dont know if its modified either. are you eager to find out?
Applied, thanks. The patch did not apply though even after converting with dos2unix. How do you diff?
I code in OS X, then use TortoiseSVN in Windows to make the diff. Then I copy the diff back to OS X, where I then upload through the web browser.
Maybe your software is getting confused due to the Windows-style line breaks? I can change the format if need be. My diffs use Windows-style line breaks in UTF-8 format.