Re: [Traverso-devel] Crash on export

[Remon Sijrier]

Hello Martin!

Thanks for pointing out the problem, I only noticed it yesterday, but couldn't 
find the problem, and had no time to investigate further.

Your fix of course works, I was wrong to remove the resizing of the render 
buses in Sheet::set_buffer_size(), thanks a lot for finding the culprit!

I've re-added those, so please make sure you undo your local changes so the 
next git pull doesn't get messed up.

There for sure will be a couple more of these bugs when we move towards a more 
complete implementation of audio io routing, i.e. sending audio data from one 
bus/track to another. (so still very basic but of great value to make Traverso  
a swiss knife among the daw 's :D )

Although current git activity is indeed high, it's mostly small changes to get 
the audio io complete, and I don't like to have work being lost so I'm pushing 
it very frequently to git once it compiles/works for me.

Keep the bug tracking going, it's very useful!

Thanks, Remon

> 
> In first of february's commit called 'More AudioIO stuff'
> (367f744ae22f18ca939bf6f3603399f7b06caec3) was a nice surprise:
> rendering stopped working in a way that results in a application crash
> as since then Traverso tries to write 32768 samples into its 1024
> sized buffer. This attempt usually leads to some crash... the function
> in which it crashes can vary.
> 
> The problem is to be found in the disparition of
> AudioBus::set_buffer_size(). Now Sheet::resize_buffer()  doesn't raise
> the size of the buffers before rendering.
> 
> I temporarly fixed it for the git version from 4 days ago but
> unfortunately the fix don't work anymore with current git :-(
> In the fix i was basically putting the functionality of
> AudioBus::set_buffer_size() directly into Sheet::resize_buffer() but
> that's probably not that clean as AudioBus::set_buffer_size() probably
> disappeared for a good reason.
> 
> So time to find a clean way to make render work again.
> 
> /Martin
>