Skip to content

fixes sample muting bug for sampletrack - #3591

Merged
BaraMGB merged 2 commits into
LMMS:stable-1.2from
BaraMGB:sampletrackmuting
Jun 8, 2017
Merged

fixes sample muting bug for sampletrack#3591
BaraMGB merged 2 commits into
LMMS:stable-1.2from
BaraMGB:sampletrackmuting

Conversation

@BaraMGB

@BaraMGB BaraMGB commented May 30, 2017

Copy link
Copy Markdown
Contributor

fixes : #3590

@PhysSong

PhysSong commented May 30, 2017

Copy link
Copy Markdown
Member

Good.
Maybe #3133 have caused #3590.
Could we change SampleTrack::updateTcos() into simpler form?

@BaraMGB

BaraMGB commented May 30, 2017

Copy link
Copy Markdown
Contributor Author

Could we change SampleTrack::updateTcos() into simpler form?

Yes, thinking about that brings me to another problem. If we have more than one sample track this would not work.

I'm on it.

@BaraMGB

BaraMGB commented Jun 1, 2017

Copy link
Copy Markdown
Contributor Author

Yes, thinking about that brings me to another problem. If we have more than one sample track this would not work.

That's not true. I updated the PR.

@BaraMGB

BaraMGB commented Jun 4, 2017

Copy link
Copy Markdown
Contributor Author

If no one has objections about this I'll merge in few hours.

@Umcaruje Umcaruje added this to the 1.2.0 milestone Jun 6, 2017

@zonkmachine zonkmachine left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tests fine. Looks good.

@BaraMGB
BaraMGB merged commit 1435717 into LMMS:stable-1.2 Jun 8, 2017
tresf pushed a commit to liushuyu/lmms that referenced this pull request Jun 8, 2017
* fixes sample muting bug for sampletrack

* simplify SampleTrack::updateTcos
@tresf

tresf commented Jun 8, 2017

Copy link
Copy Markdown
Member

FYI, cherry-picked to liushuyu/merge via 8c17b39.

PhysSong pushed a commit to PhysSong/lmms that referenced this pull request Jul 8, 2017
* fixes sample muting bug for sampletrack

* simplify SampleTrack::updateTcos
PhysSong pushed a commit to PhysSong/lmms that referenced this pull request Jul 8, 2017
* fixes sample muting bug for sampletrack

* simplify SampleTrack::updateTcos
sdasda7777 pushed a commit to sdasda7777/lmms that referenced this pull request Jun 28, 2022
* fixes sample muting bug for sampletrack

* simplify SampleTrack::updateTcos
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

sampletrack mutes the playing sample by muting a different one

5 participants