Autres/audacity
1
0
Fork 0
mirror of https://github.com/cookiengineer/audacity synced 2025-06-17 00:20:06 +02:00
Code Issues Releases Wiki Activity

Gale found the reason I'd put in the yield during Nyquist processing,

Browse Source 
so added it back for the Mac only.  See discussion in:

http://bugzilla.audacityteam.org/show_bug.cgi?id=734
This commit is contained in:
lllucius@gmail.com 2014-07-15 02:41:21 +00:00
parent fe241e4d23
commit baa8f8a701
1 changed files with 11 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

17
src/effects/nyquist/Nyquist.cpp
View File

@ -1130,13 +1130,18 @@ void EffectNyquist::OSCallback()
} }
// LLL: STF figured out that yielding while the effect is being applied // LLL: STF figured out that yielding while the effect is being applied
// produces an EXTREME slowdown. It appears that yielding it not // produces an EXTREME slowdown. It appears that yielding is not
// really necessary, so commenting out for now. // really necessary on Linux and Windows.
// //
// Possible issues that may result is the ProgressDialog may // However, on the Mac, the spinning cursor appears during longer
// not get refreshed or may not respond to button clicks... // Nyquist processing and that may cause the user to think Audacity
// just something to watch for. // has crashed or hung. In addition, yielding or not on the Mac
//wxYieldIfNeeded(); // doesn't seem to make much of a difference in execution time.
//
// So, yielding on the Mac only...
#if defined(__WXMAC__)
wxYieldIfNeeded();
#endif
} }
/**********************************************************/ /**********************************************************/