Set skintype on startup

need a feature, suggest here

Re: Set skintype on startup

Postby boomshop » Fri Dec 25, 2015 12:36 pm

Hey guys!

I've commited a possible fix. Brummer, please check if freeing the memory is still sufficient.

Best, Markus.
User avatar
boomshop
 
Posts: 21
Joined: Mon Mar 30, 2015 4:06 pm

Re: Set skintype on startup

Postby boomshop » Fri Dec 25, 2015 12:44 pm

And just for the sake of completeness:

Code: Select all
(gdb) bt full
#0  0x00007fffefc35599 in fftwf_destroy_plan () from /usr/lib/x86_64-linux-gnu/libfftw3f.so.3
No symbol table info available.
#1  0x0000000000578b5a in gx_engine::smbPitchShift::~smbPitchShift() ()
No symbol table info available.
#2  0x00000000005a5971 in gx_engine::GxEngine::~GxEngine() ()
No symbol table info available.
#3  0x00000000005a92cc in gx_engine::GxMachine::~GxMachine() ()
No symbol table info available.
#4  0x0000000000652edc in mainGtk(int, char**) ()
No symbol table info available.
#5  0x00000000004ca2c9 in main ()
No symbol table info available.
User avatar
boomshop
 
Posts: 21
Joined: Mon Mar 30, 2015 4:06 pm

Re: Set skintype on startup

Postby Broomy » Fri Dec 25, 2015 5:48 pm

Checked it out and it worked for me...

Thanks!

Hans
User avatar
Broomy
 
Posts: 147
Joined: Thu Apr 24, 2014 7:53 am

Re: Set skintype on startup

Postby brummer10 » Fri Dec 25, 2015 7:10 pm

boomshop wrote:I've commited a possible fix. Brummer, please check if freeing the memory is still sufficient.


Hi Markus

Yes, it is.
Nice catch.
In addition I've move now the initialization (to zero) of the fftplans from the init call to the class constructor, to avoid a comparison of a un-initialisated var (which seems to was the culprit)

Broomy wrote:Removed the files, pulled the newest version of Guitarix (Layout is getting nicer everytime! ;-)).


The new layout is also the work from Markus. Great eh.

regards
hermann
User avatar
brummer10
Site Admin
 
Posts: 743
Joined: Thu Mar 26, 2009 6:57 pm

Previous

Return to Suggestion Box

Who is online

Users browsing this forum: No registered users and 1 guest

cron