Page 2 of 2

Re: Empty Env/LFO tab in SF2 player

Posted: Wed Mar 19, 2014 1:28 am
by midi-pascal
Hi,

I followed the directives in the wiki and did all steps to pull my request.
But the Git repo says 'midi-pascal wants to merge 1 commit into LMMS:stable-0.4 from midi-pascal:stable-0.4'
I'm a bit worried by the 'stable-0.4' :?
Did I failed somewhere in the process?
Let me know: I'm willing to learn ;)

Thanks to you all, I fill welcome 8-)

Re: Empty Env/LFO tab in SF2 player

Posted: Wed Mar 19, 2014 1:40 am
by diiz
No no, you did everything right... our branch name is just badly outdated :lol:

Should probably bug Toby about it at some point, but he's busy enough as it is...

Re: Empty Env/LFO tab in SF2 player

Posted: Wed Mar 19, 2014 2:01 am
by midi-pascal
Yeah!

Now, hope my fix will be accepted :?
I am concious that it is a quick fix but not the ideal solution to the problem.
The perfect one would be to have two models for the plugin window: One with the ENV/LFO tab and the m_ssView member for all the plugins and one without them for the MIDI-only things.
But this is a major code change and I'm pretty sure you do not want to go there for now with 1.0.0 soon to be born.
Just to let you know that I am a developer in real life too :D

Thanks a lot for you very kind help, and I am eager to contribute again and more to the Lmms glory!
Kind regards.

Re: Empty Env/LFO tab in SF2 player

Posted: Wed Mar 19, 2014 7:36 am
by diiz
I don't see why it wouldn't be merged, Toby just has to review the request first.