From: Tim E. R. <ter...@ro...> - 2010-04-28 22:35:37
|
Somehow I knew Chris might be listening... Hi Chris, Robert. Robert, was wondering about this section in the patch: if( msg.message == WM_TIMER ) { plugin->dispatcher (plugin, effEditIdle, 0, 0, NULL, 0); if( needIdle ) { if(plugin) plugin->dispatcher (plugin, 53, 0, 0, NULL, 0); } Should there be a "if(plugin)" before that "plugin->dispatcher()" on the second line? Or is that no problem? Still, I have had no trouble with it, as is. Anyway, I think the patch would *immediately* benefit Rosegarden and MusE users, and other dssi hosts. Tim. On April 28, 2010 06:25:15 am Robert Jonsson wrote: > Hi Chris, > > I think it is the patch I sent you last year which enables > audioMasterNeedIdle support. I gather Tim has found a lot of plugs > that need it. > It would be good if you could take a look at including it in the > original package. > If it needs reworking we can probably fix that, just let me know. > > Regards, > Robert > > 2010/4/28 Chris Cannam <ca...@al...>: > > On Wed, Apr 28, 2010 at 2:04 AM, Tim E. Real <ter...@ro...> wrote: > >> ** Also, currently a small patch to dssi-vst is required, so that > >> synths and effects may update their gui controls! > >> For now, just ask about that here. > > > > Pardon? Can you give details? > > > > > > Chris > > > > ------------------------------------------------------------------------- > >----- _______________________________________________ > > Lmuse-user mailing list > > Lmu...@li... > > https://lists.sourceforge.net/lists/listinfo/lmuse-user |