2016-01-17 21:26 GMT+01:00 Dan MacDonald <[email protected]>:
> I still have the mixer drawing death occur frequently under the latest git
> b038232e104aecff6f40d7d805f3d16efc069190
>
> Yeah, there's definitely a problem but I can't find a repeatable way of
provoking it, so it takes time, if anyone has found a surefire way of
getting it to happen that would be good. I'll keep at it.
Regards,
Robert
> On Sun, Jan 17, 2016 at 7:59 PM, Robert Jonsson <[email protected]>
> wrote:
>
>> Heyho,
>>
>> 2016-01-17 20:04 GMT+01:00 Tim E. Real <[email protected]>:
>>
>>> On January 17, 2016 12:29:56 PM Robert Jonsson wrote:
>>> > Hi guys,
>>> >
>>> > In recent builds, if you have a mixer open (which seems to be left open
>>> > whenever it's been opened, another bug) and open a new project or click
>>> > New, muse crashes in the Rack when clearing away the old song.
>>> >
>>> > void EffectRack::updateContents()
>>> >
>>> > {
>>> >
>>> > for (int i = 0; i < PipelineDepth; ++i) {
>>> >
>>> > QString name = track->efxPipe()->name(i); <----- name(i)
>>> > is not retrievable.
>>> >
>>> >
>>> > Looking at the data with the debugger I wonder if there may be data
>>> > corruption.
>>> >
>>> >
>>> > I tried going back through the commits and the problem starts to occur
>>> > in commit 2c011b9 (mixer makeover)
>>> >
>>> >
>>> https://github.com/muse-sequencer/muse/commit/2c1c79ca7cfabe20a4c427cd8af423
>>> > ccd874ad99
>>> >
>>> >
>>> > I can't really see in that commit what might cause it though. Any
>>> ideas?
>>> >
>>> >
>>> > Regards,
>>> > Robert
>>>
>>> Pushed fix:
>>> Previously I had changed all deletes to deleteLater
>>> because delete was causing a crash when
>>> instrument was changed on midi strip.
>>>
>>
>> Alright thanks
>>
>>
>>>
>>> But now these changes are pretty extensive.
>>>
>>> I'm not able to test that instrument bug because
>>> the strip disappears.
>>>
>>
>> Are we talking about the mixer and the strips vanishing in certain
>> situations?
>> Does it happen all the time for you? Could you briefly describe what you
>> do when it happens?
>> I need to test more but I don't believe I have found a way to
>> conclusively provoke it.
>>
>>
>>> BTW What happened to the strip expander handles?
>>>
>>
>> Woops, I saw there were stuff for that but I didn't know I had removed
>> something. I'll backtrack.
>>
>> /Robert
>>
>>
>>> void AudioMixerApp::clearAndDelete()
>>> {
>>> if (DEBUG_MIXER)
>>> printf("clearAndDelete\n");
>>> StripList::iterator si = stripList.begin();
>>> for (; si != stripList.end(); ++si)
>>> {
>>> mixerLayout->removeWidget(*si);
>>> //(*si)->deleteLater(); // <<<<<<<<<<<< Reverted to delete.
>>> delete (*si);
>>> }
>>>
>>> stripList.clear();
>>>
>>> ...
>>> }
>>>
>>>
>>> Tim.
>>>
>>>
>>> ------------------------------------------------------------------------------
>>> Site24x7 APM Insight: Get Deep Visibility into Application Performance
>>> APM + Mobile APM + RUM: Monitor 3 App instances at just $35/Month
>>> Monitor end-to-end web transactions and take corrective actions now
>>> Troubleshoot faster and improve end-user experience. Signup Now!
>>> http://pubads.g.doubleclick.net/gampad/clk?id=267308311&iu=/4140
>>> _______________________________________________
>>> Lmuse-developer mailing list
>>> [email protected]
>>> https://lists.sourceforge.net/lists/listinfo/lmuse-developer
>>>
>>
>>
>>
>> ------------------------------------------------------------------------------
>> Site24x7 APM Insight: Get Deep Visibility into Application Performance
>> APM + Mobile APM + RUM: Monitor 3 App instances at just $35/Month
>> Monitor end-to-end web transactions and take corrective actions now
>> Troubleshoot faster and improve end-user experience. Signup Now!
>> http://pubads.g.doubleclick.net/gampad/clk?id=267308311&iu=/4140
>> _______________________________________________
>> Lmuse-developer mailing list
>> [email protected]
>> https://lists.sourceforge.net/lists/listinfo/lmuse-developer
>>
>>
>
------------------------------------------------------------------------------
Site24x7 APM Insight: Get Deep Visibility into Application Performance
APM + Mobile APM + RUM: Monitor 3 App instances at just $35/Month
Monitor end-to-end web transactions and take corrective actions now
Troubleshoot faster and improve end-user experience. Signup Now!
http://pubads.g.doubleclick.net/gampad/clk?id=267308311&iu=/4140
_______________________________________________
Lmuse-developer mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/lmuse-developer
